diff options
| author | Max Kellermann <max@duempel.org> | 2009-02-23 09:29:56 +0100 |
|---|---|---|
| committer | Max Kellermann <max@duempel.org> | 2009-02-23 09:29:56 +0100 |
| commit | 5a898c15e79ab87d2466e61549fcc20ce115c67e (patch) | |
| tree | cb65b88718b0b8f3cf05221816b193833c41fe8a /src/output/shout_plugin.c | |
| parent | d50a3d513eb0452e762f1e4eeb717318958cd83c (diff) | |
| download | mpd-5a898c15e79ab87d2466e61549fcc20ce115c67e.tar.gz mpd-5a898c15e79ab87d2466e61549fcc20ce115c67e.tar.xz mpd-5a898c15e79ab87d2466e61549fcc20ce115c67e.zip | |
output_api: play() returns a length
The old API required an output plugin to not return until all data
passed to the play() method is consumed. Some output plugins have to
loop to fulfill that requirement, and may block during that. Simplify
these, by letting them consume only part of the buffer: make play()
return the length of the consumed data.
Diffstat (limited to 'src/output/shout_plugin.c')
| -rw-r--r-- | src/output/shout_plugin.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/src/output/shout_plugin.c b/src/output/shout_plugin.c index 7887e2922..2563e8092 100644 --- a/src/output/shout_plugin.c +++ b/src/output/shout_plugin.c @@ -413,7 +413,7 @@ my_shout_open_device(void *data, struct audio_format *audio_format) return true; } -static bool +static size_t my_shout_play(void *data, const char *chunk, size_t size) { struct shout_data *sd = (struct shout_data *)data; @@ -427,7 +427,10 @@ my_shout_play(void *data, const char *chunk, size_t size) return false; } - return write_page(sd); + if (!write_page(sd)) + return 0; + + return size; } static bool |