[PATCH] D126758: [clang-format] Handle do-while loops for RemoveBracesLLVM

Owen Pan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 1 15:36:27 PDT 2022


owenpan added inline comments.


================
Comment at: clang/lib/Format/UnwrappedLineParser.h:118-119
                               TokenType NextLBracesType = TT_Unknown,
+                              bool *HasDoWhile = nullptr,
                               bool *HasLabel = nullptr);
   bool tryToParseBracedList();
----------------
HazardyKnusperkeks wrote:
> Should we merge that to something like `KeepBraces`? Or do we use `HasLabel` and `HasDoWhile` differently?
We will use them differently. There will be no configuration for labels but some configurations for do-while loops eventually.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D126758/new/

https://reviews.llvm.org/D126758



More information about the cfe-commits mailing list