Commit 18076ac9 authored by Max Kellermann's avatar Max Kellermann

PlayerThread: use pc.cond instead of main_cond

The main_cond variable was completely unnecessary. The pc.cond object can be used for both main->pc and pc->main.
parent ad15ca71
...@@ -98,8 +98,6 @@ enum { ...@@ -98,8 +98,6 @@ enum {
GThread *main_task; GThread *main_task;
GMainLoop *main_loop; GMainLoop *main_loop;
GCond *main_cond;
Partition *global_partition; Partition *global_partition;
static bool static bool
...@@ -404,7 +402,6 @@ int mpd_main(int argc, char *argv[]) ...@@ -404,7 +402,6 @@ int mpd_main(int argc, char *argv[])
main_task = g_thread_self(); main_task = g_thread_self();
main_loop = g_main_loop_new(NULL, FALSE); main_loop = g_main_loop_new(NULL, FALSE);
main_cond = g_cond_new();
GlobalEvents::Initialize(); GlobalEvents::Initialize();
GlobalEvents::Register(GlobalEvents::IDLE, idle_event_emitted); GlobalEvents::Register(GlobalEvents::IDLE, idle_event_emitted);
...@@ -536,7 +533,6 @@ int mpd_main(int argc, char *argv[]) ...@@ -536,7 +533,6 @@ int mpd_main(int argc, char *argv[])
sticker_global_finish(); sticker_global_finish();
#endif #endif
g_cond_free(main_cond);
GlobalEvents::Deinitialize(); GlobalEvents::Deinitialize();
playlist_list_global_finish(); playlist_list_global_finish();
......
...@@ -26,8 +26,6 @@ extern GThread *main_task; ...@@ -26,8 +26,6 @@ extern GThread *main_task;
extern GMainLoop *main_loop; extern GMainLoop *main_loop;
extern GCond *main_cond;
extern struct Partition *global_partition; extern struct Partition *global_partition;
/** /**
......
...@@ -76,7 +76,7 @@ static void ...@@ -76,7 +76,7 @@ static void
player_command_wait_locked(struct player_control *pc) player_command_wait_locked(struct player_control *pc)
{ {
while (pc->command != PLAYER_COMMAND_NONE) while (pc->command != PLAYER_COMMAND_NONE)
g_cond_wait(main_cond, pc->mutex); g_cond_wait(pc->cond, pc->mutex);
} }
static void static void
......
...@@ -147,7 +147,7 @@ player_command_finished_locked(struct player_control *pc) ...@@ -147,7 +147,7 @@ player_command_finished_locked(struct player_control *pc)
assert(pc->command != PLAYER_COMMAND_NONE); assert(pc->command != PLAYER_COMMAND_NONE);
pc->command = PLAYER_COMMAND_NONE; pc->command = PLAYER_COMMAND_NONE;
g_cond_signal(main_cond); g_cond_signal(pc->cond);
} }
static void static void
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment