diff options
| author | Max Kellermann <max@duempel.org> | 2009-03-12 19:49:10 +0100 |
|---|---|---|
| committer | Max Kellermann <max@duempel.org> | 2009-03-12 19:49:10 +0100 |
| commit | e3b9b57ecdf8c1396d120f1ccee0814a3e119511 (patch) | |
| tree | 3c2dfbaf635964f8189e0953ae772fa85c3c857a /src/output_all.c | |
| parent | d8218cae6b6742c56d0f570d5c8a27f1d8a17290 (diff) | |
| download | mpd-e3b9b57ecdf8c1396d120f1ccee0814a3e119511.tar.gz mpd-e3b9b57ecdf8c1396d120f1ccee0814a3e119511.tar.xz mpd-e3b9b57ecdf8c1396d120f1ccee0814a3e119511.zip | |
output_all: fix off-by-one error in audio_output_all_check()
When there are chunks which are not yet finished,
audio_output_all_check() returned the size of its music pipe minus
one. I can't remember exactly why I subtracted 1 from the return
value, it must have had something to do with a former meaning of this
function. Now it induces assertion failures.
Diffstat (limited to 'src/output_all.c')
| -rw-r--r-- | src/output_all.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/output_all.c b/src/output_all.c index a0f90df52..a95ff2d16 100644 --- a/src/output_all.c +++ b/src/output_all.c @@ -364,7 +364,7 @@ audio_output_all_check(void) if (!chunk_is_consumed(chunk)) /* at least one output is not finished playing this chunk */ - return music_pipe_size(g_mp) - 1; + return music_pipe_size(g_mp); is_tail = chunk->next == NULL; if (is_tail) |