PING^3 http://gcc.gnu.org/ml/gcc-patches/2007-02/msg01102.html
On 01/03/07, Manuel López-Ibáñez <lopezibanez@gmail.com> wrote:
> PING^2: http://gcc.gnu.org/ml/gcc-patches/2007-02/msg01102.html
>
> Thanks,
>
> Manuel.
>
> On 20/02/07, Manuel López-Ibáñez <lopezibanez@gmail.com> wrote:
> > Please review: http://gcc.gnu.org/ml/gcc-patches/2007-02/msg01102.html
> >
> > Thanks,
> >
> > Manuel.
> >
> > On 12/02/07, Manuel López-Ibáñez <lopezibanez@gmail.com> wrote:
> > > :ADDPATCH C:
> > >
> > > Wextra gives a warning when a function may return with and without a
> > > value, for example:
> > >
> > > foo (a)
> > > {
> > > if (a > 0)
> > > return a;
> > > }
> > >
> > > However, this warning is completely redundant, since Wreturn-type
> > > (that is enabled by -Wall) will give a warning for any 'return'
> > > without value, in a function whose return-type is not 'void' and about
> > > a 'return' with a value, in a function whose return-type is 'void'. I
> > > was not able to construct an example for which Wextra warns but
> > > Wreturn-type does not.
> > >
> > > So this patch removes this warning from Wextra and improves the
> > > description of Wreturn-type.
> > >
> > > Boostrapped and regression tested.
> > >
> > > OK for mainline?
> > >
> > >
> > > 2007-02-12 Manuel Lopez-Ibanez <manu@gcc.gnu.org>
> > >
> > > PR middle-end/7651
> > > * doc/invoke.texi (Wreturn-type): Complete description.
> > > (Wextra): Delete item about return-type warning.
> > > * c-decl.c: Delete redundant Wextra warning.
> > >
> > > testsuite/
> > > * gcc.dg/20030906-1.c: Replace Wextra with Wreturn-type.
> > > * gcc.dg/20030906-2.c: Likewise.
> > > * objc.dg/method-17.m: Add -Wreturn-type.
> > > * obj-c++.dg/method-21.mm: Likewise.
> > >
> > >
> >
>