[PATCH 1/3] Cygwin: dps: Fix a bug that read() could not return -1 on error.

[PATCH 1/3] Cygwin: dps: Fix a bug that read() could not return -1 on error.

[Takashi Yano]

---
 winsup/cygwin/fhandler/dsp.cc | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/winsup/cygwin/fhandler/dsp.cc b/winsup/cygwin/fhandler/dsp.cc
index 8e51a51c5..e872aa08c 100644
--- a/winsup/cygwin/fhandler/dsp.cc
+++ b/winsup/cygwin/fhandler/dsp.cc
@@ -1192,7 +1192,9 @@ fhandler_dev_dsp::_read (void *ptr, size_t& len)
       return;
     }
 
-  audio_in_->read ((char *)ptr, (int&)len);
+  int res = len;
+  audio_in_->read ((char *)ptr, res);
+  len = (size_t)res;
 }
 
 void
-- 
2.39.0

[PATCH 2/3] Cygwin: dsp: Reduce wait time for blocking read().

[Takashi Yano]

Previous wait time of 100msec is too long if application specifies
smaller buffer. With this patch, the wait time is reduced to 1msec.
---
 winsup/cygwin/fhandler/dsp.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/winsup/cygwin/fhandler/dsp.cc b/winsup/cygwin/fhandler/dsp.cc
index e872aa08c..00f2bab69 100644
--- a/winsup/cygwin/fhandler/dsp.cc
+++ b/winsup/cygwin/fhandler/dsp.cc
@@ -931,8 +931,8 @@ fhandler_dev_dsp::Audio_in::waitfordata ()
 	  set_errno (EAGAIN);
 	  return false;
 	}
-      debug_printf ("100ms");
-      switch (cygwait (100))
+      debug_printf ("1ms");
+      switch (cygwait (1))
 	{
 	case WAIT_SIGNALED:
 	  if (!_my_tls.call_signal_handler ())
-- 
2.39.0

[PATCH 3/3] Cygwin: dsp: Fix trivial editorial issue.

[Takashi Yano]

---
 winsup/cygwin/fhandler/dsp.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/winsup/cygwin/fhandler/dsp.cc b/winsup/cygwin/fhandler/dsp.cc
index 00f2bab69..861443352 100644
--- a/winsup/cygwin/fhandler/dsp.cc
+++ b/winsup/cygwin/fhandler/dsp.cc
@@ -1029,7 +1029,7 @@ fhandler_dev_dsp::write (const void *ptr, size_t len)
 void
 fhandler_dev_dsp::read (void *ptr, size_t& len)
 {
-  return base ()->_read (ptr, len);
+  base ()->_read (ptr, len);
 }
 
 int
-- 
2.39.0