On Apr 25, 2014, at 9:34 AM, Dimitris Papavasiliou <dpapa...@gmail.com> wrote:
> --Wreturn-type -Wsequence-point -Wshadow @gol
> +-Wreturn-type -Wsequence-point -Wshadow -Wshadow-ivar @gol
This has to be -Wno-shadow-ivar, we document the non-default…
> +@item -Wshadow-ivar @r{(Objective-C only)}
Likewise.
> + /* Check wheter the local variable hides the instance variable. */
spelling, whether...
> + a = private; /* { dg-warning "hides instance variable" "" { xfail *-*-*
> } } */
> + a = protected; /* { dg-warning "hides instance variable" "" { xfail *-*-*
> } } */
> + a = public; /* { dg-warning "hides instance variable" "" { xfail *-*-*
> } } */
No, we don’t expect failures. We makes the compiler do what we wants or it
gets the hose again. Then, we expect it to be perfect. If you won’t want
warning, and non are produces, then just remove the /* … */, or write /* no
warning */.
Also, synth up the ChnageLogs… :-), they are trivial enough.
And, just pop them all into one patch (cd ..; svn diff), 3 is 3x the work for
me.
Once we resolve the 3 warning tests above, this will be ok.
