<div dir="auto">Hello Tim,<div dir="auto"><br></div><div dir="auto">I am no longer working on this. I've spent my time convincing my colleagues that clang-format is useful enough to change our braces, with success.</div><div dir="auto"><br></div><div dir="auto">I remember also being stuck on the enumerations. Good luck!</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jun 12, 2019, 04:16 Tim Wojtulewicz via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">timwoj added a comment.<br>
<br>
I updated this patch to remove all of the code from `ContinuationIndenter` and to use the newer `BraceWrapping` style option instead of setting each one individually in `UnwrappedLineParser`.<br>
<br>
I still have the same issue with enums. Looking at the `RootToken` in `UnwrappedLineFormatter::formatFirstToken` when parsing an enum, the first pass through it has the entire line for the enum in it with all of the newlines stripped from my test case. It then doesn't ever return the braces as separate lines. If someone could give me a pointer as to how/why enums are tokenized differently from everything else, I can likely fix it. The test case I'm using looks like:<br>
<br>
  verifyFormat("enum X\n"<br>
               "  {\n"<br>
               "  Y = 0,\n"<br>
               "  }\n",<br>
               WhitesmithsBraceStyle);<br>
<br>
I also have the issue with a `break` after a block inside of a case statement, as mentioned in <a href="https://reviews.llvm.org/D6833#107539" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D6833#107539</a>. Other than those two, it's mostly working correctly. I removed the test for lambdas because I'm not entirely sure how that should even be formatted with Whitesmiths, but I can add it back in again.<br>
<br>
Should I update this thread with the new diff against HEAD, or should I open a new one and close this one as dead?<br>
<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D6833/new/" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D6833/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D6833" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D6833</a><br>
<br>
<br>
<br>
</blockquote></div>