[Libreoffice] [PATCH 1/7] warning cleaing binfilter sw3io

Pierre-André Jacquod pjacquod at alumni.ethz.ch
Sun Nov 28 05:56:33 PST 2010


Hi,
further warning cleaning within binfilter.

Whoever has written this liked shadowing. I got
{
    var a    <---- warning shadowing this
    {
        var a  <--- warning shadowing a
    }
}

which are nice to sort out !!!

I review twice these patches before sending, but I fear to have missed
some points. Due to shadowing "this", the compiler does not complain if
I miss a parameter within the shadowed scope. I hope my IDE helped
enough to catch my human failure.

Learning with experiences, from now, not yet done consistently within
this patch series, I will proceed following this pattern:

as soon as I get a variable aaa shadowing this
-> replace all aaa with aaa0 independently if this variable is shadowing
or not (maybe I rename also a variable within a function, where there is
not shadowing)
  -> for following shadowing, increase the postfix with aaa1, aaa2,..

this will make slightly bigger diff, but will reduce the error
possibilities, since the compiler will complain for a aaa0 which was out
of the shadowed scope.

I hope this kind of approach is OK for you

regards
Pierre-André


More information about the LibreOffice mailing list