aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2009-03-07 23:48:28 +0100
committerMax Kellermann <max@duempel.org>2009-03-07 23:48:28 +0100
commit27193d84026dca7dce8e864da5a703e203347cdc (patch)
tree3ccb6477309063056873e14b6f18706736f2601d
parentfc6d836a2da526d48d607ca5e6b09f94c20f9299 (diff)
downloadmpd-27193d84026dca7dce8e864da5a703e203347cdc.tar.gz
mpd-27193d84026dca7dce8e864da5a703e203347cdc.tar.xz
mpd-27193d84026dca7dce8e864da5a703e203347cdc.zip
output_all: fix boolean short circuit in update()
Sometimes, audio_output_update() isn't called for the second device when the first one has succeeded. The patch "audio_output_all_update() returns bool" broke it, because the boolean evaluation ended after the first "true".
Diffstat (limited to '')
-rw-r--r--src/output_all.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/output_all.c b/src/output_all.c
index eca4d191c..7b308a428 100644
--- a/src/output_all.c
+++ b/src/output_all.c
@@ -178,8 +178,8 @@ audio_output_all_update(void)
return false;
for (i = 0; i < num_audio_outputs; ++i)
- ret = ret || audio_output_update(&audio_outputs[i],
- &input_audio_format);
+ ret = audio_output_update(&audio_outputs[i],
+ &input_audio_format) || ret;
return ret;
}