pmem / pmemstream

Other
9 stars 13 forks source link

[helpers] simplify concurrent_type structure #242

Closed KFilipek closed 2 years ago

KFilipek commented 2 years ago

Since concurrent_type implements the same features as ranged, there is no need to do it twice.


This change is Reviewable

codecov-commenter commented 2 years ago

Codecov Report

Merging #242 (5030c2a) into master (55f925a) will decrease coverage by 1.36%. The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #242      +/-   ##
==========================================
- Coverage   88.84%   87.48%   -1.37%     
==========================================
  Files           9        9              
  Lines         726      727       +1     
  Branches      127      127              
==========================================
- Hits          645      636       -9     
- Misses         48       57       +9     
- Partials       33       34       +1     
Flag Coverage Δ
tests_gcc_debug_cpp17 87.48% <ø> (-1.37%) :arrow_down:

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

Impacted Files Coverage Δ
pmemstream/src/region.c 79.06% <0.00%> (-8.44%) :arrow_down:
pmemstream/src/libpmemstream.c 87.46% <0.00%> (+0.25%) :arrow_up:

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