]> git.sesse.net Git - vlc/blobdiff - src/audio_output/intf.c
aout_VolumeDown: convert to a macro
[vlc] / src / audio_output / intf.c
index 9b6afa52a6f554c38162bdde85d3d9324fe37380..bcb6f1e67e060a13d43f99950b62cf412844b964 100644 (file)
@@ -30,6 +30,7 @@
 #endif
 
 #include <vlc_common.h>
+#include <vlc_aout_intf.h>
 
 #include <stdio.h>
 #include <stdlib.h>                            /* calloc(), malloc(), free() */
@@ -40,7 +41,7 @@
 
 #include <vlc_playlist.h>
 
-static aout_instance_t *findAout (vlc_object_t *obj)
+static audio_output_t *findAout (vlc_object_t *obj)
 {
     input_thread_t *(*pf_find_input) (vlc_object_t *);
 
@@ -52,17 +53,17 @@ static aout_instance_t *findAout (vlc_object_t *obj)
     if (p_input == NULL)
        return NULL;
 
-    aout_instance_t *p_aout = input_GetAout (p_input);
+    audio_output_t *p_aout = input_GetAout (p_input);
     vlc_object_release (p_input);
     return p_aout;
 }
 #define findAout(o) findAout(VLC_OBJECT(o))
 
 /** Start a volume change transaction. */
-static void prepareVolume (vlc_object_t *obj, aout_instance_t **aoutp,
+static void prepareVolume (vlc_object_t *obj, audio_output_t **aoutp,
                            audio_volume_t *volp, bool *mutep)
 {
-    aout_instance_t *aout = findAout (obj);
+    audio_output_t *aout = findAout (obj);
 
     /* FIXME: we need interlocking even if aout does not exist! */
     *aoutp = aout;
@@ -75,7 +76,7 @@ static void prepareVolume (vlc_object_t *obj, aout_instance_t **aoutp,
 }
 
 /** Commit a volume change transaction. */
-static int commitVolume (vlc_object_t *obj, aout_instance_t *aout,
+static int commitVolume (vlc_object_t *obj, audio_output_t *aout,
                          audio_volume_t volume, bool mute)
 {
     int ret = 0;
@@ -85,12 +86,13 @@ static int commitVolume (vlc_object_t *obj, aout_instance_t *aout,
 
     if (aout != NULL)
     {
-        aout_lock_mixer (aout);
-        aout_lock_input_fifos (aout);
+        float vol = volume / (float)AOUT_VOLUME_DEFAULT;
+
+        aout_lock (aout);
+#warning FIXME: wrong test. Need to check that aout_output is ready.
         if (aout->p_mixer != NULL)
-            ret = aout->output.pf_volume_set (aout, volume, mute);
-        aout_unlock_input_fifos (aout);
-        aout_unlock_mixer (aout);
+            ret = aout->pf_volume_set (aout, vol, mute);
+        aout_unlock (aout);
 
         if (ret == 0)
             var_TriggerCallback (aout, "intf-change");
@@ -102,7 +104,7 @@ static int commitVolume (vlc_object_t *obj, aout_instance_t *aout,
 
 #if 0
 /** Cancel a volume change transaction. */
-static void cancelVolume (vlc_object_t *obj, aout_instance_t *aout)
+static void cancelVolume (vlc_object_t *obj, audio_output_t *aout)
 {
     (void) obj;
     if (aout != NULL)
@@ -120,7 +122,7 @@ static void cancelVolume (vlc_object_t *obj, aout_instance_t *aout)
 audio_volume_t aout_VolumeGet (vlc_object_t *obj)
 {
 #if 0
-    aout_instance_t *aout;
+    audio_output_t *aout;
     audio_volume_t volume;
 
     prepareVolume (obj, &aout, &volume, NULL);
@@ -138,7 +140,7 @@ audio_volume_t aout_VolumeGet (vlc_object_t *obj)
  */
 int aout_VolumeSet (vlc_object_t *obj, audio_volume_t volume)
 {
-    aout_instance_t *aout;
+    audio_output_t *aout;
     bool mute;
 
     prepareVolume (obj, &aout, NULL, &mute);
@@ -153,7 +155,7 @@ int aout_VolumeSet (vlc_object_t *obj, audio_volume_t volume)
  */
 int aout_VolumeUp (vlc_object_t *obj, int value, audio_volume_t *volp)
 {
-    aout_instance_t *aout;
+    audio_output_t *aout;
     int ret;
     audio_volume_t volume;
     bool mute;
@@ -162,8 +164,8 @@ int aout_VolumeUp (vlc_object_t *obj, int value, audio_volume_t *volp)
 
     prepareVolume (obj, &aout, &volume, &mute);
     value += volume;
-    if (value < AOUT_VOLUME_MIN)
-        volume = AOUT_VOLUME_MIN;
+    if (value < 0)
+        volume = 0;
     else
     if (value > AOUT_VOLUME_MAX)
         volume = AOUT_VOLUME_MAX;
@@ -175,22 +177,13 @@ int aout_VolumeUp (vlc_object_t *obj, int value, audio_volume_t *volp)
     return ret;
 }
 
-#undef aout_VolumeDown
-/**
- * Lowers the volume. See aout_VolumeUp().
- */
-int aout_VolumeDown (vlc_object_t *obj, int steps, audio_volume_t *volp)
-{
-    return aout_VolumeUp (obj, -steps, volp);
-}
-
 #undef aout_ToggleMute
 /**
  * Toggles the mute state.
  */
 int aout_ToggleMute (vlc_object_t *obj, audio_volume_t *volp)
 {
-    aout_instance_t *aout;
+    audio_output_t *aout;
     int ret;
     audio_volume_t volume;
     bool mute;
@@ -199,7 +192,7 @@ int aout_ToggleMute (vlc_object_t *obj, audio_volume_t *volp)
     mute = !mute;
     ret = commitVolume (obj, aout, volume, mute);
     if (volp != NULL)
-        *volp = mute ? AOUT_VOLUME_MIN : volume;
+        *volp = mute ? 0 : volume;
     return ret;
 }
 
@@ -209,7 +202,7 @@ int aout_ToggleMute (vlc_object_t *obj, audio_volume_t *volp)
 bool aout_IsMuted (vlc_object_t *obj)
 {
 #if 0
-    aout_instance_t *aout;
+    audio_output_t *aout;
     bool mute;
 
     prepareVolume (obj, &aout, NULL, &mute);
@@ -225,14 +218,14 @@ bool aout_IsMuted (vlc_object_t *obj)
  */
 int aout_SetMute (vlc_object_t *obj, audio_volume_t *volp, bool mute)
 {
-    aout_instance_t *aout;
+    audio_output_t *aout;
     int ret;
     audio_volume_t volume;
 
     prepareVolume (obj, &aout, &volume, NULL);
     ret = commitVolume (obj, aout, volume, mute);
     if (volp != NULL)
-        *volp = mute ? AOUT_VOLUME_MIN : volume;
+        *volp = mute ? 0 : volume;
     return ret;
 }
 
@@ -241,22 +234,20 @@ int aout_SetMute (vlc_object_t *obj, audio_volume_t *volp, bool mute)
  * The next functions are not supposed to be called by the interface, but
  * are placeholders for software-only scaling.
  */
-static int aout_VolumeSoftSet (aout_instance_t *aout, audio_volume_t volume,
-                               bool mute)
+static int aout_VolumeSoftSet (audio_output_t *aout, float volume, bool mute)
 {
-    float f = mute ? 0. : (volume / (float)AOUT_VOLUME_DEFAULT);
-    aout->mixer_multiplier = f;
+    aout->mixer_multiplier = mute ? 0. : volume;
     return 0;
 }
 
 /* Meant to be called by the output plug-in's Open(). */
-void aout_VolumeSoftInit (aout_instance_t *aout)
+void aout_VolumeSoftInit (audio_output_t *aout)
 {
     audio_volume_t volume = var_InheritInteger (aout, "volume");
     bool mute = var_InheritBool (aout, "mute");
 
-    aout->output.pf_volume_set = aout_VolumeSoftSet;
-    aout_VolumeSoftSet (aout, volume, mute);
+    aout->pf_volume_set = aout_VolumeSoftSet;
+    aout_VolumeSoftSet (aout, volume / (float)AOUT_VOLUME_DEFAULT, mute);
 }
 
 
@@ -264,17 +255,16 @@ void aout_VolumeSoftInit (aout_instance_t *aout)
  * The next functions are not supposed to be called by the interface, but
  * are placeholders for unsupported scaling.
  */
-static int aout_VolumeNoneSet (aout_instance_t *aout, audio_volume_t volume,
-                               bool mute)
+static int aout_VolumeNoneSet (audio_output_t *aout, float volume, bool mute)
 {
     (void)aout; (void)volume; (void)mute;
     return -1;
 }
 
 /* Meant to be called by the output plug-in's Open(). */
-void aout_VolumeNoneInit( aout_instance_t * p_aout )
+void aout_VolumeNoneInit( audio_output_t * p_aout )
 {
-    p_aout->output.pf_volume_set = aout_VolumeNoneSet;
+    p_aout->pf_volume_set = aout_VolumeNoneSet;
 }
 
 
@@ -289,60 +279,48 @@ void aout_VolumeNoneInit( aout_instance_t * p_aout )
  * This function is used whenever the parameters of the output plug-in are
  * changed (eg. selecting S/PDIF or PCM).
  *****************************************************************************/
-static int aout_Restart( aout_instance_t * p_aout )
+static int aout_Restart( audio_output_t * p_aout )
 {
-    bool b_error = 0;
+    aout_input_t *p_input;
 
-    aout_lock_mixer( p_aout );
-
-    if( p_aout->p_input == NULL )
+    aout_lock( p_aout );
+    p_input = p_aout->p_input;
+    if( p_input == NULL )
     {
-        aout_unlock_mixer( p_aout );
+        aout_unlock( p_aout );
         msg_Err( p_aout, "no decoder thread" );
         return -1;
     }
 
-    aout_lock_input( p_aout, p_aout->p_input );
-    aout_lock_input_fifos( p_aout );
-    aout_InputDelete( p_aout, p_aout->p_input );
-    aout_unlock_input_fifos( p_aout );
-
-    /* Lock all inputs. */
-    aout_lock_input_fifos( p_aout );
+    /* Reinitializes the output */
+    aout_InputDelete( p_aout, p_input );
     aout_MixerDelete( p_aout );
-
-    /* Re-open the output plug-in. */
     aout_OutputDelete( p_aout );
 
     /* FIXME: This function is notoriously dangerous/unsafe.
      * By the way, if OutputNew or MixerNew fails, we are totally screwed. */
-    if ( aout_OutputNew( p_aout, &p_aout->p_input->input ) == -1 )
+    if ( aout_OutputNew( p_aout, &p_input->input ) == -1 )
     {
         /* Release all locks and report the error. */
-        vlc_mutex_unlock( &p_aout->p_input->lock );
-        aout_unlock_input_fifos( p_aout );
-        aout_unlock_mixer( p_aout );
+        aout_unlock( p_aout );
         return -1;
     }
 
     if ( aout_MixerNew( p_aout ) == -1 )
     {
         aout_OutputDelete( p_aout );
-        vlc_mutex_unlock( &p_aout->p_input->lock );
-        aout_unlock_input_fifos( p_aout );
-        aout_unlock_mixer( p_aout );
+        aout_unlock( p_aout );
         return -1;
     }
 
-    /* Re-open the input. */
-    aout_input_t * p_input = p_aout->p_input;
-    b_error |= aout_InputNew( p_aout, p_input, &p_input->request_vout );
-    aout_unlock_input( p_aout, p_input );
-
-    aout_unlock_input_fifos( p_aout );
-    aout_unlock_mixer( p_aout );
-
-    return b_error;
+    if( aout_InputNew( p_aout, p_input, &p_input->request_vout ) )
+    {
+#warning FIXME: deal with errors
+        aout_unlock( p_aout );
+        return -1;
+    }
+    aout_unlock( p_aout );
+    return 0;
 }
 
 /*****************************************************************************
@@ -352,7 +330,7 @@ int aout_ChannelsRestart( vlc_object_t * p_this, const char * psz_variable,
                           vlc_value_t oldval, vlc_value_t newval,
                           void *p_data )
 {
-    aout_instance_t * p_aout = (aout_instance_t *)p_this;
+    audio_output_t * p_aout = (audio_output_t *)p_this;
     (void)oldval; (void)newval; (void)p_data;
 
     if ( !strcmp( psz_variable, "audio-device" ) )
@@ -374,7 +352,7 @@ int aout_ChannelsRestart( vlc_object_t * p_this, const char * psz_variable,
 void aout_EnableFilter( vlc_object_t *p_this, const char *psz_name,
                         bool b_add )
 {
-    aout_instance_t *p_aout = findAout( p_this );
+    audio_output_t *p_aout = findAout( p_this );
 
     if( aout_ChangeFilterString( p_this, p_aout, "audio-filter", psz_name, b_add ) )
     {