-
Notifications
You must be signed in to change notification settings - Fork 888
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
Fix: ICE with expanded code #6105 #6112
Conversation
@Tanjaint21 thanks for helping out with this one. When you have a moment could you add the code snippet listed in #6105 (comment) as a test case. It would also be great if you could add a test case for the code snippet in #5885 as I believe that we should no longer panic when reformatting that macro. |
Awesome! one last ask. When you get a second can you add these new files to this test. I've been adding test cases here to prevent any regressions. |
Thanks again for your help on this one, and cheers on your first contribution to rustfmt 🎉 |
Thanks for fixing those! |
rustfmt
ICE with expanded code #6105src/tools/rustfmt/src/expr.rs:407:13
#5885This is my first pull request 🎉
In this, I am fixing this issue as per suggested by @ytmimi
Thank You.