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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits