This is the mail archive of the
mailing list for the Cygwin project.
[PATCH draft 0/6] Remove the fhandler_base_overlapped class
- From: Ken Brown <kbrown at cornell dot edu>
- To: "cygwin-patches at cygwin dot com" <cygwin-patches at cygwin dot com>
- Date: Sun, 26 May 2019 15:10:35 +0000
- Subject: [PATCH draft 0/6] Remove the fhandler_base_overlapped class
fhandler_pipe is currently the only class derived from
fhandler_base_overlapped. This patch series rewrites parts of
fhandler_pipe so that it can be derived from fhandler_base instead.
We can then simplify the code by removing fhandler_base_overlapped.
In particular, this gets rid of the peculiar situation in which a
non-blocking write can return with I/O pending, leading to the
ugliness in fhandler_base_overlapped::close.
I've marked these patches as drafts because I've undoubtedly
overlooked some things. Also, I haven't systematically done any
regression tests. I have, however, run all the sample pipe programs
in Kerrisk's book "The Linux Programming Interface: Linux and UNIX
System Programming Handbook". I've also run emacs-X11, gdb, git,
make, etc., so far without problems.
Ken Brown (6):
Cygwin: fhandler_pipe: derive from fhandler_base
Cygwin: fhandler_pipe: add raw_read and raw_write
Cygwin: fhandler_pipe: set the blocking mode of the Windows pipe
Cygwin: fhandler_pipe: fix permission problem
Cygwin: remove the fhandler_base_overlapped class
Cygwin: add fhandler_base::npfs_handle
winsup/cygwin/fhandler.cc | 395 ++------------------------
winsup/cygwin/fhandler.h | 90 +-----
winsup/cygwin/fhandler_fifo.cc | 30 --
winsup/cygwin/fhandler_pipe.cc | 354 ++++++++++++++++++++++-
winsup/cygwin/fhandler_socket_unix.cc | 30 --
winsup/cygwin/syscalls.cc | 1 -
6 files changed, 380 insertions(+), 520 deletions(-)