Skip to content
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

Replace Option<T> with MaybeUninit<T> #192

Merged

Conversation

Kixunil
Copy link
Contributor

@Kixunil Kixunil commented Oct 9, 2021

This avoids unneeded tag in Option and addresses FIXME comment.

This avoids unneeded tag in `Option` and addresses FIXME comment.
@hniksic
Copy link

hniksic commented Nov 1, 2021

Perhaps this should bump the minimally supported version from 1.27.2 to 1.36.0:

diff --git a/.travis.yml b/.travis.yml
index ae60bd2..67b5787 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -1,7 +1,7 @@
 language: rust
 matrix:
   include:
-    - rust: 1.27.2
+    - rust: 1.36.0
       script:
         - cargo test --tests
     - rust: stable
diff --git a/README.md b/README.md
index 5729f4e..7f43e8c 100644
--- a/README.md
+++ b/README.md
@@ -15,7 +15,7 @@ as well as anything that requires non-const function calls to be computed.
 
 ## Minimum supported `rustc`
 
-`1.27.2+`
+`1.36.0+`
 
 This version is explicitly tested in CI and may only be bumped in new minor versions. Any changes to the supported minimum version will be called out in the release notes.
 

@Kixunil
Copy link
Contributor Author

Kixunil commented Nov 2, 2021

Oh, I'm not sure why I didn't notice that. Perhaps "stable since 1.36" confused me?

Copy link
Contributor

@KodrAus KodrAus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @Kixunil! This looks good to me. It's clearer with MaybeUninit what our expectations of that field are.

@KodrAus KodrAus merged commit 59f776e into rust-lang-nursery:master Apr 20, 2022
@Kixunil Kixunil deleted the change-option-to-maybeuninit branch April 20, 2022 14:41
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 this pull request may close these issues.

None yet

3 participants