[patch] misc. warning fixes
Corinna Vinschen
vinschen@redhat.com
Tue Aug 23 10:55:00 GMT 2011
On Aug 22 18:45, Ralf Corsepius wrote:
> On 08/22/2011 06:18 PM, Corinna Vinschen wrote:
> >On Aug 20 06:38, Ralf Corsepius wrote:
> >>Hi,
> >>
> >>I'd like to apply the patch below.
> >>
> >>It addresses misc. warnings gcc issues when building newlib.
> >>
> >>Ralf
> >
> >>2011-08-20 Ralf Cors??pius<...>
> >>
> >> * libc/posix/regexec.c: Define "nope" only #ifndef NDEBUG.
> >> * libc/stdio/vfscanf.c: Define "state" only #ifdef _MB_CAPABLE.
> >> * libc/string/wcwidth.c: Include<wctypes.h> for "iswprint" and
> >> "iswcntrl".
> >
> >The latter two are ok. For the first one, I think we should just
> >remove `nope'. It's not used anywhere in regexec.c, nor is NDEBUG.
>
> Both are used:
>
> # grep nope libc/posix/*
> libc/posix/engine.c: assert(nope);
> ...
> libc/posix/regexec.c:static int nope = 0; /* for use in asserts;
> shuts lint up */
>
> (Using "assert()" implies using NDEBUG)
Uh, I missed that. Please apply.
Thanks,
Corinna
--
Corinna Vinschen
Cygwin Project Co-Leader
Red Hat
More information about the Newlib
mailing list