-
Notifications
You must be signed in to change notification settings - Fork 101
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
Add associated constants of type Self
to Field
and PrimeField
#94
Conversation
Prior to merging this, I want to confirm that the major implementors of the
|
@str4d the @RustCrypto crates are all fine. They support both |
I confirmed that |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK
0dd4723
to
58741b7
Compare
e8845d6
to
e9744d0
Compare
e9744d0
to
620be67
Compare
Rebased on |
We now require that the type implementing `Field`, and its particular values for these constants, can be constructed in a const context. Once upon a time this might have been onerous, but it should now be a reasonable requirement given our MSRV of 1.56.0. Closes #87.
620be67
to
6bf93ee
Compare
Force-pushed to fix a documentation lint. |
We now require that the type implementing
Field
, and its particular values for these constants, can be constructed in a const context. Once upon a time this might have been onerous, but it should now be a reasonable requirement given our MSRV of 1.56.0.Closes #87.