aboutsummaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2013-01-10 10:44:04 +0100
committerMax Kellermann <max@duempel.org>2013-01-10 10:44:04 +0100
commitb8cda53bd36809b6f61c2e42a673dfdcd43f0c1d (patch)
tree297e3f5d87d2304a7311549399d1896d377d0099 /src
parente0a97a030fc4c6f978b2855c32f5da98b3e7fb78 (diff)
downloadmpd-b8cda53bd36809b6f61c2e42a673dfdcd43f0c1d.tar.gz
mpd-b8cda53bd36809b6f61c2e42a673dfdcd43f0c1d.tar.xz
mpd-b8cda53bd36809b6f61c2e42a673dfdcd43f0c1d.zip
notify: convert to C++
Diffstat (limited to '')
-rw-r--r--src/OutputAll.cxx11
-rw-r--r--src/OutputControl.cxx4
-rw-r--r--src/OutputThread.cxx4
-rw-r--r--src/notify.c58
-rw-r--r--src/notify.cxx45
-rw-r--r--src/notify.hxx (renamed from src/notify.h)56
6 files changed, 79 insertions, 99 deletions
diff --git a/src/OutputAll.cxx b/src/OutputAll.cxx
index f83f51c0c..a18a63385 100644
--- a/src/OutputAll.cxx
+++ b/src/OutputAll.cxx
@@ -32,10 +32,7 @@ extern "C" {
#include "MusicChunk.hxx"
#include "mpd_error.h"
#include "conf.h"
-
-extern "C" {
-#include "notify.h"
-}
+#include "notify.hxx"
#include <assert.h>
#include <string.h>
@@ -113,8 +110,6 @@ audio_output_all_init(struct player_control *pc)
unsigned int i;
GError *error = NULL;
- notify_init(&audio_output_client_notify);
-
num_audio_outputs = audio_output_config_count();
audio_outputs = g_new(struct audio_output *, num_audio_outputs);
@@ -161,8 +156,6 @@ audio_output_all_finish(void)
g_free(audio_outputs);
audio_outputs = NULL;
num_audio_outputs = 0;
-
- notify_deinit(&audio_output_client_notify);
}
void
@@ -211,7 +204,7 @@ audio_output_all_finished(void)
static void audio_output_wait_all(void)
{
while (!audio_output_all_finished())
- notify_wait(&audio_output_client_notify);
+ audio_output_client_notify.Wait();
}
/**
diff --git a/src/OutputControl.cxx b/src/OutputControl.cxx
index 005003d2d..13625ade2 100644
--- a/src/OutputControl.cxx
+++ b/src/OutputControl.cxx
@@ -26,9 +26,9 @@ extern "C" {
#include "output_internal.h"
#include "mixer_control.h"
#include "mixer_plugin.h"
-#include "notify.h"
}
+#include "notify.hxx"
#include "filter/ReplayGainFilterPlugin.hxx"
#include "filter_plugin.h"
@@ -52,7 +52,7 @@ static void ao_command_wait(struct audio_output *ao)
{
while (ao->command != AO_COMMAND_NONE) {
g_mutex_unlock(ao->mutex);
- notify_wait(&audio_output_client_notify);
+ audio_output_client_notify.Wait();
g_mutex_lock(ao->mutex);
}
}
diff --git a/src/OutputThread.cxx b/src/OutputThread.cxx
index 6fe833330..59394e072 100644
--- a/src/OutputThread.cxx
+++ b/src/OutputThread.cxx
@@ -26,9 +26,9 @@ extern "C" {
#include "pcm_mix.h"
#include "filter_plugin.h"
#include "filter/convert_filter_plugin.h"
-#include "notify.h"
}
+#include "notify.hxx"
#include "filter/ReplayGainFilterPlugin.hxx"
#include "PlayerControl.hxx"
#include "MusicPipe.hxx"
@@ -52,7 +52,7 @@ static void ao_command_finished(struct audio_output *ao)
ao->command = AO_COMMAND_NONE;
g_mutex_unlock(ao->mutex);
- notify_signal(&audio_output_client_notify);
+ audio_output_client_notify.Signal();
g_mutex_lock(ao->mutex);
}
diff --git a/src/notify.c b/src/notify.c
deleted file mode 100644
index 3c0112c91..000000000
--- a/src/notify.c
+++ /dev/null
@@ -1,58 +0,0 @@
-/*
- * Copyright (C) 2003-2011 The Music Player Daemon Project
- * http://www.musicpd.org
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License along
- * with this program; if not, write to the Free Software Foundation, Inc.,
- * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
- */
-
-#include "config.h"
-#include "notify.h"
-
-void notify_init(struct notify *notify)
-{
- notify->mutex = g_mutex_new();
- notify->cond = g_cond_new();
- notify->pending = false;
-}
-
-void notify_deinit(struct notify *notify)
-{
- g_mutex_free(notify->mutex);
- g_cond_free(notify->cond);
-}
-
-void notify_wait(struct notify *notify)
-{
- g_mutex_lock(notify->mutex);
- while (!notify->pending)
- g_cond_wait(notify->cond, notify->mutex);
- notify->pending = false;
- g_mutex_unlock(notify->mutex);
-}
-
-void notify_signal(struct notify *notify)
-{
- g_mutex_lock(notify->mutex);
- notify->pending = true;
- g_cond_signal(notify->cond);
- g_mutex_unlock(notify->mutex);
-}
-
-void notify_clear(struct notify *notify)
-{
- g_mutex_lock(notify->mutex);
- notify->pending = false;
- g_mutex_unlock(notify->mutex);
-}
diff --git a/src/notify.cxx b/src/notify.cxx
new file mode 100644
index 000000000..64018968c
--- /dev/null
+++ b/src/notify.cxx
@@ -0,0 +1,45 @@
+/*
+ * Copyright (C) 2003-2013 The Music Player Daemon Project
+ * http://www.musicpd.org
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+ */
+
+#include "config.h"
+#include "notify.hxx"
+
+void
+notify::Wait()
+{
+ const ScopeLock protect(mutex);
+ while (!pending)
+ cond.wait(mutex);
+ pending = false;
+}
+
+void
+notify::Signal()
+{
+ const ScopeLock protect(mutex);
+ pending = true;
+ cond.signal();
+}
+
+void
+notify::Clear()
+{
+ const ScopeLock protect(mutex);
+ pending = false;
+}
diff --git a/src/notify.h b/src/notify.hxx
index 40821690c..6b9e95368 100644
--- a/src/notify.h
+++ b/src/notify.hxx
@@ -1,5 +1,5 @@
/*
- * Copyright (C) 2003-2011 The Music Player Daemon Project
+ * Copyright (C) 2003-2013 The Music Player Daemon Project
* http://www.musicpd.org
*
* This program is free software; you can redistribute it and/or modify
@@ -17,37 +17,37 @@
* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
*/
-#ifndef MPD_NOTIFY_H
-#define MPD_NOTIFY_H
+#ifndef MPD_NOTIFY_HXX
+#define MPD_NOTIFY_HXX
-#include <glib.h>
-
-#include <stdbool.h>
+#include "thread/Mutex.hxx"
+#include "thread/Cond.hxx"
struct notify {
- GMutex *mutex;
- GCond *cond;
+ Mutex mutex;
+ Cond cond;
bool pending;
-};
-
-void notify_init(struct notify *notify);
-
-void notify_deinit(struct notify *notify);
-
-/**
- * Wait for a notification. Return immediately if we have already
- * been notified since we last returned from notify_wait().
- */
-void notify_wait(struct notify *notify);
-/**
- * Notify the thread. This function never blocks.
- */
-void notify_signal(struct notify *notify);
-
-/**
- * Clears a pending notification.
- */
-void notify_clear(struct notify *notify);
+#ifndef WIN32
+ constexpr
+#endif
+ notify():pending(false) {}
+
+ /**
+ * Wait for a notification. Return immediately if we have already
+ * been notified since we last returned from notify_wait().
+ */
+ void Wait();
+
+ /**
+ * Notify the thread. This function never blocks.
+ */
+ void Signal();
+
+ /**
+ * Clears a pending notification.
+ */
+ void Clear();
+};
#endif