Skip to content

crypto: Rename ModArith to MontgomeryArith - #1666

Open
chfast wants to merge 1 commit into
masterfrom
crypto/montgomery-arith-rename
Open

crypto: Rename ModArith to MontgomeryArith#1666
chfast wants to merge 1 commit into
masterfrom
crypto/montgomery-arith-rename

crypto: Rename ModArith to MontgomeryArith

c2d6511
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 20, 2026 in 0s

97.72% (+0.00%) compared to 9344010

View this Pull Request on Codecov

97.72% (+0.00%) compared to 9344010

Details

Codecov Report

❌ Patch coverage is 82.05128% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 97.72%. Comparing base (9344010) to head (c2d6511).

Files with missing lines Patch % Lines
test/unittests/modarith_test.cpp 69.56% 7 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1666   +/-   ##
=======================================
  Coverage   97.72%   97.72%           
=======================================
  Files         171      171           
  Lines       15636    15636           
  Branches     3616     3616           
=======================================
  Hits        15280    15280           
  Misses        269      269           
  Partials       87       87           
Components Coverage Δ
core 96.11% <100.00%> (ø)
tooling 91.92% <ø> (ø)
tests 99.80% <73.07%> (ø)
Files with missing lines Coverage Δ
lib/evmone_precompiles/ecc.hpp 97.24% <100.00%> (ø)
lib/evmone_precompiles/modarith.hpp 100.00% <100.00%> (ø)
lib/evmone_precompiles/secp256r1.cpp 100.00% <100.00%> (ø)
test/unittests/secp256k1_test.cpp 100.00% <100.00%> (ø)
test/unittests/modarith_test.cpp 89.04% <69.56%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.