[PATCH] D75022: clang-format: Extend AllowShortLoopsOnASingleLine to do ... while loops.

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 25 00:23:57 PST 2020


MyDeveloperDay added a comment.

I think its look good, please add the extra tests, then lets give people a couple of days



================
Comment at: unittests/Format/FormatTest.cpp:570
+               AllowsMergedLoops);
 }
 
----------------
horrible code though they are, could you add a test for the label and comment case

```
do
label:
a++
white(true);
```

and

```
do
// comment
a++
white(true);
```


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

https://reviews.llvm.org/D75022





More information about the cfe-commits mailing list