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.

output_all: fix off-by-one error in audio_output_all_check()
e3b9b57e · Max Kellermann · d8218cae · e3b9b57e
Commit e3b9b57e authored Mar 12, 2009 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

output_all.c src/output_all.c +1 -1

No files found.
--- 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)