-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
Refactor: Mv OsStr
/OsStrExt
/Path
to core, OsString
/OsStringExt
/PathBuf
to alloc
#129411
base: master
Are you sure you want to change the base?
Conversation
rustbot has assigned @Mark-Simulacrum. Use |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
☔ The latest upstream changes (presumably #129443) made this pull request unmergeable. Please resolve the merge conflicts. |
43db21c
to
3317d1b
Compare
Thanks so much for working on this, this is going to make working on backtrace so much more tolerable. |
library/alloc/src/ffi/mod.rs
Outdated
@@ -89,3 +89,18 @@ pub use self::c_str::{FromVecWithNulError, IntoStringError, NulError}; | |||
|
|||
#[unstable(feature = "c_str_module", issue = "112134")] | |||
pub mod c_str; | |||
|
|||
#[stable(feature = "rust1", since = "1.0.0")] |
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.
Presumably only the re-exports in libstd should have these, and the alloc/core originals should be marked unstable for now.
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.
Thanks, I've marked them as unstable for now.
Someone else interested in it could stablise it.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
OsStr
/OsStrExt
to core, OsString
/OsStringExt
to allocOsStr
/OsStrExt
/Path
to core, OsString
/OsStringExt
to alloc
This comment has been minimized.
This comment has been minimized.
OsStr
/OsStrExt
/Path
to core, OsString
/OsStringExt
to allocOsStr
/OsStrExt
/Path
to core, OsString
/OsStringExt
/PathBuf
to alloc
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
☔ The latest upstream changes (presumably #129063) made this pull request unmergeable. Please resolve the merge conflicts. |
1907594
to
46dde7e
Compare
This comment has been minimized.
This comment has been minimized.
…xt` to alloc Signed-off-by: Jiahao XU <Jiahao_XU@outlook.com>
46dde7e
to
9550e18
Compare
The job Click to see the possible cause of the failure (guessed by this bot)
|
☔ The latest upstream changes (presumably #129962) made this pull request unmergeable. Please resolve the merge conflicts. |
I'm giving up on this PR as I don't have much time to work on it lately. And I'm not sure I can resolve the concern raised by the team. For rust-lang/libs-team#399, I might have to think of a different solution, like marking Arc/Rc of being fundamental |
I may be able to take up at least fixing up this PR, minus fixing the design issues with the other change. I think that at least lowering the requirements for these in an unstable way is fine. |
For rust-lang/libs-team#399