<div dir="ltr">Reverted in r221158. I think just removing the semicolon should be good in almost all cases. We can reconsider later if that proves to be wrong.</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 3, 2014 at 4:46 PM, Tobias Grosser <span dir="ltr"><<a href="mailto:tobias@grosser.es" target="_blank">tobias@grosser.es</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 03.11.2014 04:00, Daniel Jasper wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Author: djasper<br>
Date: Sun Nov  2 21:00:42 2014<br>
New Revision: 221125<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=221125&view=rev" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project?rev=221125&view=rev</a><br>
Log:<br>
clang-format: [Java] Allow trailing semicolons after enums.<br>
<br>
Before:<br>
   enum SomeThing { ABC, CDE }<br>
   ;<br>
<br>
After:<br>
   enum SomeThing { ABC, CDE };<br>
</blockquote>
<br></span>
This change seems to cause regressions on C/C++ code.<br>
<br>
clang-format-trunk<br>
------------------------------<u></u>--<br>
enum TestEnum { A, B }; typedef int L;<br>
enum TestEnum2 {<br>
  D,<br>
  C<br>
};<br>
<br>
    /// Hallo<br>
    class P {<br>
public:<br>
  int foo();<br>
};<br>
<br>
class B {<br>
  enum TestEnum3 { X, Y }; public : int foo();<br>
};<br>
------------------------------<u></u>--<br>
<br>
<br>
clang-format-3.5<br>
--------------------------<br>
enum TestEnum { A, B };<br>
typedef int L;<br>
enum TestEnum2 { D, C };<br>
<br>
/// Hallo<br>
class P {<br>
public:<br>
  int foo();<br>
};<br>
<br>
class B {<br>
  enum TestEnum3 { X, Y };<br>
<br>
public:<br>
  int foo();<br>
};<br>
------<br>
<br>
This change is visible on the Polly code base as well. For now I disabled the syntax check to silence the buildbots such that we can<br>
have a look at this without being pressured.<br>
<br>
Cheers,<br>
Tobias<br>
<br>
P.S.: Thanks for the clang-format library dependency removal, bisecting this was quick and easy.<br>
<br>
</blockquote></div><br></div>