-
Notifications
You must be signed in to change notification settings - Fork 13.4k
Tweak some ‘expected…’ error messages #21223
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
Conversation
r? @huonw (rust_highfive has picked a reviewer for you, use r? to override) |
true | ||
} else { | ||
false | ||
} |
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.
Why not just self.token.is_keyword(kw)
?
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.
Whoops! Thanks for pointing this out. Fix incoming…
5d6d32a
to
d4487c8
Compare
Thanks for this; r=me with a rebase (sorry for the delay :( ). |
d4487c8
to
c0f903b
Compare
c0f903b
to
597b4fa
Compare
@huonw Rebased. |
⌛ Testing commit 597b4fa with merge 5f58d3c... |
💔 Test failed - auto-win-32-nopt-t |
@bors: retry |
⌛ Testing commit 597b4fa with merge a6079b2... |
💔 Test failed - auto-mac-64-nopt-t |
@bors: retry |
⌛ Testing commit 597b4fa with merge fe20812... |
💔 Test failed - auto-win-32-opt |
@bors: retry |
⌛ Testing commit 597b4fa with merge 45fb2eb... |
💔 Test failed - auto-linux-64-x-android-t |
@bors: retry |
⌛ Testing commit 597b4fa with merge 7263fcb... |
💔 Test failed - auto-linux-64-x-android-t |
@bors: retry |
⌛ Testing commit 597b4fa with merge 42fdb93... |
💔 Test failed - auto-mac-64-opt |
@bors: retry |
⌛ Testing commit 597b4fa with merge 4e69c11... |
💔 Test failed - auto-linux-64-x-android-t |
@bors: retry |
⌛ Testing commit 597b4fa with merge c9e7931... |
💔 Test failed - auto-mac-32-opt |
@bors: retry Sorry for the noise :( |
Fixes #21153.