<div class="gmail_quote">On Mon, Feb 13, 2012 at 1:59 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">On Mon, Feb 13, 2012 at 1:52 PM, Richard Trieu <<a href="mailto:rtrieu@google.com">rtrieu@google.com</a>> wrote:<br>
> On Mon, Jan 30, 2012 at 10:40 AM, Richard Trieu <<a href="mailto:rtrieu@google.com">rtrieu@google.com</a>> wrote:<br>
>><br>
>> On Wed, Jan 25, 2012 at 5:24 PM, Richard Trieu <<a href="mailto:rtrieu@google.com">rtrieu@google.com</a>> wrote:<br>
>>><br>
>>> This patch adds a warning to catch semi-colons after function<br>
>>> definitions.  The motivation of removing extraneous semi-colons is to make<br>
>>> it easier to see which scopes are being ended by a brace.  Function scopes<br>
>>> will end with a plain brace.  Class scopes will end with brace and<br>
>>> semi-colon.<br>
>>><br>
>>> class C {<br>
>>>   void F() {};<br>
>>> };<br>
>>><br>
>>> extra-semi.cc:2:14: warning: extra ';' after function definition<br>
>>> [-Wextra-semi]<br>
>>>   void F() {};<br>
>>>              ^<br>
>>> 1 warning generated.<br>
>><br>
>><br>
>> Ping.<br>
><br>
> Ping.<br>
<br>
</div></div>Not to derail this review - but just out of curiosity: do we have a<br>
more general 'unnecessary ;' warning? or could we make a general one<br>
that covers all cases where ';' is just redudndant & suggests a<br>
removal?<br></blockquote><div>There's a few in the parse diagnostics.  They are:</div><div><br></div><div>ext_top_level_semi</div><div>warn_cxx98_compat_top_level_semi</div><div>ext_extra_struct_semi</div><div>ext_extra_ivar_semi</div>
<div><br></div><div>warn_cxx98_compat_top_level_semi is a warning that is in a diagnostic group so it must be left separate.  The other 3 are Extensions and could probably be folded in with my diagnostic if I switched mine to Extension as well and added a select into the message.</div>
</div>