> From: Bruno Haible <[email protected]>
> Cc: [email protected], [email protected]
> Date: Sat, 12 May 2012 23:28:12 +0200
> 
> >         (O_TEXT) [!O_BINARY]: Define if not defined.
> 
> Why? You don't need it in the diffutils change.

Yes, I do.  See this line:

              (void) setmode (current->desc, O_TEXT);

> This part is basically OK, but lacks comments and a unit test.
> Here's a proposed patch. It passes all tests on glibc, mingw, MSVC, Cygwin.

Thanks.

Reply via email to