Skip to content

remove redundant clones, found by clippy - #68252

Merged
bors merged 1 commit into
rust-lang:masterfrom
matthiaskrgr:redundant_clones
Jan 15, 2020
Merged

remove redundant clones, found by clippy#68252
bors merged 1 commit into
rust-lang:masterfrom
matthiaskrgr:redundant_clones

Conversation

@matthiaskrgr

Copy link
Copy Markdown
Member

No description provided.

@oli-obk

Copy link
Copy Markdown
Contributor

r? @oli-obk
@bors r+ rollup

@bors

bors commented Jan 15, 2020

Copy link
Copy Markdown
Collaborator

📌 Commit 312c3a0 has been approved by oli-obk

@rust-highfiverust-highfive assigned oli-obk and unassigned eddybJan 15, 2020
@borsbors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Jan 15, 2020
Dylan-DPC-zz pushed a commit to Dylan-DPC-zz/rust that referenced this pull request Jan 15, 2020
…i-obk
remove redundant clones, found by clippy
bors added a commit that referenced this pull request Jan 15, 2020
Rollup of 6 pull requests
Successful merges:
- #68123 (Implement Cursor for linked lists. (RFC 2570).)
- #68212 (Suggest to shorten temporary lifetime during method call inside generator)
- #68232 (Optimize size/speed of Unicode datasets)
- #68236 (Add some regression tests)
- #68237 (Account for `Path`s in `is_suggestable_infer_ty`)
- #68252 (remove redundant clones, found by clippy)
Failed merges:
r? @ghost
@bors
bors merged commit 312c3a0 into rust-lang:masterJan 15, 2020
@eddyb

Copy link
Copy Markdown
Contributor

This reminds me, are we linting .clone() on Copy types yet?

@sinkuu

Copy link
Copy Markdown
Contributor

There's clone_on_copy lint.

@eddyb

Copy link
Copy Markdown
Contributor

Yeah, but are we enabling it on rustc?

@matthiaskrgr

Copy link
Copy Markdown
MemberAuthor

None of the clippy lints are run on rustc code by "default", but you can get clippy defaults run with ./x.py clippy

Meanwhile I've prepared a PR here: #68292

@matthiaskrgr
matthiaskrgr deleted the redundant_clones branch February 29, 2020 14:13
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: 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.

5 participants

@matthiaskrgr@oli-obk@bors@eddyb@sinkuu