-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Refactor SyntaxContext::ctxt
logic.
#110497
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
Conversation
r? @b-naber (rustbot has picked a reviewer for you, use r? to override) |
The change looks good to me. I tried to check whether this fixes the example in the issue, but I cannot even reproduce the issue. @cjgillot Do you want this to be merged now or wait until you come up with a test? |
The issue is very hard to reproduce, as it relies on the exact number of characters. I think merging without a test is fine: the old code was wrong. |
@bors r+ rollup |
Refactor `SyntaxContext::ctxt` logic. I'm still trying to make a test from the issue. cc `@deepink-mas` does this solve the issue? Fixes rust-lang#110230
Refactor `SyntaxContext::ctxt` logic. I'm still trying to make a test from the issue. cc ``@deepink-mas`` does this solve the issue? Fixes rust-lang#110230
Refactor `SyntaxContext::ctxt` logic. I'm still trying to make a test from the issue. cc ```@deepink-mas``` does this solve the issue? Fixes rust-lang#110230
☀️ Test successful - checks-actions |
Finished benchmarking commit (f12a7fa): comparison URL. Overall result: ❌ regressions - no action needed@rustbot label: -perf-regression Instruction countThis is a highly reliable metric that was used to determine the overall result at the top of this comment.
Max RSS (memory usage)ResultsThis is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
CyclesResultsThis is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
|
I'm still trying to make a test from the issue.
cc @deepink-mas does this solve the issue?
Fixes #110230