Skip to content

Conversation

@basic-calculus
Copy link
Contributor

No description provided.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you make this task_rng? (Creating a StdRng is needlessly expensive.)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That makes sense. However, I'll try to do better. I'll create a new pull request that hunts through test cases for StdRng::new() and replaces it with task_rng(). NVM I was confused.

bors added a commit that referenced this pull request Mar 23, 2014
@bors bors closed this Mar 24, 2014
@bors bors merged commit 8feb2dd into rust-lang:master Mar 24, 2014
@basic-calculus basic-calculus deleted the cleanup-test-warnings branch March 24, 2014 03:34
JohnTitor pushed a commit to JohnTitor/rust that referenced this pull request Aug 30, 2022
…=jonas-schievink

fix: Allow leading `|` in more pattern positions

fixes rust-lang/rust-analyzer#12894, fixes rust-lang/rust-analyzer#13094

Oddly, the leading `|` token does not end up inside the `OR_PAT` node, since `pattern_top_r` consumes it first. This is a preexisting issue in match arms though, so I didn't fix it here.
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jul 25, 2024
fix [`excessive_precision`] suggestions on floats written in scientific notation

fixes rust-lang#12954

changelog: fix [`excessive_precision`] suggestions on float literal written in scientific notation
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