Skip to content
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.

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

Suggest .copied() for map_clone on iterators too #4043

Merged
merged 1 commit into from
Apr 28, 2019
Merged

Conversation

Manishearth
Copy link
Member

fixes #3958

changelog: Make map_clone suggest the newly-stable Iterator::copied() when applicable

r? @mikerite @matthiaskrgr

@matthiaskrgr
Copy link
Member

Thanks!
@bors r+

@bors
Copy link
Contributor

bors commented Apr 28, 2019

📌 Commit 770de14 has been approved by matthiaskrgr

@bors
Copy link
Contributor

bors commented Apr 28, 2019

⌛ Testing commit 770de14 with merge 78d13e0...

bors added a commit that referenced this pull request Apr 28, 2019
Suggest .copied() for map_clone on iterators too

fixes #3958

changelog: Make `map_clone` suggest the newly-stable `Iterator::copied()` when applicable

r? @mikerite @matthiaskrgr
@bors
Copy link
Contributor

bors commented Apr 28, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: matthiaskrgr
Pushing 78d13e0 to master...

@bors bors merged commit 770de14 into master Apr 28, 2019
@flip1995 flip1995 deleted the map-clone-iter branch October 28, 2019 08:14
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.

Should map_clone trigger on copies of double references?
3 participants