<div dir="ltr">I meant removing the semicolon after Java enums, where they aren't necessary.</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 3, 2014 at 5:49 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">On 03.11.2014 16:53, Daniel Jasper wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Reverted in r221158.<br>
</blockquote>
<br>
Thanks.<span class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I think just removing the semicolon should be good<br>
in almost all cases. We can reconsider later if that proves to be wrong.<br>
</blockquote>
<br></span>
Removing the semicolon from were? If I just drop it from the C++ code I get an error:<br>
<br>
/tmp/test.cpp:3:23: error: expected ';' after enum<br>
enum TestEnum2 { D, C}<br>
                      ^<br>
                      ;<br>
<br>
Are you saying there is a way I can change the C++ code such that clang-format will do the right thing even with this patch? Or are you talking about removing semicolons at another place (maybe ignoring them in clang-format?).<br>
<br>
I am mostly asking to understand if I can just put the check back in place or if I could also update some code to avoid issues in the future.<br>
<br>
Cheers,<br>
Tobias<br>
</blockquote></div><br></div>