-
Notifications
You must be signed in to change notification settings - Fork 56
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
Adopt core
naming conventions for bigint methods
#693
Comments
I'd like to punt on this for the v0.6 release. We can always add the new names incrementally, deprecate the old ones, and remove them in a v0.7 release. While I'm here though, I would also like to note some inconsistencies in modular inverse naming: we have The Edit: hmm, I now notice this is used in a few places to make Edit again: these concerns probably apply to the |
Is this one going to be delayed until v0.7? |
Yes, per my comment immediately above yours |
Sorry, I wasn't sure what "punt on this" meant. |
Aah sorry, that is perhaps a little jargony. But yes, the plan is to introduce new names some time after the stable v0.6 release and deprecate the old ones. |
See also: #537
The upcoming
bigint_helper_methods
feature ofcore
defines method names and type signatures which it would be good to adopt here as well where it makes sense.Here's a name bikeshedding thread, if anyone has any opinions on these names: https://internals.rust-lang.org/t/naming-bigint-helper-methods-bike-shedding/21688
The text was updated successfully, but these errors were encountered: