Skip to content

Reword trying to operate in immutable fields #47098

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
Jan 1, 2018

Conversation

estebank
Copy link
Contributor

The previous message ("cannot assign/mutably borrow immutable field")
when trying to modify a field of an immutable binding gave the
(incorrect) impression that fields can be mutable independently of their
ADT's binding. Slightly reword the message to read "cannot
assign/mutably borrow field of immutable binding".

Re #35937.

The previous message ("cannot assign/mutably borrow immutable field")
when trying to modify a field of an immutable binding gave the
(incorrect) impression that fields can be mutable independently of their
ADT's binding. Slightly reword the message to read "cannot
assign/mutably borrow field of immutable binding".
@rust-highfive
Copy link
Contributor

r? @michaelwoerister

(rust_highfive has picked a reviewer for you, use r? to override)

@nagisa
Copy link
Member

nagisa commented Jan 1, 2018

@bors r+

@bors
Copy link
Collaborator

bors commented Jan 1, 2018

📌 Commit 6384568 has been approved by nagisa

@bors
Copy link
Collaborator

bors commented Jan 1, 2018

⌛ Testing commit 6384568 with merge d5f2745...

bors added a commit that referenced this pull request Jan 1, 2018
Reword trying to operate in immutable fields

The previous message ("cannot assign/mutably borrow immutable field")
when trying to modify a field of an immutable binding gave the
(incorrect) impression that fields can be mutable independently of their
ADT's binding. Slightly reword the message to read "cannot
assign/mutably borrow field of immutable binding".

Re #35937.
@bors
Copy link
Collaborator

bors commented Jan 1, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: nagisa
Pushing d5f2745 to master...

@bors bors merged commit 6384568 into rust-lang:master Jan 1, 2018
# 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.

5 participants