<div dir="ltr">OK, thank you for clarifications!</div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature">Thanks,<br>--Serge<br></div></div>
<br><div class="gmail_quote">2014-12-16 0:14 GMT+06:00 Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span>:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, Dec 15, 2014 at 1:01 PM, Serge Pavlov <<a href="mailto:sepavloff@gmail.com">sepavloff@gmail.com</a>> wrote:<br>
> Maybe turn this warning off by default? As Nico pointed out, there are<br>
> additional cases:<br>
><br>
> #define inline _inline<br>
<br>
</span>I think this is fine to cover from my suggestions above. _inline is an<br>
extension just the same as __inline is.<br>
<span class=""><br>
> #define final(a,b,c) some_computation(a, b,c )<br>
<br>
</span>This *should* warn for modes where final is a keyword because it's<br>
overriding the keyword's behavior.<br>
<br>
I am not keen on off-by-default warnings. I think the general rule of<br>
thumb is that if it's not important enough to be on by default, it may<br>
not be important enough to include. Personally, I think this is<br>
borderline, but have a preference to see it remain on by default, with<br>
the proper heuristics.<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
><br>
> They are not covered by the rules above.<br>
><br>
> Thanks,<br>
> --Serge<br>
><br>
> 2014-12-15 23:45 GMT+06:00 Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>>:<br>
>><br>
>> On Mon, Dec 15, 2014 at 12:11 PM, Serge Pavlov <<a href="mailto:sepavloff@gmail.com">sepavloff@gmail.com</a>><br>
>> wrote:<br>
>> > 2014-12-15 22:32 GMT+06:00 Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>>:<br>
>> >><br>
>> >><br>
>> >> This commit appears to be lacking tests, and I would like to see some<br>
>> >> added for each of the new diagnostics, as well as for cases that<br>
>> >> should be valid (such as keywords in language modes that do not have<br>
>> >> the keyword).<br>
>> >><br>
>> >><br>
>> > Test are added in r224100.<br>
>><br>
>> So they are, thank you for pointing that out!<br>
>><br>
>> ~Aaron<br>
</div></div></blockquote></div></div>