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

Björn Schäpers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 2 00:41:52 PDT 2022


HazardyKnusperkeks accepted this revision.
HazardyKnusperkeks added inline comments.


================
Comment at: clang/lib/Format/UnwrappedLineParser.h:118-119
                               TokenType NextLBracesType = TT_Unknown,
+                              bool *HasDoWhile = nullptr,
                               bool *HasLabel = nullptr);
   bool tryToParseBracedList();
----------------
owenpan wrote:
> 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.
Ok.


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

https://reviews.llvm.org/D126758



More information about the cfe-commits mailing list