Skip to content

don't try to blame tuple fields for immutability #41108

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 1 commit into from
Apr 7, 2017

Conversation

arielb1
Copy link
Contributor

@arielb1 arielb1 commented Apr 6, 2017

Tuple fields don't have an &T in their declaration that can be changed
to &mut T - skip them..

Fixes #41104.

r? @nikomatsakis

Tuple fields don't have an `&T` in their declaration that can be changed
to `&mut T` - skip them..

Fixes rust-lang#41104.
@estebank
Copy link
Contributor

estebank commented Apr 6, 2017

@bors r+

@bors
Copy link
Collaborator

bors commented Apr 6, 2017

📌 Commit 95bd41e has been approved by estebank

frewsxcv added a commit to frewsxcv/rust that referenced this pull request Apr 6, 2017
don't try to blame tuple fields for immutability

Tuple fields don't have an `&T` in their declaration that can be changed
to `&mut T` - skip them..

Fixes rust-lang#41104.

r? @nikomatsakis
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Apr 6, 2017
don't try to blame tuple fields for immutability

Tuple fields don't have an `&T` in their declaration that can be changed
to `&mut T` - skip them..

Fixes rust-lang#41104.

r? @nikomatsakis
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Apr 6, 2017
don't try to blame tuple fields for immutability

Tuple fields don't have an `&T` in their declaration that can be changed
to `&mut T` - skip them..

Fixes rust-lang#41104.

r? @nikomatsakis
bors added a commit that referenced this pull request Apr 6, 2017
Rollup of 8 pull requests

- Successful merges: #40878, #40976, #41089, #41090, #41108, #41111, #41112, #41114
- Failed merges:
@bors bors merged commit 95bd41e into rust-lang:master Apr 7, 2017
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants