Skip to content

libcore: Export core::from_str::FromStr from core::prelude#6185

Merged
bors merged 2 commits into
rust-lang:incomingfrom
gifnksm:prelude-from_str
May 3, 2013
Merged

libcore: Export core::from_str::FromStr from core::prelude#6185
bors merged 2 commits into
rust-lang:incomingfrom
gifnksm:prelude-from_str

Conversation

@gifnksm

@gifnksm gifnksm commented May 2, 2013

Copy link
Copy Markdown
Contributor

core::prelude re-exports core::to_str::ToStr, but doesn't re-export core::from_str::FromStr.
That is inconsistent.

@thestinger

Copy link
Copy Markdown
Contributor

I don't think you need the $[cfg(stage0)] attributes on the FromStr use statements. AFAIK changes to core don't need to worry about the stages because the snapshot compiler starts off by compiling libcore.

@gifnksm

gifnksm commented May 3, 2013

Copy link
Copy Markdown
Contributor Author

@thestinger yes, you are right. I've misunderstood the stages.
I removed extra #[cfg(stage0)]s in e34a7ec.

bors added a commit that referenced this pull request May 3, 2013
`core::prelude`  re-exports `core::to_str::ToStr`, but doesn't re-export `core::from_str::FromStr`.
That is inconsistent.
@bors bors closed this May 3, 2013
@bors bors merged commit e34a7ec into rust-lang:incoming May 3, 2013
@gifnksm gifnksm deleted the prelude-from_str branch May 4, 2013 02:15
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.

3 participants