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