-
Notifications
You must be signed in to change notification settings - Fork 13k
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
Make the NonZero*
methods const fn
#53331
Comments
|
Ah I thought there were more trivial methods. This issue is only acctionable for |
Hi, I'd like to help out on this. I would appreciate if some one gave me pointers on what needs to be accomplished. |
Steps:
|
@MagnumOpus21 you may take a look at #53697. These are the steps you have to do in order to add |
@cyres I will do that sir. Was busy with my internship. |
@MagnumOpus21 If you need help, don't hesitate to ping me here, in Discord (Cyres#8322) or on StackOverflow in the rust chat room. |
Thank you 😄 @cyres |
and undo the const -> fn changes in https://github.com/rust-lang/rust/pull/53315/files#diff-a4a060a3c734b42317d2b9136d68918eL186
This requires changing applicable methods on
NonZero*
fromfn
toconst fn
, adding the#[rustc_const_unstable="const_nonzero*_methods"]
attribute to all of them and adding some tests ensuring that the methods are not const fn on stable rust.The text was updated successfully, but these errors were encountered: