-
Notifications
You must be signed in to change notification settings - Fork 302
Blog for Rust 1.50.0 #771
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
Blog for Rust 1.50.0 #771
Conversation
What about |
Co-authored-by: Camelid <camelidcamel@gmail.com> Co-authored-by: Jacob Lifshay <programmerjake@gmail.com> Co-authored-by: lzutao <15225902+lzutao@users.noreply.github.com>
posts/2021-02-11-Rust-1.50.0.md
Outdated
which variant is valid (they don't drop any old value on assignment), so | ||
safe Rust has limited this to `Copy` types only, which never `Drop`. Of | ||
course, that's also true of `ManuallyDrop`, so now Rust 1.50 allows safe | ||
assignments to these fields as well. |
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.
Can we split the first sentence of this paragraph up? I think it's carrying a lot of weight right now in a relatively short space and would be nice to add some breaks for readers.
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.
I split and rearranged this a bit.
Co-authored-by: Mark Rousskov <mark.simulacrum@gmail.com>
Co-authored-by: Camelid <camelidcamel@gmail.com>
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.
Looking good modulo one small nit - I think maybe just cutting the sentence early is best.
r? @rust-lang/release @rust-lang/core