Do not provide suggestions when the spans come from expanded code that doesn't point at user code#109082
Do not provide suggestions when the spans come from expanded code that doesn't point at user code#109082oli-obk wants to merge 14 commits intorust-lang:masterfrom
Conversation
rustbot
added
S-waiting-on-review
estebank
changed the title
|
clippy also needs some blessing |
compiler/rustc_middle/src/ty/mod.rs
Outdated
There was a problem hiding this comment.
When should one use adjust_span? (needs some docs on this function)
All these call sites make me think that this is a bit fragile, if we forget it somewhere, a diagnostic will be subpar and it will be hard to find out where the problematic span is from. Can we do this in a more principled way? Like somewhere during ast lowering or even macro expansion?
tests/ui/alloc-error/alloc-error-handler-bad-signature-1.stderr
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
this note shows up in json but not in rendered diagnostics
There was a problem hiding this comment.
I don't think we'll ever want it to come up. I'll see if I can fully silence it.
There was a problem hiding this comment.
I had some trouble finding the right place to silence this. I thought emitter.rs was the right place, but the place where this seemed to be coming from, wasn't.
There was a problem hiding this comment.
🤷 not really a problem
|
Some changes occurred in src/tools/clippy cc @rust-lang/clippy |
Yeah, went ahead and fixed the lint properly in its own commit :) |
in the name of the father, the son, and the holy @ghost, i bless thee |
|
Some changes occurred in src/tools/rustfmt cc @rust-lang/rustfmt |
This comment has been minimized.
This comment has been minimized.
estebank
force-pushed
the
macro-spans
branch
2 times, most recently
from
05b32a6 to
4661971
Compare
estebank
changed the title
|
@bors try @rust-timer queue |
This comment has been minimized.
This comment has been minimized.
rustbot
added
the
S-waiting-on-perf
|
⌛ Trying commit 019556d with merge a9ff946a44b0ea2531c5536d077f72f18806d291... |
|
☀️ Try build successful - checks-actions |
1 similar comment
|
☀️ Try build successful - checks-actions |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
The Miri subtree was changed cc @rust-lang/miri |
|
☔ The latest upstream changes (presumably #109128) made this pull request unmergeable. Please resolve the merge conflicts. |
…id-sugg, r=estebank Remove invalid suggestion of into_iter for extern macro Fixes rust-lang#137345 rust-lang#109082 is closed due to performance issue, do we have any other solution for this kind of issue?
…id-sugg, r=estebank Remove invalid suggestion of into_iter for extern macro Fixes rust-lang#137345 rust-lang#109082 is closed due to performance issue, do we have any other solution for this kind of issue?
…id-sugg, r=estebank Remove invalid suggestion of into_iter for extern macro Fixes rust-lang#137345 rust-lang#109082 is closed due to performance issue, do we have any other solution for this kind of issue?
…id-sugg, r=estebank Remove invalid suggestion of into_iter for extern macro Fixes rust-lang#137345 rust-lang#109082 is closed due to performance issue, do we have any other solution for this kind of issue?
Rollup merge of rust-lang#137415 - chenyukang:yukang-fix-137345-invalid-sugg, r=estebank Remove invalid suggestion of into_iter for extern macro Fixes rust-lang#137345 rust-lang#109082 is closed due to performance issue, do we have any other solution for this kind of issue?
8 participants
Hide invalid proc-macro suggestions and track spans
coming from proc-macros pointing at attribute.
Effectively, unless the proc-macro keeps user spans,
suggestions will not be produced for the code they
produce.
r? @ghost
Fix #107113, fix #107976, fix #107977, fix #108748, fix #106720, fix #90557.
Could potentially address #50141, #67373, #55146, #78862, #74043, #88514, #83320, #91520, #104071. CC #50122, #76360.