Now that we print >1 line of code snippet, we printed another line of code for now reason, because the source range we created for the fixit expanded to the next line, if the next token was there. Don't do that.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
As far as the fixit goes, I guess the previous version was better, since it didn't leave an empty line behind?
Comment Actions
Yes, but that's a formatting-related concern with the output, and we traditionally leave that to another tool like clang-format to address.
LGTM, but add a release note since there technically is a change in behavior with the fix-it?