aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2014-07-09 19:05:20 +0200
committerMax Kellermann <max@duempel.org>2014-07-09 19:05:20 +0200
commit20538516b92082067ce3477d612fb404ba8671ad (patch)
tree1365268be018b7ce410786ceb621e24a564a00ee
parent0759421d119d8d3d829978ede81683ca54d65d69 (diff)
downloadmpd-20538516b92082067ce3477d612fb404ba8671ad.tar.gz
mpd-20538516b92082067ce3477d612fb404ba8671ad.tar.xz
mpd-20538516b92082067ce3477d612fb404ba8671ad.zip
decoder/audiofile: use decoder_read_full()
Works around WAV stream playback bug, because libaudiofile does not like partial reads (Mantis 0004028).
-rw-r--r--NEWS1
-rw-r--r--src/decoder/AudiofileDecoderPlugin.cxx7
2 files changed, 7 insertions, 1 deletions
diff --git a/NEWS b/NEWS
index 19e3b2219..d6fd59519 100644
--- a/NEWS
+++ b/NEWS
@@ -4,6 +4,7 @@ ver 0.18.12 (not yet released)
- proxy: fix add/search and other commands with libmpdclient < 2.9
* decoder
- audiofile: improve responsiveness
+ - audiofile: fix WAV stream playback
ver 0.18.11 (2014/05/12)
* decoder
diff --git a/src/decoder/AudiofileDecoderPlugin.cxx b/src/decoder/AudiofileDecoderPlugin.cxx
index 7c76deb7d..3fb23bc20 100644
--- a/src/decoder/AudiofileDecoderPlugin.cxx
+++ b/src/decoder/AudiofileDecoderPlugin.cxx
@@ -43,7 +43,12 @@ struct AudioFileInputStream {
InputStream &is;
size_t Read(void *buffer, size_t size) {
- return decoder_read(decoder, is, buffer, size);
+ /* libaudiofile does not like partial reads at all,
+ and wil abort playback; therefore always force full
+ reads */
+ return decoder_read_full(decoder, is, buffer, size)
+ ? size
+ : 0;
}
};