[PATCH] D126157: [clang-format][NFC] Insert/remove braces in clang/lib/Format/

owenpan added inline comments. Comment at: clang/lib/Format/UnwrappedLineParser.cpp:2769 - if (Style.isCSharp()) { do { owenpan wrote: > curdeius wrote: > > owenpan wrote: > > > From > > > https://llvm.org/docs/CodingStandards.html#don-t-use-braces-on-si

[PATCH] D126157: [clang-format][NFC] Insert/remove braces in clang/lib/Format/

owenpan added a comment. In D126157#3531635 , @MyDeveloperDay wrote: > LGTM too.. @owenpan this is one of my favourite features!! I love it too! :D Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D126157/new

[PATCH] D126157: [clang-format][NFC] Insert/remove braces in clang/lib/Format/

owenpan added inline comments. Comment at: clang/lib/Format/UnwrappedLineParser.cpp:2769 - if (Style.isCSharp()) { do { curdeius wrote: > owenpan wrote: > > From > > https://llvm.org/docs/CodingStandards.html#don-t-use-braces-on-simple-single-statement-b

[PATCH] D126157: [clang-format][NFC] Insert/remove braces in clang/lib/Format/

curdeius added inline comments. Comment at: clang/lib/Format/UnwrappedLineParser.cpp:2769 - if (Style.isCSharp()) { do { owenpan wrote: > From > https://llvm.org/docs/CodingStandards.html#don-t-use-braces-on-simple-single-statement-bodies-of-if-else-loop

[PATCH] D126157: [clang-format][NFC] Insert/remove braces in clang/lib/Format/

owenpan added inline comments. Comment at: clang/lib/Format/UnwrappedLineParser.cpp:2769 - if (Style.isCSharp()) { do { From https://llvm.org/docs/CodingStandards.html#don-t-use-braces-on-simple-single-statement-bodies-of-if-else-loop-statements, it's u

[PATCH] D126157: [clang-format][NFC] Insert/remove braces in clang/lib/Format/

MyDeveloperDay accepted this revision. MyDeveloperDay added a comment. LGTM too.. @owenpan this is one of my favourite features!! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D126157/new/ https://reviews.llvm.org/D126157 __

[PATCH] D126157: [clang-format][NFC] Insert/remove braces in clang/lib/Format/

curdeius accepted this revision. curdeius added a comment. This revision is now accepted and ready to land. Ok. So we mainly missed braces on complex conditionals. LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D126157/new/ https://reviews.llvm