[C++] Coding rule enforcement

Nathan Sidwell nathan@acm.org
Thu Sep 17 14:56:00 GMT 2015


On 09/16/15 10:23, Jason Merrill wrote:
> On 09/16/2015 08:02 AM, Nathan Sidwell wrote:
>> +      else if (warn_multiple_inheritance)
>> +    warning (OPT_Wmultiple_inheritance,
>> +         "%qT defined with multiple direct bases", ref);
>
> You don't need to guard the warning with a check of the warning flag; warning
> will only give the warning if the option is enabled.

the spelling mistake on the option I used to experiment on didn't help, but I 
discovered one must have the Var clause in the options file too, which I didn't 
expect.

Anyway, fixed now.  ok?

nathan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: disable.patch
Type: text/x-patch
Size: 7976 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20150917/82c80295/attachment.bin>


More information about the Gcc-patches mailing list