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.

output/sndio: No need to use a loop in Play()
9a049de8 · Dimitris Papastamos · Max Kellermann · 58b8398a · 9a049de8
Commit 9a049de8 authored Jun 23, 2016 by Dimitris Papastamos Committed by Max Kellermann Jun 24, 2016
Show whitespace changes
Inline Side-by-side

Showing with 1 addition and 7 deletions

SndioOutputPlugin.cxx src/output/plugins/SndioOutputPlugin.cxx +1 -7

No files found.
--- 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)) {
+	if (n == 0 && sio_eof(sio_hdl) != 0)
 		error.Set(sndio_output_domain, -1, "sndio write failed");
-				return 0;
-			}
-		}
 	return n;
-	}
 }

 typedef AudioOutputWrapper<SndioOutput> Wrapper;