-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
Replace "lvalue" terminology with "place". #47837
Conversation
src/test/ui/issue-26093.stderr
Outdated
@@ -4,7 +4,7 @@ error[E0070]: invalid left-hand side expression | |||
13 | $thing = 42; | |||
| ^^^^^^^^^^^ left-hand of expression not valid | |||
... | |||
19 | not_an_lvalue!(99); | |||
19 | not_a_place!(99); | |||
| ------------------- in this macro invocation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line needs to be shortened.
[00:45:32] ---- [ui] ui/issue-26093.rs stdout ----
[00:45:32] diff of stderr:
[00:45:32]
[00:45:32] 5 | ^^^^^^^^^^^ left-hand of expression not valid
[00:45:32] 6 ...
[00:45:32] 7 19 | not_a_place!(99);
[00:45:32] + | ----------------- in this macro invocation
[00:45:32] - | ------------------- in this macro invocation
[00:45:32] 9
[00:45:32] 10 error: aborting due to previous error
[00:45:32] 11
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh right, I'm not running ui
tests locally because they were broken at some point...
@bors r+ |
📌 Commit bba81c9 has been approved by |
@bors p=2 |
Replace "lvalue" terminology with "place". See #46425 for the previous PR (which only changed MIR-related code). r? @nikomatsakis
☀️ Test successful - status-appveyor, status-travis |
See #46425 for the previous PR (which only changed MIR-related code).
r? @nikomatsakis