Idle: use std::atomic instead of GMutex

src/ClientMessage.cxx

@@ -19,7 +19,6 @@
 
 #include "ClientMessage.hxx"
 
-#include <assert.h>
 #include <glib.h>
 
 G_GNUC_PURE

src/Idle.cxx

@@ -26,11 +26,11 @@
 #include "Idle.hxx"
 #include "GlobalEvents.hxx"
 
+#include <atomic>
+
 #include <assert.h>
-#include <glib.h>
 
-static unsigned idle_flags;
-static GMutex *idle_mutex = NULL;
+static std::atomic_uint idle_flags;
 
 static const char *const idle_names[] = {
 	"database",
@@ -44,32 +44,15 @@ static const char *const idle_names[] = {
 	"update",
 	"subscription",
 	"message",
-        NULL
+	nullptr
 };
 
 void
-idle_init(void)
-{
-	g_assert(idle_mutex == NULL);
-	idle_mutex = g_mutex_new();
-}
-
-void
-idle_deinit(void)
-{
-	g_assert(idle_mutex != NULL);
-	g_mutex_free(idle_mutex);
-	idle_mutex = NULL;
-}
-
-void
 idle_add(unsigned flags)
 {
 	assert(flags != 0);
 
-	g_mutex_lock(idle_mutex);
 	idle_flags |= flags;
-	g_mutex_unlock(idle_mutex);
 
 	GlobalEvents::Emit(GlobalEvents::IDLE);
 }
@@ -77,14 +60,7 @@ idle_add(unsigned flags)
 unsigned
 idle_get(void)
 {
-	unsigned flags;
-
-	g_mutex_lock(idle_mutex);
-	flags = idle_flags;
-	idle_flags = 0;
-	g_mutex_unlock(idle_mutex);
-
-	return flags;
+	return idle_flags.fetch_and(0);
 }
 
 const char*const*

src/Idle.hxx

@@ -62,18 +62,6 @@ enum {
 };
 
 /**
- * Initialize the mutex
- */
-void
-idle_init(void);
-
-/**
- * Destroy the mutex
- */
-void
-idle_deinit(void);
-
-/**
  * Adds idle flag (with bitwise "or") and queues notifications to all
  * clients.
  */

src/Main.cxx

@@ -375,7 +375,6 @@ int mpd_main(int argc, char *argv[])
 
 	io_thread_init();
 	winsock_init();
-	idle_init();
 	config_global_init();
 
 	success = parse_cmdline(argc, argv, &options, &error);
@@ -565,7 +564,6 @@ int mpd_main(int argc, char *argv[])
 	archive_plugin_deinit_all();
 #endif
 	config_global_finish();
-	idle_deinit();
 	stats_global_finish();
 	io_thread_deinit();
 	daemonize_finish();