Change order of evaluation to avoid underflow error

[mdvillagra]

Change order of evaluation to avoid underflow error

Description

When the number of limbs in an unsigned integer is 1, like in the babybear field, the line

let index = 2 * i - NUM_LIMBS + 1;

panicks. This is because all variables are usize and when i=0 then NUM_LIMBS is subtracted and gives -1.

To solve the problem mentioned above is suffices to change the evaluation order.

Type of change

• Bug fix

[codecov-commenter]

Codecov Report

Merging #575 (8553708) into main (da688cd) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #575   +/-   ##
=======================================
  Coverage   95.50%   95.50%           
=======================================
  Files         112      112           
  Lines       19006    19006           
=======================================
  Hits        18152    18152           
  Misses        854      854           

Files Changed	Coverage Δ
math/src/unsigned_integer/element.rs	97.43% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[MauroToscano]

LGTM. Thanks for the fix

[pablodeymo]

👍