[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