Skip to content

DecimalLiteralRepresentation simplification#4758

Merged
bors merged 1 commit into
masterfrom
unknown repository
Nov 7, 2019
Merged

DecimalLiteralRepresentation simplification#4758
bors merged 1 commit into
masterfrom
unknown repository

Conversation

@ghost

@ghost ghost commented Oct 31, 2019

Copy link
Copy Markdown

Remove recalculation of literal value.

changelog: none

Remove recalculation of literal value.
@flip1995

Copy link
Copy Markdown
Member

@bors r+

@bors

bors commented Oct 31, 2019

Copy link
Copy Markdown
Contributor

📌 Commit 4e78547 has been approved by flip1995

@bors

bors commented Oct 31, 2019

Copy link
Copy Markdown
Contributor

⌛ Testing commit 4e78547 with merge ce5d22e...

bors added a commit that referenced this pull request Oct 31, 2019
`DecimalLiteralRepresentation` simplification

Remove recalculation of literal value.

changelog: none
@bors

bors commented Oct 31, 2019

Copy link
Copy Markdown
Contributor

💔 Test failed - checks-travis

@ghost

ghost commented Nov 1, 2019

Copy link
Copy Markdown
Author

The stdarch (stdsimd) build is failing. Should we remove it from our integration tests or wait for them to fix it?

@flip1995

flip1995 commented Nov 1, 2019

Copy link
Copy Markdown
Member

Yes, it's not a Clippy bug.

flip1995 added a commit to flip1995/rust-clippy that referenced this pull request Nov 7, 2019
`DecimalLiteralRepresentation` simplification

Remove recalculation of literal value.

changelog: none
flip1995 added a commit to flip1995/rust-clippy that referenced this pull request Nov 7, 2019
`DecimalLiteralRepresentation` simplification

Remove recalculation of literal value.

changelog: none
bors added a commit that referenced this pull request Nov 7, 2019
Rollup of 4 pull requests

Successful merges:

 - #4697 (restriction lint for `std::process::exit`)
 - #4757 (Fix Deprecated lints don't expand)
 - #4758 (`DecimalLiteralRepresentation` simplification)
 - #4769 (don't warn on CRLF in `with_newline` lints)

Failed merges:

r? @ghost
bors added a commit that referenced this pull request Nov 7, 2019
Rollup of 4 pull requests

Successful merges:

 - #4697 (restriction lint for `std::process::exit`)
 - #4757 (Fix Deprecated lints don't expand)
 - #4758 (`DecimalLiteralRepresentation` simplification)
 - #4769 (don't warn on CRLF in `with_newline` lints)

Failed merges:

r? @ghost
@bors

bors commented Nov 7, 2019

Copy link
Copy Markdown
Contributor

⌛ Testing commit 4e78547 with merge 426c05a...

bors added a commit that referenced this pull request Nov 7, 2019
`DecimalLiteralRepresentation` simplification

Remove recalculation of literal value.

changelog: none
@bors

bors commented Nov 7, 2019

Copy link
Copy Markdown
Contributor

☀️ Test successful - checks-travis, status-appveyor
Approved by: flip1995
Pushing 426c05a to master...

@bors bors merged commit 4e78547 into rust-lang:master Nov 7, 2019
@ghost ghost deleted the dec_lit_20191031 branch December 26, 2019 07:28
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.

2 participants