diff options
| author | Max Kellermann <max@duempel.org> | 2013-09-27 07:58:48 +0200 |
|---|---|---|
| committer | Max Kellermann <max@duempel.org> | 2013-09-27 07:58:48 +0200 |
| commit | ada67a6a4ff53bef4d874eafea83272eac8e18f1 (patch) | |
| tree | 4687c92b75f5de8f8f44f3a19affb1d047b4c191 /src/PlayerThread.cxx | |
| parent | b6a5d1ad5a52063fcd846951367a5c39f4b34196 (diff) | |
| download | mpd-ada67a6a4ff53bef4d874eafea83272eac8e18f1.tar.gz mpd-ada67a6a4ff53bef4d874eafea83272eac8e18f1.tar.xz mpd-ada67a6a4ff53bef4d874eafea83272eac8e18f1.zip | |
PlayerThread: move code to player_control::CommandFinished()
Diffstat (limited to '')
| -rw-r--r-- | src/PlayerThread.cxx | 33 |
1 files changed, 12 insertions, 21 deletions
diff --git a/src/PlayerThread.cxx b/src/PlayerThread.cxx index 24cd9d0ef..fa39dbb18 100644 --- a/src/PlayerThread.cxx +++ b/src/PlayerThread.cxx @@ -271,19 +271,10 @@ struct player { }; static void -player_command_finished_locked(player_control &pc) -{ - assert(pc.command != PLAYER_COMMAND_NONE); - - pc.command = PLAYER_COMMAND_NONE; - pc.ClientSignal(); -} - -static void player_command_finished(player_control &pc) { pc.Lock(); - player_command_finished_locked(pc); + pc.CommandFinished(); pc.Unlock(); } @@ -608,7 +599,7 @@ player::ProcessCommand() pc.Unlock(); audio_output_all_enable_disable(); pc.Lock(); - player_command_finished_locked(pc); + pc.CommandFinished(); break; case PLAYER_COMMAND_QUEUE: @@ -617,7 +608,7 @@ player::ProcessCommand() assert(!IsDecoderAtNextSong()); queued = true; - player_command_finished_locked(pc); + pc.CommandFinished(); break; case PLAYER_COMMAND_PAUSE: @@ -641,7 +632,7 @@ player::ProcessCommand() pc.Lock(); } - player_command_finished_locked(pc); + pc.CommandFinished(); break; case PLAYER_COMMAND_SEEK: @@ -670,7 +661,7 @@ player::ProcessCommand() pc.next_song->Free(); pc.next_song = nullptr; queued = false; - player_command_finished_locked(pc); + pc.CommandFinished(); break; case PLAYER_COMMAND_REFRESH: @@ -684,7 +675,7 @@ player::ProcessCommand() if (pc.elapsed_time < 0.0) pc.elapsed_time = elapsed_time; - player_command_finished_locked(pc); + pc.CommandFinished(); break; } } @@ -930,7 +921,7 @@ player::Run() if (pc.command == PLAYER_COMMAND_SEEK) elapsed_time = pc.seek_where; - player_command_finished_locked(pc); + pc.CommandFinished(); while (true) { ProcessCommand(); @@ -1138,7 +1129,7 @@ player_task(gpointer arg) pc.next_song = nullptr; } - player_command_finished_locked(pc); + pc.CommandFinished(); break; case PLAYER_COMMAND_CLOSE_AUDIO: @@ -1147,7 +1138,7 @@ player_task(gpointer arg) audio_output_all_release(); pc.Lock(); - player_command_finished_locked(pc); + pc.CommandFinished(); assert(buffer.IsEmptyUnsafe()); @@ -1157,7 +1148,7 @@ player_task(gpointer arg) pc.Unlock(); audio_output_all_enable_disable(); pc.Lock(); - player_command_finished_locked(pc); + pc.CommandFinished(); break; case PLAYER_COMMAND_EXIT: @@ -1176,12 +1167,12 @@ player_task(gpointer arg) pc.next_song = nullptr; } - player_command_finished_locked(pc); + pc.CommandFinished(); break; case PLAYER_COMMAND_REFRESH: /* no-op when not playing */ - player_command_finished_locked(pc); + pc.CommandFinished(); break; case PLAYER_COMMAND_NONE: |