On Mon, 12 Aug 2013, Viresh Kumar wrote: > On 11 August 2013 22:21, Julia Lawall <Julia.Lawall@xxxxxxx> wrote: > > From: Julia Lawall <Julia.Lawall@xxxxxxx> > > > > Replace ARRAY_AND_SIZE(e) in function argument position to avoid hiding the > > arity of the called function. > > Makes sense. > > > The semantic match that makes this change is as follows: > > (http://coccinelle.lip6.fr/) > > What is this doing here? I'm not sure to understand the sense of the question. The semantic patch explains exactly what considerations were used when making the transformation. That is, any function call, having a call to ARRAY_AND_SIZE as an argument is rewritten to replace the call to ARRAY_AND_SIZE with the correspondingtwo arguments. The transformation was performed entirely automatically, although in a lot of cases it was necessary to add newlines afterwards. julia > > // <smpl> > > @@ > > expression e,f; > > @@ > > > > f(..., > > - ARRAY_AND_SIZE(e) > > + e,ARRAY_SIZE(e) > > ,...) > > // </smpl> > > > > Signed-off-by: Julia Lawall <Julia.Lawall@xxxxxxx> > > > > --- > > Not compiled. > > > > drivers/cpufreq/pxa3xx-cpufreq.c | 6 ++++-- > > 1 file changed, 4 insertions(+), 2 deletions(-) > > Acked-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx> > -- > To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe cpufreq" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
