-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
Always const qualify literals by type #69146
Conversation
@bors try @rust-timer queue |
Awaiting bors try build completion |
Always const qualify literals by type r? @eddyb
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
// | ||
// Note: If this is a pointer to a static, this uses | ||
// `constant.literal.ty` which is a reference or pointer to | ||
// the type of the actual static item. |
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 is technically correct, but also sounds somewhat tautological.
Since it's a reference/pointer, it should be treated like one, no need to special-case statics.
In fact I doubt borrow-checking even needs to know about these, it's just a matter of tweaking the messaging.
r=me with or without any comment changes. |
☀️ Try build successful - checks-azure |
Queued 50896e7 with parent d538b80, future comparison URL. |
c30254e
to
55de9fc
Compare
2abb336
to
f2980e7
Compare
Pushed a somewhat speculative perf fix |
Awaiting bors try build completion |
Always const qualify literals by type r? @eddyb
☀️ Try build successful - checks-azure |
Queued 7b7dc6e with parent dbef353, future comparison URL. |
Finished benchmarking try commit 7b7dc6e, comparison URL. |
@bors r=eddyb |
📌 Commit f2980e7 has been approved by |
⌛ Testing commit f2980e7 with merge 3c15ab90396ca113f17259b41b7583a72e82a3a2... |
💔 Test failed - checks-azure |
Yielded rollup |
Always const qualify literals by type r? @eddyb
Rollup of 6 pull requests Successful merges: - #69146 (Always const qualify literals by type) - #69159 (Select an appropriate unused lifetime name in suggestion) - #69194 (parse: fuse associated and extern items up to defaultness) - #69211 (parser: Simplify treatment of macro variables in `Parser::bump`) - #69217 (Do not emit note suggesting to implement operation trait to foreign type) - #69236 (parse: recover `mut (x @ y)` as `(mut x @ mut y)`.) Failed merges: r? @ghost
r? @eddyb