[PATCH] D95168: [clang-format] Add InsertBraces option
MyDeveloperDay via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Feb 2 08:24:07 PST 2021
MyDeveloperDay added inline comments.
================
Comment at: clang/unittests/Format/FormatTest.cpp:17904
+ " call_function(arg, arg);";
+ EXPECT_EQ(IfSourceShort, format(IfSourceShort, Style));
+
----------------
can these all be verifyFormats
================
Comment at: clang/unittests/Format/FormatTest.cpp:17996
+ format(ForSourceLong, Style));
+}
+
----------------
MyDeveloperDay wrote:
> are you testing do/while?
whilst people discuss the ethics of modifying the code ;-)
Can you add some comment based examples
```
if (condition) // my test
you_do_you();
if (condition)
you_do_you(); // my test
```
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D95168/new/
https://reviews.llvm.org/D95168
More information about the cfe-commits
mailing list