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

syntax: correct the Rand::rand call to select enum variants in #[deriving(Rand)]. #7101

Closed
wants to merge 3 commits into from

Conversation

huonw
Copy link
Member

@huonw huonw commented Jun 13, 2013

Previously, this was not a global call, and so when #[deriving(Rand)]
was in any module other than the top-level one, it failed (unless there
was a use std; in scope).

Also, fix a minor inconsistency between uints and u32s for this piece
of code.

…ving(Rand)].

Previously, this was not a global call, and so when `#[deriving(Rand)]`
was in any module other than the top-level one, it failed (unless there
was a `use std;` in scope).

Also, fix a minor inconsistency between uints and u32s for this piece
of code.
@huonw
Copy link
Member Author

huonw commented Jun 13, 2013

@pnkfelix done. The reason I didn't add an issue number is because it seems that very few (none that I could find) of the other tests xfail'd because of an extern mod had an issue number... so I opened #7103.

@graydon graydon closed this Jun 13, 2013
@pnkfelix
Copy link
Member

@huonw ping me after you rebase this to master. Sorry for the inconvenience; but at least there's a good chance this won't happen again. :)

flip1995 pushed a commit to flip1995/rust that referenced this pull request Apr 22, 2021
Add flat_map_option lint

changelog: Add flat_map_option lint

Closes rust-lang#2241
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.

5 participants