oysteikt/mpd
0
0
Fork 0
Code Issues Pull Requests Releases Activity

output/Thread: simplify the main loop switch

Browse Source 
Move the InternalPlay() call and the wake_cond.wait() call into the
`case Command::NONE` and revert all `continue` statements to a simple
`break`.
This commit is contained in:
Max Kellermann 2020-10-02 10:09:11 +02:00
parent 8f0a1a5d82
commit b0596291a8
1 changed files with 14 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

36
src/output/Thread.cxx
View File

@ -422,6 +422,16 @@ AudioOutputControl::Task() noexcept
while (true) {
switch (command) {
case Command::NONE:
/* no pending command: play (or wait for a
command) */
if (open && allow_play && InternalPlay(lock))
/* don't wait for an event if there
are more chunks in the pipe */
continue;
woken_for_play = false;
wake_cond.wait(lock);
break;
case Command::ENABLE:
@ -454,11 +464,7 @@ AudioOutputControl::Task() noexcept
}
InternalPause(lock);
/* don't "break" here: this might cause
Play() to be called when command==CLOSE
ends the paused state - "continue" checks
the new command first */
continue;
break;
case Command::RELEASE:
if (!open) {
@ -483,18 +489,14 @@ AudioOutputControl::Task() noexcept
CommandFinished();
}
/* don't "break" here: this might cause
Play() to be called when command==CLOSE
ends the paused state - "continue" checks
the new command first */
continue;
break;
case Command::DRAIN:
if (open)
InternalDrain();
CommandFinished();
continue;
break;
case Command::CANCEL:
source.Cancel();
@ -505,7 +507,7 @@ AudioOutputControl::Task() noexcept
}
CommandFinished();
continue;
break;
case Command::KILL:
InternalDisable();
@ -513,16 +515,6 @@ AudioOutputControl::Task() noexcept
CommandFinished();
return;
}
if (open && allow_play && InternalPlay(lock))
/* don't wait for an event if there are more
chunks in the pipe */
continue;
if (command == Command::NONE) {
woken_for_play = false;
wake_cond.wait(lock);
}
}
}