Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor decimal128 and decimal128_fast #683

Merged
merged 19 commits into from
Jun 27, 2024
Merged

Refactor decimal128 and decimal128_fast #683

merged 19 commits into from
Jun 27, 2024

Conversation

mborland
Copy link
Member

@mborland mborland commented Jun 27, 2024

Closes: #678
Closes: #679
Closes: #680
Closes: #681
Closes: #618

Copy link

codecov bot commented Jun 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.0%. Comparing base (8d6d270) to head (ba77d62).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##           develop    #683     +/-   ##
=========================================
- Coverage     99.0%   99.0%   -0.0%     
=========================================
  Files          225     230      +5     
  Lines        16424   16442     +18     
  Branches      1792    1792             
=========================================
+ Hits         16253   16265     +12     
- Misses         171     177      +6     
Files Coverage Δ
include/boost/decimal/decimal128.hpp 98.0% <100.0%> (-<0.1%) ⬇️
include/boost/decimal/decimal128_fast.hpp 96.6% <100.0%> (-<0.1%) ⬇️
include/boost/decimal/decimal32.hpp 99.8% <100.0%> (ø)
include/boost/decimal/decimal32_fast.hpp 97.3% <100.0%> (ø)
include/boost/decimal/decimal64.hpp 99.3% <100.0%> (ø)
include/boost/decimal/decimal64_fast.hpp 98.7% <100.0%> (-<0.1%) ⬇️
include/boost/decimal/detail/div_impl.hpp 100.0% <ø> (ø)
include/boost/decimal/detail/emulated128.hpp 97.4% <100.0%> (-0.2%) ⬇️
include/boost/decimal/detail/mul_impl.hpp 92.9% <100.0%> (-0.4%) ⬇️
include/boost/decimal/detail/sub_impl.hpp 97.4% <100.0%> (+1.1%) ⬆️

... and 12 files with indirect coverage changes


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 8d6d270...ba77d62. Read the comment docs.

@mborland mborland merged commit 84d31e4 into develop Jun 27, 2024
62 checks passed
@mborland mborland deleted the dec128_basics branch June 27, 2024 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant