[cfe-commits] [Patch] Add warning for semi-colon after class function definition

Richard Trieu rtrieu at google.com
Mon Feb 13 13:52:59 PST 2012


On Mon, Jan 30, 2012 at 10:40 AM, Richard Trieu <rtrieu at google.com> wrote:

> On Wed, Jan 25, 2012 at 5:24 PM, Richard Trieu <rtrieu at google.com> wrote:
>
>> 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.
>>
>> class C {
>>   void F() {};
>> };
>>
>> extra-semi.cc:2:14: warning: extra ';' after function definition
>> [-Wextra-semi]
>>   void F() {};
>>              ^
>> 1 warning generated.
>>
>
> Ping.
>
Ping.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120213/3d5b7e73/attachment.html>


More information about the cfe-commits mailing list