Skip to content

Removal of the const_generics feature #24

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

Closed
emlaufer opened this issue Sep 17, 2021 · 0 comments · Fixed by #25
Closed

Removal of the const_generics feature #24

emlaufer opened this issue Sep 17, 2021 · 0 comments · Fixed by #25

Comments

@emlaufer
Copy link

The const_generics feature has recently been removed in favor of adt_const_params, which breaks unstable builds the most recent nightly. I believe the fix is as simple as replacing #![cfg_attr(feature = "unstable", feature(const_generics))] with #![cfg_attr(feature = "unstable", feature(adt_const_params))]

hawkw added a commit to hawkw/volatile that referenced this issue Mar 31, 2022
Currently, the "unstable" feature no longer compiles on recent nightly
Rust toolchains, because it enables the `const_generics` feature flag.
Since const generics have stabilized, this feature flag is no longer
necessary, and results in a compilation error on recent toolchains.

This commit removes `#[feature(const_generics)]` from the crate. Now,
the "unstable" feature builds on recent nightly compilers. Additionally,
we can remove the "unstable" feature flag from the impls for converting
volatile arrays to slices, as that impl no longer requires an unstable
feature.

In order to make the tests pass on stable Rust after removing the
unstable feature flag from the impls for arrays, I had to modify the
examples so that they no longer use other unstable methods, such as
`copy_from_slice`. I think the new examples still get the point across,
although they're maybe not quite as nice...

Fixes rust-osdev#24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant