On 30 May 2018 at 21:21, Prathamesh Kulkarni
<prathamesh.kulka...@linaro.org> wrote:
> On 30 May 2018 at 18:12, Jonathan Wakely <jwakely....@gmail.com> wrote:
>> On 30 May 2018 at 11:40, Prathamesh Kulkarni wrote:
>>> gcc with -Wmain warns for local variables named main.
>>>
>>> int foo()
>>> {
>>>   int main = 0;
>>>   return main;
>>> }
>>>
>>> a.c: In function ‘foo’:
>>> a.c:3:7: warning: ‘main’ is usually a function [-Wmain]
>>>    int main = 1;
>>>        ^~~~
>>>
>>> Is this intended ? I assumed that the warning's intent was for
>>> diagnosing variable named main having only external linkage.
>>
>> It was added more than 20 years ago by https://gcc.gnu.org/r13517 and
>> looks like it has always worked as it does now, without considering
>> linkage.
>>
>> Only warning for entities with external linkage seems reasonable to
>> me, but that would be for the C front-end or diagnostics maintainers
>> to decide.
> Thanks for the suggestions. I have attached (untested) patch that warns
> for Wmain if TREE_PUBLIC (decl) is true.
> Does it look OK ?
ping https://gcc.gnu.org/ml/gcc-patches/2018-05/msg01739.html

Thanks,
Prathamesh
>
> Thanks,
> Prathamesh

Reply via email to