[PATCH] On non-Cygwin, use O_TEXT when O_BINARY is not set.

[Corinna Vinschen]

On Feb 21 09:08, Michael Haubenwallner wrote:
> * libc/stdio/flags.c (__sflags): Fix typo when testing if O_BINARY is
> set, to really set O_TEXT if not O_BINARY, applies to non-Cygwin only.
> ---
>  newlib/libc/stdio/flags.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/newlib/libc/stdio/flags.c b/newlib/libc/stdio/flags.c
> index 71fc1f60a..cd73385ce 100644
> --- a/newlib/libc/stdio/flags.c
> +++ b/newlib/libc/stdio/flags.c
> @@ -90,7 +90,7 @@ __sflags (struct _reent *ptr,
>  	}
>      }
>  #if defined (O_TEXT) && !defined (__CYGWIN__)
> -  if (!(m | O_BINARY))
> +  if (!(m & O_BINARY))
>      m |= O_TEXT;
>  #endif
>    *optr = m | o;
> -- 
> 2.19.2

Given that O_TEXT is under #if defined (__CYGWIN__) guard, do
we need this code at all?  Does anybody actually *want* O_TEXT
outside of the Windows world?


Corinna

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