Re: [PATCH] Do not include wctype.h in wchar.h with _GNU_SOURCE

[Corinna Vinschen <vinschen at redhat dot com>]

On Aug 12 13:47, Yaakov Selkowitz wrote:
> _GNU_SOURCE generally enables all features, but in this case the POSIX
> requirement to #include <wctype.h> for these is preferred.
> 
> Signed-off-by: Yaakov Selkowitz <yselkowi@redhat.com>
> ---
>  newlib/libc/include/wchar.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/newlib/libc/include/wchar.h b/newlib/libc/include/wchar.h
> index f4996ec..3a2b89b 100644
> --- a/newlib/libc/include/wchar.h
> +++ b/newlib/libc/include/wchar.h
> @@ -33,7 +33,7 @@ typedef __gnuc_va_list va_list;
>  #endif
>  #endif /* __POSIX_VISIBLE >= 200809 || __XSI_VISIBLE */
>  
> -#if __XSI_VISIBLE /* && __XSI_VISIBLE < 800 */
> +#if __XSI_VISIBLE /* && __XSI_VISIBLE < 800 */ && !__GNU_VISIBLE
>  #include <wctype.h>
>  #endif
>  
> -- 
> 2.8.3

ACK, as discussed on IRC.


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer
Red Hat

Attachment: signature.asc
Description: PGP signature