aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2013-04-16 23:46:36 +0200
committerMax Kellermann <max@duempel.org>2013-04-16 23:50:41 +0200
commit7b5f7c041b5926cb151e0d42cd602498bd697369 (patch)
tree7cb8473b081591f5fafb45ea398f4130c81190e9
parent8ce9b530933bb260dccd136b1f3cb553adac8ca3 (diff)
downloadmpd-7b5f7c041b5926cb151e0d42cd602498bd697369.tar.gz
mpd-7b5f7c041b5926cb151e0d42cd602498bd697369.tar.xz
mpd-7b5f7c041b5926cb151e0d42cd602498bd697369.zip
MixerInternal: use Mutex instead of GMutex
Diffstat (limited to '')
-rw-r--r--src/MixerControl.cxx31
-rw-r--r--src/MixerInternal.hxx6
2 files changed, 10 insertions, 27 deletions
diff --git a/src/MixerControl.cxx b/src/MixerControl.cxx
index 7bba4d46b..8a3fbd6b3 100644
--- a/src/MixerControl.cxx
+++ b/src/MixerControl.cxx
@@ -21,6 +21,8 @@
#include "MixerControl.hxx"
#include "MixerInternal.hxx"
+#include <glib.h>
+
#include <assert.h>
#include <stddef.h>
@@ -48,14 +50,11 @@ mixer_free(Mixer *mixer)
{
assert(mixer != NULL);
assert(mixer->plugin != NULL);
- assert(mixer->mutex != NULL);
/* mixers with the "global" flag set might still be open at
this point (see mixer_auto_close()) */
mixer_close(mixer);
- g_mutex_free(mixer->mutex);
-
mixer->plugin->finish(mixer);
}
@@ -67,7 +66,7 @@ mixer_open(Mixer *mixer, GError **error_r)
assert(mixer != NULL);
assert(mixer->plugin != NULL);
- g_mutex_lock(mixer->mutex);
+ const ScopeLock protect(mixer->mutex);
if (mixer->open)
success = true;
@@ -78,8 +77,6 @@ mixer_open(Mixer *mixer, GError **error_r)
mixer->failed = !success;
- g_mutex_unlock(mixer->mutex);
-
return success;
}
@@ -102,12 +99,10 @@ mixer_close(Mixer *mixer)
assert(mixer != NULL);
assert(mixer->plugin != NULL);
- g_mutex_lock(mixer->mutex);
+ const ScopeLock protect(mixer->mutex);
if (mixer->open)
mixer_close_internal(mixer);
-
- g_mutex_unlock(mixer->mutex);
}
void
@@ -142,7 +137,7 @@ mixer_get_volume(Mixer *mixer, GError **error_r)
!mixer_open(mixer, error_r))
return -1;
- g_mutex_lock(mixer->mutex);
+ const ScopeLock protect(mixer->mutex);
if (mixer->open) {
GError *error = NULL;
@@ -155,16 +150,12 @@ mixer_get_volume(Mixer *mixer, GError **error_r)
} else
volume = -1;
- g_mutex_unlock(mixer->mutex);
-
return volume;
}
bool
mixer_set_volume(Mixer *mixer, unsigned volume, GError **error_r)
{
- bool success;
-
assert(mixer != NULL);
assert(volume <= 100);
@@ -172,14 +163,8 @@ mixer_set_volume(Mixer *mixer, unsigned volume, GError **error_r)
!mixer_open(mixer, error_r))
return false;
- g_mutex_lock(mixer->mutex);
-
- if (mixer->open) {
- success = mixer->plugin->set_volume(mixer, volume, error_r);
- } else
- success = false;
-
- g_mutex_unlock(mixer->mutex);
+ const ScopeLock protect(mixer->mutex);
- return success;
+ return mixer->open &&
+ mixer->plugin->set_volume(mixer, volume, error_r);
}
diff --git a/src/MixerInternal.hxx b/src/MixerInternal.hxx
index f922a22bd..e421a34b4 100644
--- a/src/MixerInternal.hxx
+++ b/src/MixerInternal.hxx
@@ -22,8 +22,7 @@
#include "MixerPlugin.hxx"
#include "MixerList.hxx"
-
-#include <glib.h>
+#include "thread/Mutex.hxx"
class Mixer {
public:
@@ -33,7 +32,7 @@ public:
* This mutex protects all of the mixer struct, including its
* implementation, so plugins don't have to deal with that.
*/
- GMutex *mutex;
+ Mutex mutex;
/**
* Is the mixer device currently open?
@@ -49,7 +48,6 @@ public:
public:
Mixer(const mixer_plugin &_plugin)
:plugin(&_plugin),
- mutex(g_mutex_new()),
open(false),
failed(false) {}