Re: [PATCH v3 4/5] modpost: Changes parameter name to avoid shadowing.

From: Leonardo Bras
Date: Wed Oct 31 2018 - 23:31:53 EST


Em qua, 2018-10-31 Ãs 20:24 -0300, Leonardo Bras escreveu:
> Em seg, 2018-10-29 Ãs 01:42 +0900, Masahiro Yamada escreveu:
> > On Wed, Oct 24, 2018 at 1:05 PM Leonardo Bras <leobras.c@xxxxxxxxx>
> > wrote:
> > > Changes the parameter name to avoid shadowing a variable.
> > >
> > > Signed-off-by: Leonardo Bras <leobras.c@xxxxxxxxx>
> >
> > For this one, I'd rather like to see code refactoring
> > than renaming the variable.
> >
> > I will take a closer look.
>
> What do you suggest to refactor?
> I volunteer for this work.
>
> Leonardo Bras
>

I refactored the code to move the global variable *modules to a local
context at function main, and changed the functions to accept *modules
as a parameter.

Is that what you had in mind?

https://lkml.org/lkml/2018/11/1/725

Thanks for reading,

Leonardo Bras