[Musicpd-dev-team] [PATCH] pulse mixer

2009-03-06 Thread David Guibert
This patch introduces the mixer for the pulse output.

Technically speaking, the pulse index is needed to get or set
the volume. You must define callback fonctions to get this index since
the pulse output in mpd is done using the simpe api. The pulse simple api
does not provide the index of the newly defined output.

So callback fonctions are associated to the pulse context.
The list of all the sink input is then retreived.
Then we select the name of the mpd pulse output and control
its volume by its associated index number.

Signed-off-by: Patrice Linel patnathan...@gmail.com
Signed-off-by: David Guibert david.guib...@gmail.com

---
 Makefile.am   |1 +
 src/main.c|3 +-
 src/mixer/pulse_mixer.c   |  284 +
 src/mixer_api.h   |1 +
 src/output/pulse_plugin.c |   16 +++
 5 files changed, 303 insertions(+), 2 deletions(-)

diff --git a/Makefile.am b/Makefile.am
index 25478f8..2511680 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -489,6 +489,7 @@ endif
 
 if HAVE_PULSE
 OUTPUT_SRC += src/output/pulse_plugin.c
+MIXER_SRC  += src/mixer/pulse_mixer.c
 endif
 
 if HAVE_SHOUT
diff --git a/src/main.c b/src/main.c
index a3f613d..2889933 100644
--- a/src/main.c
+++ b/src/main.c
@@ -259,6 +259,7 @@ int main(int argc, char *argv[])
 
command_init();
initialize_decoder_and_player();
+   daemonize(options.daemon);
volume_init();
initAudioConfig();
audio_output_all_init();
@@ -267,8 +268,6 @@ int main(int argc, char *argv[])
initNormalization();
input_stream_global_init();
 
-   daemonize(options.daemon);
-
setup_log_output(options.stdOutput);
 
initSigHandlers();
diff --git a/src/mixer/pulse_mixer.c b/src/mixer/pulse_mixer.c
new file mode 100644
index 000..78b77c0
--- /dev/null
+++ b/src/mixer/pulse_mixer.c
@@ -0,0 +1,284 @@
+#include ../output_api.h
+#include ../mixer_api.h
+
+#include glib.h
+#include pulse/volume.h
+#include pulse/pulseaudio.h
+
+struct pulse_mixer {
+   struct mixer base;
+   char *server;
+   char *sink;
+   char *output_name;
+   uint32_t index;
+   boolonline;
+   struct pa_context *context;
+   struct pa_threaded_mainloop *mainloop;
+   struct pa_cvolume *volume;
+
+};
+
+
+
+static void sink_input_cb(G_GNUC_UNUSED pa_context *context, const 
pa_sink_input_info *i, int eol, void *userdata)
+{
+
+   struct pulse_mixer *pm = userdata;
+   if (eol) {
+   g_debug(eol error sink_input_cb\n);
+   return;
+   }
+
+   if (!i) {
+   g_debug(Sink input callback failure\n);
+   return;
+   }
+   g_debug(sink input cb %s, index %d \n,i-name,i-index);
+   if(strcmp(i-name,pm-output_name)==0)
+   { 
+   pm-index=i-index;
+   pm-online=true;
+   *pm-volume=i-volume;
+   }
+   else g_debug(bad name\n);
+}
+
+static void sink_input_vol(G_GNUC_UNUSED pa_context *context, const 
pa_sink_input_info *i, int eol, void *userdata) 
+{
+
+   struct pulse_mixer *pm = userdata;
+   if (eol) {
+   g_debug(eol error sink_input_vol\n);
+   return;
+   }
+
+   if (!i) {
+   g_debug(Sink input callback failure\n);
+   return;
+   }
+   g_debug(sink input vol %s, index %d \n,i-name,i-index);
+   *pm-volume=i-volume;
+}
+
+static void 
+subscribe_cb(G_GNUC_UNUSED pa_context *c, pa_subscription_event_type_t t, 
uint32_t idx, void *userdata) 
+{
+
+   struct pulse_mixer *pm = userdata;
+   g_debug(pulse_mixer: subscribe call back\n);
+   switch (t  PA_SUBSCRIPTION_EVENT_FACILITY_MASK) 
+   {
+
+   case PA_SUBSCRIPTION_EVENT_SINK_INPUT:
+   if ((t  PA_SUBSCRIPTION_EVENT_TYPE_MASK) == 
+   PA_SUBSCRIPTION_EVENT_REMOVE)
+   pm-online =false;
+   else 
+   {
+   pa_operation *o;
+
+   if (!(o = 
pa_context_get_sink_input_info(pm-context, idx, sink_input_cb, pm))) 
+   {
+   g_debug(pulse_mixer: 
pa_context_get_sink_input_info() failed\n);
+   return;
+   }
+   pa_operation_unref(o);
+   }
+   break;
+   }
+}
+
+static void
+context_state_cb(pa_context *context, void *userdata) 
+{
+   struct pulse_mixer *pm = userdata;
+   switch (pa_context_get_state(context)) 
+   {
+   case PA_CONTEXT_READY: 
+   {
+   pa_operation *o;
+
+   pa_context_set_subscribe_callback(context, 
subscribe_cb, pm);
+
+   if (!(o = pa_context_subscribe(context, 

[Musicpd-dev-team] [PATCH] pulse mixer

2009-03-06 Thread David Guibert
Sorry. I forgot to do the make after minor changes...

I resend the patch wich compiles cleanly.

--
Open Source Business Conference (OSBC), March 24-25, 2009, San Francisco, CA
-OSBC tackles the biggest issue in open source: Open Sourcing the Enterprise
-Strategies to boost innovation and cut costs with open source participation
-Receive a $600 discount off the registration fee with the source code: SFAD
http://p.sf.net/sfu/XcvMzF8H
___
Musicpd-dev-team mailing list
Musicpd-dev-team@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/musicpd-dev-team


[Musicpd-dev-team] [PATCH] pulse mixer

2009-03-06 Thread David Guibert
This patch introduces the mixer for the pulse output.

Technically speaking, the pulse index is needed to get or set
the volume. You must define callback fonctions to get this index since
the pulse output in mpd is done using the simpe api. The pulse simple api
does not provide the index of the newly defined output.

So callback fonctions are associated to the pulse context.
The list of all the sink input is then retreived.
Then we select the name of the mpd pulse output and control
its volume by its associated index number.

Signed-off-by: Patrice Linel patnathan...@gmail.com
Signed-off-by: David Guibert david.guib...@gmail.com

---
 Makefile.am   |1 +
 src/main.c|3 +-
 src/mixer/pulse_mixer.c   |  284 +
 src/mixer_api.h   |1 +
 src/output/pulse_plugin.c |   16 +++
 5 files changed, 303 insertions(+), 2 deletions(-)

diff --git a/Makefile.am b/Makefile.am
index 25478f8..2511680 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -489,6 +489,7 @@ endif
 
 if HAVE_PULSE
 OUTPUT_SRC += src/output/pulse_plugin.c
+MIXER_SRC  += src/mixer/pulse_mixer.c
 endif
 
 if HAVE_SHOUT
diff --git a/src/main.c b/src/main.c
index a3f613d..2889933 100644
--- a/src/main.c
+++ b/src/main.c
@@ -259,6 +259,7 @@ int main(int argc, char *argv[])
 
command_init();
initialize_decoder_and_player();
+   daemonize(options.daemon);
volume_init();
initAudioConfig();
audio_output_all_init();
@@ -267,8 +268,6 @@ int main(int argc, char *argv[])
initNormalization();
input_stream_global_init();
 
-   daemonize(options.daemon);
-
setup_log_output(options.stdOutput);
 
initSigHandlers();
diff --git a/src/mixer/pulse_mixer.c b/src/mixer/pulse_mixer.c
new file mode 100644
index 000..e12a844
--- /dev/null
+++ b/src/mixer/pulse_mixer.c
@@ -0,0 +1,284 @@
+#include ../output_api.h
+#include ../mixer_api.h
+
+#include glib.h
+#include pulse/volume.h
+#include pulse/pulseaudio.h
+
+struct pulse_mixer {
+   struct mixer base;
+   char *server;
+   char *sink;
+   char *output_name;
+   uint32_t index;
+   boolonline;
+   struct pa_context *context;
+   struct pa_threaded_mainloop *mainloop;
+   struct pa_cvolume *volume;
+
+};
+
+
+
+static void sink_input_cb(G_GNUC_UNUSED pa_context *context, const 
pa_sink_input_info *i, int eol, void *userdata)
+{
+
+   struct pulse_mixer *pm = userdata;
+   if (eol) {
+   g_debug(eol error sink_input_cb\n);
+   return;
+   }
+
+   if (!i) {
+   g_debug(Sink input callback failure\n);
+   return;
+   }
+   g_debug(sink input cb %s, index %d \n,i-name,i-index);
+   if(strcmp(i-name,pm-output_name)==0)
+   { 
+   pm-index=i-index;
+   pm-online=true;
+   *pm-volume=i-volume;
+   }
+   else g_debug(bad name\n);
+}
+
+static void sink_input_vol(G_GNUC_UNUSED pa_context *context, const 
pa_sink_input_info *i, int eol, void *userdata) 
+{
+
+   struct pulse_mixer *pm = userdata;
+   if (eol) {
+   g_debug(eol error sink_input_vol\n);
+   return;
+   }
+
+   if (!i) {
+   g_debug(Sink input callback failure\n);
+   return;
+   }
+   g_debug(sink input vol %s, index %d \n,i-name,i-index);
+   *pm-volume=i-volume;
+}
+
+static void 
+subscribe_cb(G_GNUC_UNUSED pa_context *c, pa_subscription_event_type_t t, 
uint32_t idx, void *userdata) 
+{
+
+   struct pulse_mixer *pm = userdata;
+   g_debug(pulse_mixer: subscribe call back\n);
+   switch (t  PA_SUBSCRIPTION_EVENT_FACILITY_MASK) 
+   {
+
+   case PA_SUBSCRIPTION_EVENT_SINK_INPUT:
+   if ((t  PA_SUBSCRIPTION_EVENT_TYPE_MASK) == 
+   PA_SUBSCRIPTION_EVENT_REMOVE)
+   pm-online =false;
+   else 
+   {
+   pa_operation *o;
+
+   if (!(o = 
pa_context_get_sink_input_info(pm-context, idx, sink_input_cb, pm))) 
+   {
+   g_debug(pulse_mixer: 
pa_context_get_sink_input_info() failed\n);
+   return;
+   }
+   pa_operation_unref(o);
+   }
+   break;
+   }
+}
+
+static void
+context_state_cb(pa_context *context, void *userdata) 
+{
+   struct pulse_mixer *pm = userdata;
+   switch (pa_context_get_state(context)) 
+   {
+   case PA_CONTEXT_READY: 
+   {
+   pa_operation *o;
+
+   pa_context_set_subscribe_callback(context, 
subscribe_cb, pm);
+
+   if (!(o = pa_context_subscribe(context,