[PATCH] D93938: [clang-format] Fixed AfterEnum handling
Ally Tiritoglu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Mar 10 01:16:09 PST 2021
atirit added a comment.
In D93938#2614825 <https://reviews.llvm.org/D93938#2614825>, @HazardyKnusperkeks wrote:
> If the bugs are (very) similar, I could live with one fix for both. Otherwise you should fix the other bug first, if its blocking this change.
The only thing linking the bugs is `AfterEnum`. Other than that I'd say they're separate. If the bug should be fixed first (instead of merging incorrect but passing tests) then I think it should be a separate diff.
My concern with that course of action is that whoever reviews that diff will all but certainly ask for unit tests, and if I wrote unit tests for `AfterEnum` and `AllowShortCaseLabelsOnASingleLine` as I have here, they'll still be incorrect because this bug is not merged. Would you foresee this being an issue?
I'll start working on a fix regardless.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D93938/new/
https://reviews.llvm.org/D93938
More information about the cfe-commits
mailing list