[PATCH] D150614: [clang-format] Ignore first token when finding MustBreak

Björn Schäpers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 17 12:20:30 PDT 2023


HazardyKnusperkeks accepted this revision.
HazardyKnusperkeks added inline comments.


================
Comment at: clang/unittests/Format/FormatTest.cpp:20136
                "#ifdef _DEBUG\n"
-               "void bar()\n"
-               "  {\n"
-               "  }\n"
+               "void bar() {}\n"
                "#else\n"
----------------
rymiel wrote:
> rymiel wrote:
> > HazardyKnusperkeks wrote:
> > > This doesn't seem right.
> > WhitesmithsBraceStyle.AllowShortFunctionsOnASingleLine = FormatStyle::SFS_Empty; is set above
> (this is mentioned in the summary)
> (this is mentioned in the summary)

Sorry about that, I must've skipped that part.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150614



More information about the cfe-commits mailing list