From dc28055962948b3f0993f07d3c014557fa6d9d14 Mon Sep 17 00:00:00 2001
From: Dimitris Papastamos <s...@2f30.org>
Date: Thu, 23 Jun 2016 13:04:53 +0100
Subject: [PATCH] output/sndio: No need to use a loop in Play()
This is a left-over from the previous version of the code
that was retrying on EINTR.
---
src/output/plugins/SndioOutputPlugin.cxx | 14 ++++----------
1 file changed, 4 insertions(+), 10 deletions(-)
diff --git a/src/output/plugins/SndioOutputPlugin.cxx
b/src/output/plugins/SndioOutputPlugin.cxx
index 69747ae..d5741ea 100644
--- a/src/output/plugins/SndioOutputPlugin.cxx
+++ b/src/output/plugins/SndioOutputPlugin.cxx
@@ -180,16 +180,10 @@ SndioOutput::Play(const void *chunk, size_t size, Error
&error)
{
size_t n;
- while (1) {
- n = sio_write(sio_hdl, chunk, size);
- if (n == 0) {
- if (sio_eof(sio_hdl)) {
- error.Set(sndio_output_domain, -1, "sndio write
failed");
- return 0;
- }
- }
- return n;
- }
+ n = sio_write(sio_hdl, chunk, size);
+ if (n == 0 && sio_eof(sio_hdl) != 0)
+ error.Set(sndio_output_domain, -1, "sndio write failed");
+ return n;
}
typedef AudioOutputWrapper<SndioOutput> Wrapper;
--
2.8.4
_______________________________________________
mpd-devel mailing list
mpd-devel@musicpd.org
http://mailman.blarg.de/listinfo/mpd-devel
