Skip to content

Update Clippy #76034

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 175 commits into from
Aug 29, 2020
Merged

Update Clippy #76034

merged 175 commits into from
Aug 29, 2020

Conversation

flip1995
Copy link
Member

Bi-weekly Clippy update, as per the new policy.

r? @Manishearth

pnadon and others added 30 commits August 8, 2020 07:53
Renamed remaining references to "undef" to "uninit" when referring to Miri.

Impacted directories are:

- src/librustc_codegen_llvm/consts.rs
- src/librustc_middle/mir/interpret/
- src/librustc_middle/ty/print/pretty.rs
- src/librustc_mir/
- src/tools/clippy/clippy_lints/src/consts.rs

Upon building Miri based on the new changes it was verified that no changes needed to be made with the Miri project.

Related issue rust-lang#71193
Miri: Renamed "undef" to "uninit"

Renamed remaining references to "undef" to "uninit" when referring to Miri.

Impacted directories are:

- `src/librustc_codegen_llvm/consts.rs`
- `src/librustc_middle/mir/interpret/`
- `src/librustc_middle/ty/print/pretty.rs`
- `src/librustc_mir/`
- `src/tools/clippy/clippy_lints/src/consts.rs`

Upon building Miri based on the new changes it was verified that no changes needed to be made with the Miri project.

Related issue rust-lang#71193
unnecessary-mut-passed: make lint message say if fn is a function or a method

changelog: refine "unnecessary-mut-passed" lint message
Update Clippy

Biweekly Clippy update (2 days late, since I wanted to wait for rust-lang#75098)

r? @Manishearth
Co-authored-by: Philipp Krones <hello@philkrones.com>
giraffate and others added 11 commits August 27, 2020 23:37
Don't emit a lint when `.to_string()` on anything that is not `self`
…ay, r=ebroto

Fix FP in `to_string_in_display`

Don't emit a lint when `.to_string()` on anything that is not `self`

Fix rust-lang#5967

changelog: Fix FP in `to_string_in_display` when calling `.to_string()` on anything that is not `self`
Co-authored-by: Eduardo Broto <ebroto@tutanota.com>
Co-authored-by: Eduardo Broto <ebroto@tutanota.com>
Co-authored-by: Eduardo Broto <ebroto@tutanota.com>
Fix FP in `new_ret_no_self`: trigger in trait def instead of impl block

Lint in trait def instead of impl block.

Fixes: rust-lang#5435

changelog: none
@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 28, 2020
@Manishearth
Copy link
Member

@bors r+

@bors
Copy link
Collaborator

bors commented Aug 28, 2020

📌 Commit 575e27d has been approved by Manishearth

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 28, 2020
@Dylan-DPC-zz
Copy link

@bors p=never (changes lock files better to be run separately)

@Dylan-DPC-zz
Copy link

lol

@bors p=0 rollup=never

@bors
Copy link
Collaborator

bors commented Aug 29, 2020

⌛ Testing commit 575e27d with merge fe8ab8a...

@bors
Copy link
Collaborator

bors commented Aug 29, 2020

☀️ Test successful - checks-actions, checks-azure
Approved by: Manishearth
Pushing fe8ab8a to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Aug 29, 2020
@bors bors merged commit fe8ab8a into rust-lang:master Aug 29, 2020
@cuviper cuviper added this to the 1.48.0 milestone Nov 17, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.