[PATCH] D53482: Add clang-format stability check with FormatTests
Wouter van Oortmerssen via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 4 19:24:25 PST 2019
aardappel added a comment.
@krasimir first you say that fixing it is hard, then when someone wants to attempt, that it is not necessary?
Plenty of people run `clang-format` in automated fashion as part of some pipeline, seems to me results ping-ponging between two formattings and cluttering commits would be pretty bad.
I just ran into this bug (https://bugs.llvm.org/show_bug.cgi?id=39280) and now instructions to my contributors are going to have to be: please run `clang-format` twice :)
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D53482/new/
https://reviews.llvm.org/D53482
More information about the cfe-commits
mailing list