search

pmem / pmemstream

Other
9 stars 13 forks source link

Fix klocwork issues #212

Closed lukaszstolarczuk closed 2 years ago

lukaszstolarczuk commented 2 years ago

This change is Reviewable

codecov-commenter commented 2 years ago

Codecov Report

Merging #212 (1d35a05) into master (d138c8f) will increase coverage by 0.18%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #212      +/-   ##
==========================================
+ Coverage   91.47%   91.65%   +0.18%     
==========================================
  Files           9        9              
  Lines         774      779       +5     
==========================================
+ Hits          708      714       +6     
+ Misses         66       65       -1
Flag Coverage Δ
tests_clang_debug_cpp17 92.54% <100.00%> (+0.04%) :arrow_up:
tests_gcc_debug_cpp17 90.06% <100.00%> (+1.04%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/libpmemstream.c 88.19% <100.00%> (-0.11%) :arrow_down:
src/region.c 93.61% <0.00%> (+1.41%) :arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more