From b8cda53bd36809b6f61c2e42a673dfdcd43f0c1d Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Thu, 10 Jan 2013 10:44:04 +0100 Subject: notify: convert to C++ --- Makefile.am | 3 +-- src/OutputAll.cxx | 11 ++-------- src/OutputControl.cxx | 4 ++-- src/OutputThread.cxx | 4 ++-- src/notify.c | 58 --------------------------------------------------- src/notify.cxx | 45 +++++++++++++++++++++++++++++++++++++++ src/notify.h | 53 ---------------------------------------------- src/notify.hxx | 53 ++++++++++++++++++++++++++++++++++++++++++++++ 8 files changed, 105 insertions(+), 126 deletions(-) delete mode 100644 src/notify.c create mode 100644 src/notify.cxx delete mode 100644 src/notify.h create mode 100644 src/notify.hxx diff --git a/Makefile.am b/Makefile.am index 146170060..8011e104a 100644 --- a/Makefile.am +++ b/Makefile.am @@ -46,7 +46,6 @@ src_mpd_LDADD = \ mpd_headers = \ src/check.h \ - src/notify.h \ src/ack.h \ src/ape.h \ src/audio_format.h \ @@ -166,7 +165,7 @@ src_mpd_SOURCES = \ src/thread/GLibCond.hxx \ src/glib_socket.h \ src/clock.c src/clock.h \ - src/notify.c \ + src/notify.cxx src/notify.hxx \ src/audio_config.c src/audio_config.h \ src/audio_check.c \ src/audio_format.c \ 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 #include @@ -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.h deleted file mode 100644 index 40821690c..000000000 --- a/src/notify.h +++ /dev/null @@ -1,53 +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. - */ - -#ifndef MPD_NOTIFY_H -#define MPD_NOTIFY_H - -#include - -#include - -struct notify { - GMutex *mutex; - GCond *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); - -#endif diff --git a/src/notify.hxx b/src/notify.hxx new file mode 100644 index 000000000..6b9e95368 --- /dev/null +++ b/src/notify.hxx @@ -0,0 +1,53 @@ +/* + * 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. + */ + +#ifndef MPD_NOTIFY_HXX +#define MPD_NOTIFY_HXX + +#include "thread/Mutex.hxx" +#include "thread/Cond.hxx" + +struct notify { + Mutex mutex; + Cond cond; + bool pending; + +#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 -- cgit v1.2.3