Skip to content

fix asan failures#902

Merged
grisumbras merged 1 commit intoboostorg:developfrom
grisumbras:fix/asan-failures
Jun 8, 2023
Merged

fix asan failures#902
grisumbras merged 1 commit intoboostorg:developfrom
grisumbras:fix/asan-failures

Conversation

@grisumbras
Copy link
Copy Markdown
Member

Changes to array sizes in tests are due to asan bug.

@grisumbras grisumbras self-assigned this Jun 2, 2023
@grisumbras grisumbras added the Bug label Jun 2, 2023
@codecov
Copy link
Copy Markdown

codecov bot commented Jun 2, 2023

Codecov Report

Merging #902 (3956bdb) into develop (be759c5) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #902   +/-   ##
========================================
  Coverage    99.25%   99.25%           
========================================
  Files           72       72           
  Lines         6959     6968    +9     
========================================
+ Hits          6907     6916    +9     
  Misses          52       52           
Impacted Files Coverage Δ
include/boost/json/basic_parser_impl.hpp 98.58% <100.00%> (+<0.01%) ⬆️
include/boost/json/impl/value_stack.ipp 100.00% <100.00%> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update be759c5...3956bdb. Read the comment docs.

@cppalliance-bot
Copy link
Copy Markdown

Changes to array sizes in tests are due to asan bug.
@grisumbras grisumbras force-pushed the fix/asan-failures branch from 7647494 to 3956bdb Compare June 7, 2023 11:13
@cppalliance-bot
Copy link
Copy Markdown

@grisumbras grisumbras marked this pull request as ready for review June 7, 2023 14:36
@grisumbras grisumbras merged commit 3956bdb into boostorg:develop Jun 8, 2023
@grisumbras grisumbras deleted the fix/asan-failures branch June 8, 2023 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants