[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