Make `std::env::{Args, ArgsOs}` `Send` and `Sync`
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 119k
- Forks
- 16.1k
- PR merge metrics
- PR metrics pending
Description
#48004 requests to make these 2 types !Send and !Sync and #48005 implements it.
From this comment, at that time, they were really not Send and Sync, but now, it seems both Unix and Wndows are defining Args as:
struct Args {
args: vec::IntoIter<OsString>
}
which has no reason to be !Send and !Sync, I am not sure about that cloudabi platform though given that I didn't find it in the source code.
If the Std allocates for these types on all the platforms(i.e., basically a Vec<OsString>), we can probably make them Send and Sync
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by tracing the std::env::{Args, ArgsOs} definitions for Unix and Windows, then investigate the cloudabi implementation mentioned in the issue. Done means confirming whether these types can be Send and Sync on every supported platform and updating the relevant standard-library behavior accordingly.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- operating-systems
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100