[PATCH 20/20] unlocked stdio extensions

[Corinna Vinschen]

On Dec 18 11:32, Corinna Vinschen wrote:
> On Dec 17 15:23, Yaakov Selkowitz wrote:
> > On 2014-12-17 15:17, Corinna Vinschen wrote:
> > >Looks good, but there's one problem left:
> > 
> > Revised patch attached.
> 
> Thanks.  I'm seeing a problem with missing declarations:
> 
>   libc/stdio/getwc_u.c: In function ‘getwc_unlocked’:
>   libc/stdio/getwc_u.c:51:3: error: implicit declaration of function
>   ‘fgetwc_unlocked’ [-Werror=implicit-function-declaration]
> 
>   libc/stdio/getwchar_u.c: In function ‘getwchar_unlocked’:
>   libc/stdio/getwchar_u.c:49:3: error: implicit declaration of function
>   ‘fgetwc_unlocked’ [-Werror=implicit-function-declaration]
> 
>   libc/stdio/putwc_u.c: In function ‘putwc_unlocked’:
>   libc/stdio/putwc_u.c:52:3: error: implicit declaration of function
>   ‘fputwc_unlocked’ [-Werror=implicit-function-declaration]
> 
>   libc/stdio/putwchar_u.c: In function ‘putwchar_unlocked’:
>   libc/stdio/putwchar_u.c:51:3: error: implicit declaration of function
>   ‘fputwc_unlocked’ [-Werror=implicit-function-declaration]
> 
> The files should #define _GNU_SOURCE beforet he #include block to get
> the matching prototypes from stdio.h.
> 
> Two bugs in the documentaion part, both of them missing bracketing:
> 
> fputws.c:
> 
>   PORTABILITY
>   <<fputws> is required by C99 and POSIX.1-2001.
>          ^^^
> 
> getwchar.c:
> 
>   The alternate functions <<_getwchar_r>> and <<_getwchar_unlocked_r> are
>                                                                    ^^^
> 
> With these changes, ok to apply.

Oh, btw.   the addition to the comment in fflush.c:

  "Any future change must also be synced in fflush_u.c."

is redundant, now that you changed fflush_u.c to just #include fflush.c,


Corinna

-- 
Corinna Vinschen
Cygwin Maintainer
Red Hat
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/newlib/attachments/20141218/0bc6f6ec/attachment.sig>