`BigInt` fixes corresponding to algebra#372 #94

mmagician · 2022-01-10T15:50:18Z

Description

Companion to algebra's PR 372: arkworks-rs/algebra#372

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

Targeted PR against correct branch (master)
Linked to Github issue with discussion and accepted design OR have an explanation in the PR that describes this work.
Wrote unit tests
Updated relevant documentation in the code
Added a relevant changelog entry to the Pending section in CHANGELOG.md
Re-reviewed Files changed in the Github PR explorer

Replace BigInteger([...]) with BigInt::new([...])

bls12_377/src/fields/tests.rs

bls12_381/src/fields/tests.rs

bn254/src/fields/tests.rs

bls12_381/src/fields/tests.rs

weikengchen · 2022-01-10T18:50:56Z

Fingers crossed on the nightly :)

mmagician added 5 commits January 10, 2022 14:17

Fix bigint constructor after the BigInt const generics refactor

a316611

Replace BigInteger([...]) with BigInt::new([...])

Use local version of bls12-381

93af5f8

Fix remaining tests

7cba273

Clean up imports formatting

a4abcdf

Fix bls12-381 tests

84cf8b1

mmagician requested review from Pratyush and weikengchen January 10, 2022 17:51