Skip to content

needless_borrow reported on &&T when only &T implements Trait and &Trait is required #957

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 27, 2016
Merged

needless_borrow reported on &&T when only &T implements Trait and &Trait is required #957

merged 1 commit into from
May 27, 2016

Conversation

oli-obk
Copy link
Contributor

@oli-obk oli-obk commented May 27, 2016

This was found by Zr40 on irc

I almost had a test for this, but I made it generic, and that didn't trigger

@oli-obk
Copy link
Contributor Author

oli-obk commented May 27, 2016 •
edited
Loading

@mcarton told me so when I wrote the lint xD

@mcarton mcarton merged commit ef46015 into rust-lang:master May 27, 2016
@mcarton
Copy link
Member

mcarton commented May 27, 2016

@mcarton told me so when I wrote the lint xD

😄

@oli-obk oli-obk deleted the needs_borrow branch May 27, 2016 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@oli-obk @mcarton