<div dir="ltr">Sounds reasonable.<div>Probably we should warn on this construct (it is suspicious and may be a mistype) but it looks like such warning may be safely turned off by default.</div><div><br></div><div>Warning on redefinition of 'inline' looks useful, redefining it is indeed prety often. For instance, PHP sources it and I know a real case when linker complains confused developers. BTW Microsoft C complains on keyword redefinition including 'inline'.</div></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-13 1:46 GMT+06:00 Arthur O'Dwyer <span dir="ltr"><<a href="mailto:arthur.j.odwyer@gmail.com" target="_blank">arthur.j.odwyer@gmail.com</a>></span>:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Serge: I think Joerg was suggesting that<br>
<br>
    #define foo foo<br>
<br>
should never warn (even when "foo" is a keyword or reserved<br>
identifier), because it is pretty much a no-op.<br>
Whether "foo" is identical to "inline" makes no difference; what<br>
matters is that it's being #define'd to itself.<br>
<span class="HOEnZb"><font color="#888888"><br>
–Arthur<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On Fri, Dec 12, 2014 at 9:10 AM, Serge Pavlov <<a href="mailto:sepavloff@gmail.com">sepavloff@gmail.com</a>> wrote:<br>
> Probably, on the other hand the pattern '#define inline' often make<br>
> troubles, - code with inline functions starts producing link errors when<br>
> 'inline is removed by such define.<br>
><br>
> Are there objection to excluding the pattern '#define inline' to this<br>
> warning? It would require different -W flag.<br>
><br>
> Thanks,<br>
> --Serge<br>
><br>
> 2014-12-12 21:02 GMT+06:00 Joerg Sonnenberger <<a href="mailto:joerg@britannica.bec.de">joerg@britannica.bec.de</a>>:<br>
>><br>
>> On Fri, Dec 12, 2014 at 04:48:40PM +0600, Serge Pavlov wrote:<br>
>> > Thank you for feedback.<br>
>> ><br>
>> > I removed warning on #undef keyword, and changed wording in r224100.<br>
>><br>
>> Similar, it should not warning about #define inline inline, which is<br>
>> also a very common pattern.<br>
>><br>
>> Joerg<br>
><br>
><br>
</div></div><div class="HOEnZb"><div class="h5">> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
</div></div></blockquote></div></div>