Patch to optionally disable overlapped pipes

Christopher Faylor cgf-use-the-mailinglist-please@cygwin.com
Wed Jan 8 18:18:00 GMT 2014


On Wed, Jan 08, 2014 at 06:00:54PM -0000, James Johnston wrote:
>The function I modified is fhandler_pipe::create(fhandler_pipe**, unsigned,
>int).  This function is a thin wrapper around a more specific
>fhandler_pipe::create(LPSECURITY_ATTRIBUTES, PHANDLE, PHANDLE, DWORD, const
>char*, DWORD open_mode) with default values for some of the parameters for
>that more specific function, and it passes FILE_FLAG_OVERLAPPED by default.
>My change involved optionally removing FILE_FLAG_OVERLAPPED from the
>default.
>
>Critically, my change does NOT affect any code that uses the
>fhandler_pipe::create overload that takes 6 parameters.

I'm the author of the code and I'm familiar with the implications of what
you did.  You modified the way pipes are commonly created.  I'm not
comfortable supporting code which has that option.

cgf



More information about the Cygwin-patches mailing list