Re: [PATCH 1/4] Adds -Wshadow=local on KBUILD_HOSTCFLAGS

From: Leonardo Bras
Date: Wed Oct 17 2018 - 20:41:08 EST


Hello Boris,


> Next question: why not -Wshadow simply?

Good question. I can change it to -Wshadow and fix stuff for v2.

>
> Also, -Wshadow for the target compiler is an extra warning (W=2). Why is
> it enabled by default here?

The idea was to put it as default and fix all the shadowing warnings.
What do you think? I am open to suggestions.

Thank you,
Leonardo Bras


> --
> Regards/Gruss,
> Boris.
>
> Good mailing practices for 400: avoid top-posting and trim the reply.