-
Notifications
You must be signed in to change notification settings - Fork 890
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
thread 'main' panicked at src/tools/rustfmt/src/parse/session.rs:42:9: silent emitter attempted to translate a diagnostic #6082
Comments
No worries. What version of rustfmt are you using? |
@aDifferentJT Do you also have an input snippet that we could run rustfmt on to reproduce the issue? |
The latest nightly, I ran rustup update about an hour ago. |
Not to hand unfortunately, I haven't yet tried to narrow down the code that causes it, I'll try to do that in the morning, it's quite late here. |
@aDifferentJT I don't think this is an issue with a particular code snippet. If I had to guess Id say changes made in rust-lang/rust#121085 are causing this issue. |
were you running rustfmt on a public repo? That might be enough to reproduce the issue. |
Sorry, no, it's private for now, if I get time in the morning I'll try to get a more minimal case I can send |
I've minimised the file, this seems to be the smallest file that causes the issue
|
Perfect! I'm also able to reproduce the bug using the snippet. Thank you very much! I think this will help us figure out what's going on. |
rust-lang/rust#121487 doesn't explicitly reference this issue, but I'm hoping that it also resolves the problem 🤞🏼 |
…r=pnkfelix errors: share `SilentEmitter` between rustc and rustfmt Fixes rust-lang/rustfmt#6082. Shares the `SilentEmitter` between rustc and rustfmt, and gives it a fallback bundle (since it can emit diagnostics in some contexts).
…r=pnkfelix errors: share `SilentEmitter` between rustc and rustfmt Fixes rust-lang/rustfmt#6082. Shares the `SilentEmitter` between rustc and rustfmt, and gives it a fallback bundle (since it can emit diagnostics in some contexts).
…r=pnkfelix errors: share `SilentEmitter` between rustc and rustfmt Fixes rust-lang/rustfmt#6082. Shares the `SilentEmitter` between rustc and rustfmt, and gives it a fallback bundle (since it can emit diagnostics in some contexts).
…r=pnkfelix errors: share `SilentEmitter` between rustc and rustfmt Fixes rust-lang/rustfmt#6082. Shares the `SilentEmitter` between rustc and rustfmt, and gives it a fallback bundle (since it can emit diagnostics in some contexts).
…r=pnkfelix errors: share `SilentEmitter` between rustc and rustfmt Fixes rust-lang/rustfmt#6082. Shares the `SilentEmitter` between rustc and rustfmt, and gives it a fallback bundle (since it can emit diagnostics in some contexts).
Rollup merge of rust-lang#121301 - davidtwco:rustfmt-silent-emitter, r=pnkfelix errors: share `SilentEmitter` between rustc and rustfmt Fixes rust-lang/rustfmt#6082. Shares the `SilentEmitter` between rustc and rustfmt, and gives it a fallback bundle (since it can emit diagnostics in some contexts).
@aDifferentJT when you have a moment can you confirm that this is resolved on the latest nightly (rustfmt 1.7.0-nightly (7d3702e4 2024-03-06)? I just ran your input snippet with a few recent nighty versions, and I can no longer reproduce the panic using the most recent nightly. steps to verify:
|
@ytmimi Looks like it's working, thanks |
rustc-ice-2024-02-19T01_32_59-84679.txt
Sorry, not a very helpful bug report, but it said to open an issue
The text was updated successfully, but these errors were encountered: