<div class="gmail_quote">On Mon, Jan 30, 2012 at 10:40 AM, Richard Trieu <span dir="ltr"><<a href="mailto:rtrieu@google.com">rtrieu@google.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"><div class="gmail_quote">On Wed, Jan 25, 2012 at 5:24 PM, Richard Trieu <span dir="ltr"><<a href="mailto:rtrieu@google.com" target="_blank">rtrieu@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
This patch adds a warning to catch semi-colons after function definitions.  The motivation of removing extraneous semi-colons is to make it easier to see which scopes are being ended by a brace.  Function scopes will end with a plain brace.  Class scopes will end with brace and semi-colon.<div>


<br></div><div>class C {</div><div>  void F() {};</div><div>};</div><div><br></div><div><div>extra-semi.cc:2:14: warning: extra ';' after function definition [-Wextra-semi]</div><div>  void F() {};</div><div>             ^</div>


<div>1 warning generated.</div></div>
</blockquote></div><br></div></div><div>Ping.</div>
</blockquote></div>Ping.