This is the mail archive of the cygwin-patches mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Patch to optionally disable overlapped pipes


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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]