<div dir="ltr"><div>Maybe turn this warning off by default? As Nico pointed out, there are additional cases:</div><div><br></div><div><span style="font-size:13px">#define inline _inline</span><br></div><div><span style="font-size:13px">#define final(a,b,c) some_computation(a, b,c )</span><span style="font-size:13px"><br></span></div><div><span style="font-size:13px"><br></span></div><div><span style="font-size:13px">They are not covered by the rules above. </span></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-15 23:45 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"><div class="HOEnZb"><div class="h5">On Mon, Dec 15, 2014 at 12:11 PM, Serge Pavlov <<a href="mailto:sepavloff@gmail.com">sepavloff@gmail.com</a>> 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>
</div></div>So they are, thank you for pointing that out!<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span></blockquote></div></div></div>