libs/newlib-cygwin
libs/newlib-cygwin
4
0
Fork 0
mirror of git://sourceware.org/git/newlib-cygwin.git synced 2025-02-22 00:38:06 +08:00
Code Issues Projects Releases Wiki Activity

* fhandler.cc (fhandler_base::setup_overlapped): Don't set signalled state to

Browse Source 
true initially.
(fhandler_base::has_ongoing_io): Delete ill-advised function.
(fhandler_base::read_overlapped): Rely on caller having checked nonblocking
state.  Don't attempt to (incorrectly) check it here.
(fhandler_base::write_overlapped): Remove call to has_ongoing_io.
* select.cc (peek_pipe): Ditto.
* fhandler.h (fhandler_base::has_ongoing_io): Delete declaration.
This commit is contained in:
Christopher Faylor 2010-02-15 01:45:05 +00:00
parent d67a6ce4a8
commit 8103e40743
4 changed files with 12 additions and 26 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
winsup/cygwin/ChangeLog
View File

@ -1,3 +1,14 @@
2010-02-14 Christopher Faylor <me+cygwin@cgf.cx>
* fhandler.cc (fhandler_base::setup_overlapped): Don't set signalled
state to true initially.
(fhandler_base::has_ongoing_io): Delete ill-advised function.
(fhandler_base::read_overlapped): Rely on caller having checked
nonblocking state. Don't attempt to (incorrectly) check it here.
(fhandler_base::write_overlapped): Remove call to has_ongoing_io.
* select.cc (peek_pipe): Ditto.
* fhandler.h (fhandler_base::has_ongoing_io): Delete declaration.
2010-02-14 Christopher Faylor <me+cygwin@cgf.cx> 2010-02-14 Christopher Faylor <me+cygwin@cgf.cx>
* regex/regcomp.c (xwcrtomb): Fix one explicable and one inexcplicable * regex/regcomp.c (xwcrtomb): Fix one explicable and one inexcplicable

24
winsup/cygwin/fhandler.cc
View File

@ -1647,7 +1647,7 @@ fhandler_base::setup_overlapped (bool doit)
if (doit) if (doit)
{ {
set_overlapped (ov); set_overlapped (ov);
res = !!(ov->hEvent = CreateEvent (&sec_none_nih, true, true, NULL)); res = !!(ov->hEvent = CreateEvent (&sec_none_nih, true, false, NULL));
} }
else else
{ {
@ -1751,29 +1751,12 @@ fhandler_base::wait_overlapped (bool inres, bool writing, DWORD *bytes, DWORD le
return res; return res;
} }
bool __stdcall
fhandler_base::has_ongoing_io (bool testit)
{
if (testit && get_overlapped () && get_overlapped ()->hEvent
&& WaitForSingleObject (get_overlapped ()->hEvent, 0) != WAIT_OBJECT_0)
{
set_errno (EAGAIN);
return true;
}
return false;
}
void __stdcall void __stdcall
fhandler_base::read_overlapped (void *ptr, size_t& len) fhandler_base::read_overlapped (void *ptr, size_t& len)
{ {
DWORD nbytes; DWORD nbytes;
while (1) while (1)
{ {
if (has_ongoing_io (is_nonblocking ()))
{
nbytes = (DWORD) -1;
break;
}
bool res = ReadFile (get_handle (), ptr, len, &nbytes, bool res = ReadFile (get_handle (), ptr, len, &nbytes,
get_overlapped ()); get_overlapped ());
int wres = wait_overlapped (res, false, &nbytes); int wres = wait_overlapped (res, false, &nbytes);
@ -1789,11 +1772,6 @@ fhandler_base::write_overlapped (const void *ptr, size_t len)
DWORD nbytes; DWORD nbytes;
while (1) while (1)
{ {
if (has_ongoing_io (is_nonblocking ()))
{
nbytes = (DWORD) -1;
break;
}
bool res = WriteFile (get_output_handle (), ptr, len, &nbytes, bool res = WriteFile (get_output_handle (), ptr, len, &nbytes,
get_overlapped ()); get_overlapped ());
int wres = wait_overlapped (res, true, &nbytes, (size_t) len); int wres = wait_overlapped (res, true, &nbytes, (size_t) len);

1
winsup/cygwin/fhandler.h
View File

@ -311,7 +311,6 @@ class fhandler_base
virtual char const *ttyname () { return get_name (); } virtual char const *ttyname () { return get_name (); }
virtual void __stdcall read (void *ptr, size_t& len) __attribute__ ((regparm (3))); virtual void __stdcall read (void *ptr, size_t& len) __attribute__ ((regparm (3)));
virtual void __stdcall read_overlapped (void *ptr, size_t& len) __attribute__ ((regparm (3))); virtual void __stdcall read_overlapped (void *ptr, size_t& len) __attribute__ ((regparm (3)));
virtual bool __stdcall has_ongoing_io (bool) __attribute__ ((regparm (2)));
virtual ssize_t __stdcall write (const void *ptr, size_t len); virtual ssize_t __stdcall write (const void *ptr, size_t len);
virtual ssize_t __stdcall write_overlapped (const void *ptr, size_t len); virtual ssize_t __stdcall write_overlapped (const void *ptr, size_t len);
virtual ssize_t __stdcall readv (const struct iovec *, int iovcnt, ssize_t tot = -1); virtual ssize_t __stdcall readv (const struct iovec *, int iovcnt, ssize_t tot = -1);

2
winsup/cygwin/select.cc
View File

@ -526,8 +526,6 @@ out:
else if (fh->get_device () == FH_PIPER) else if (fh->get_device () == FH_PIPER)
select_printf ("%s, select for write on read end of pipe", select_printf ("%s, select for write on read end of pipe",
fh->get_name ()); fh->get_name ());
else if (fh->has_ongoing_io (true))
s->write_ready = false;
else else
{ {
IO_STATUS_BLOCK iosb = {0}; IO_STATUS_BLOCK iosb = {0};