]> git.sesse.net Git - vlc/blobdiff - src/audio_output/intf.c
fix [0b988497] : initialize volume_vars_lock
[vlc] / src / audio_output / intf.c
index 17e59ba268bb5e6723392f100d9033a6f222d9fb..db55c4ade4d33a0d27866185bf5ac5865107fdcd 100644 (file)
@@ -1,8 +1,8 @@
 /*****************************************************************************
  * intf.c : audio output API towards the interface modules
  *****************************************************************************
- * Copyright (C) 2002 VideoLAN
- * $Id: intf.c,v 1.4 2002/09/26 22:40:25 massiot Exp $
+ * Copyright (C) 2002-2007 the VideoLAN team
+ * $Id$
  *
  * Authors: Christophe Massiot <massiot@via.ecp.fr>
  *
@@ -10,7 +10,7 @@
  * 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
  *
  * 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., 59 Temple Place - Suite 330, Boston, MA  02111, USA.
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
  *****************************************************************************/
 
 /*****************************************************************************
  * Preamble
  *****************************************************************************/
+
+#ifdef HAVE_CONFIG_H
+# include "config.h"
+#endif
+
+#include <vlc_common.h>
+
+#include <stdio.h>
 #include <stdlib.h>                            /* calloc(), malloc(), free() */
 #include <string.h>
 
-#include <vlc/vlc.h>
-
-#include "audio_output.h"
+#include <vlc_aout.h>
 #include "aout_internal.h"
 
-
 /*
  * Volume management
  *
@@ -41,7 +46,7 @@
  * mixer is running, so we need the mixer lock (too).
  *
  * Here is a schematic of the i_volume range :
- * 
+ *
  * |------------------------------+---------------------------------------|
  * 0                           pi_soft                                   1024
  *
  * a very low volume and users may complain.
  */
 
+enum {
+    SET_MUTE=1,
+    SET_VOLUME=2,
+    INCREMENT_VOLUME=4,
+    TOGGLE_MUTE=8
+};
 /*****************************************************************************
- * aout_VolumeGet : get the volume of the output device
+ * doVolumeChanges : handle all volume changes. Internal use only to ease
+ *                   variables locking.
  *****************************************************************************/
-int aout_VolumeGet( aout_instance_t * p_aout, audio_volume_t * pi_volume )
+int doVolumeChanges( unsigned action, vlc_object_t * p_object, int i_nb_steps,
+                audio_volume_t i_volume, audio_volume_t * i_return_volume,
+                bool b_mute )
 {
-    int i_result;
-
-    vlc_mutex_lock( &p_aout->mixer_lock );
+    int i_result = VLC_SUCCESS;
+    int i_volume_step = 1;
+    bool b_var_mute = false;
+    aout_instance_t *p_aout = vlc_object_find( p_object, VLC_OBJECT_AOUT,
+                                               FIND_ANYWHERE );
+
+    if ( p_aout ) aout_lock_volume( p_aout );
+
+    b_var_mute = (bool)var_GetBool( p_object->p_libvlc, "volume-muted");
+
+    const bool b_unmute_condition = ( /* Also unmute on increments */
+                    ( action == INCREMENT_VOLUME )
+                    || /* On explicit unmute */
+                    ( ( action == SET_MUTE ) && ( b_var_mute && !b_mute ) )
+                    || /* On toggle from muted */
+                    ( ( action == TOGGLE_MUTE ) && b_var_mute ) );
+
+    const bool b_mute_condition = ( !b_var_mute
+                    && ( /* explicit */
+                        ( ( action == SET_MUTE ) && b_mute )
+                        || /* or toggle */
+                        ( action == TOGGLE_MUTE )
+                    ));
+
+    /* On UnMute */
+    if ( b_unmute_condition )
+    {
+        /* Restore saved volume */
+        var_Create( p_object->p_libvlc, "saved-volume", VLC_VAR_INTEGER );
+        i_volume = (audio_volume_t)var_GetInteger( p_object->p_libvlc,
+                                                   "saved-volume" );
+        var_SetBool( p_object->p_libvlc, "volume-muted", false );
+    }
+    else if ( b_mute_condition )
+    {
+        /* We need an initial value to backup later */
+        i_volume = config_GetInt( p_object, "volume" );
+    }
 
-    if ( p_aout->mixer.b_error )
+    if ( action == INCREMENT_VOLUME )
     {
-        /* The output module is destroyed. */
-        vlc_mutex_unlock( &p_aout->mixer_lock );
-        msg_Err( p_aout, "VolumeGet called without output module" );
-        return -1;
+        i_volume_step = config_GetInt( p_object->p_libvlc, "volume-step" );
+
+        if ( !b_unmute_condition )
+            i_volume = config_GetInt( p_object, "volume" );
+        i_volume += i_volume_step * i_nb_steps;
+
+        if ( i_volume > AOUT_VOLUME_MAX )
+            i_volume = AOUT_VOLUME_MAX;
+        else if ( i_volume < AOUT_VOLUME_MIN )
+            i_volume = AOUT_VOLUME_MIN;
+        if ( i_return_volume != NULL )
+            *i_return_volume = i_volume;
     }
 
-    i_result = p_aout->output.pf_volume_get( p_aout, pi_volume );
+    var_Create( p_object->p_libvlc, "saved-volume", VLC_VAR_INTEGER );
+    var_SetInteger( p_object->p_libvlc, "saved-volume" , i_volume );
 
-    vlc_mutex_unlock( &p_aout->mixer_lock );
-    return i_result;
-}
-
-/*****************************************************************************
- * aout_VolumeSet : set the volume of the output device
- *****************************************************************************/
-int aout_VolumeSet( aout_instance_t * p_aout, audio_volume_t i_volume )
-{
-    int i_result;
+    /* On Mute */
+    if ( b_mute_condition )
+    {
+        i_volume = AOUT_VOLUME_MIN;
+        var_SetBool( p_object->p_libvlc, "volume-muted", true );
+    }
 
-    vlc_mutex_lock( &p_aout->mixer_lock );
+    /* Commit volume changes */
+    config_PutInt( p_object, "volume", i_volume );
 
-    if ( p_aout->mixer.b_error )
+    if ( p_aout )
     {
-        /* The output module is destroyed. */
-        vlc_mutex_unlock( &p_aout->mixer_lock );
-        msg_Err( p_aout, "VolumeSet called without output module" );
-        return -1;
+        aout_lock_mixer( p_aout );
+        aout_lock_input_fifos( p_aout );
+            if ( p_aout->p_mixer )
+                i_result = p_aout->output.pf_volume_set( p_aout, i_volume );
+        aout_unlock_input_fifos( p_aout );
+        aout_unlock_mixer( p_aout );
     }
 
-    i_result = p_aout->output.pf_volume_set( p_aout, i_volume );
+    /* trigger callbacks */
+    var_TriggerCallback( p_object->p_libvlc, "volume-change");
+    if ( p_aout ) var_SetBool( p_aout, "intf-change", true );
+
+    if ( p_aout )
+    {
+        aout_unlock_volume( p_aout );
+        vlc_object_release( p_aout );
+    }
 
-    vlc_mutex_unlock( &p_aout->mixer_lock );
     return i_result;
 }
 
 /*****************************************************************************
- * aout_VolumeInfos : get the boundaries pi_low_soft and pi_high_soft
+ * aout_VolumeGet : get the volume of the output device
  *****************************************************************************/
-int aout_VolumeInfos( aout_instance_t * p_aout, audio_volume_t * pi_soft )
+int __aout_VolumeGet( vlc_object_t * p_object, audio_volume_t * pi_volume )
 {
-    int i_result;
+    int i_result = 0;
+    aout_instance_t * p_aout = vlc_object_find( p_object, VLC_OBJECT_AOUT,
+                                                FIND_ANYWHERE );
 
-    vlc_mutex_lock( &p_aout->mixer_lock );
+    if ( pi_volume == NULL ) return -1;
 
-    if ( p_aout->mixer.b_error )
+    if ( p_aout == NULL )
     {
-        /* The output module is destroyed. */
-        vlc_mutex_unlock( &p_aout->mixer_lock );
-        msg_Err( p_aout, "VolumeInfos called without output module" );
-        return -1;
+        *pi_volume = (audio_volume_t)config_GetInt( p_object, "volume" );
+        return 0;
     }
 
-    i_result = p_aout->output.pf_volume_infos( p_aout, pi_soft );
+    aout_lock_volume( p_aout );
+    aout_lock_mixer( p_aout );
+    if ( p_aout->p_mixer )
+    {
+        i_result = p_aout->output.pf_volume_get( p_aout, pi_volume );
+    }
+    else
+    {
+        *pi_volume = (audio_volume_t)config_GetInt( p_object, "volume" );
+    }
+    aout_unlock_mixer( p_aout );
+    aout_unlock_volume( p_aout );
 
-    vlc_mutex_unlock( &p_aout->mixer_lock );
+    vlc_object_release( p_aout );
     return i_result;
 }
 
+/*****************************************************************************
+ * aout_VolumeSet : set the volume of the output device
+ *****************************************************************************/
+int __aout_VolumeSet( vlc_object_t * p_object, audio_volume_t i_volume )
+{
+    return doVolumeChanges( SET_VOLUME, p_object, 1, i_volume, NULL, true );
+}
+
 /*****************************************************************************
  * aout_VolumeUp : raise the output volume
  *****************************************************************************
  * If pi_volume != NULL, *pi_volume will contain the volume at the end of the
  * function.
  *****************************************************************************/
-int aout_VolumeUp( aout_instance_t * p_aout, int i_nb_steps,
+int __aout_VolumeUp( vlc_object_t * p_object, int i_nb_steps,
                    audio_volume_t * pi_volume )
 {
-    int i_result;
-    audio_volume_t i_volume;
-
-    vlc_mutex_lock( &p_aout->mixer_lock );
-
-    if ( p_aout->mixer.b_error )
-    {
-        /* The output module is destroyed. */
-        vlc_mutex_unlock( &p_aout->mixer_lock );
-        msg_Err( p_aout, "VolumeUp called without output module" );
-        return -1;
-    }
-
-    if ( p_aout->output.pf_volume_get( p_aout, &i_volume ) )
-    {
-        vlc_mutex_unlock( &p_aout->mixer_lock );
-        return -1;
-    }
-
-    i_volume += AOUT_VOLUME_STEP * i_nb_steps;
-    if ( i_volume > 1024 ) i_volume = 1024;
-
-    i_result = p_aout->output.pf_volume_set( p_aout, i_volume );
-
-    vlc_mutex_unlock( &p_aout->mixer_lock );
-
-    if ( pi_volume != NULL ) *pi_volume = i_volume;
-    return i_result;
+    return doVolumeChanges( INCREMENT_VOLUME, p_object, i_nb_steps, 0, pi_volume, true );
 }
 
 /*****************************************************************************
@@ -170,39 +226,47 @@ int aout_VolumeUp( aout_instance_t * p_aout, int i_nb_steps,
  * If pi_volume != NULL, *pi_volume will contain the volume at the end of the
  * function.
  *****************************************************************************/
-int aout_VolumeDown( aout_instance_t * p_aout, int i_nb_steps,
+int __aout_VolumeDown( vlc_object_t * p_object, int i_nb_steps,
                      audio_volume_t * pi_volume )
 {
-    int i_result;
-    audio_volume_t i_volume;
-
-    vlc_mutex_lock( &p_aout->mixer_lock );
-
-    if ( p_aout->mixer.b_error )
-    {
-        /* The output module is destroyed. */
-        vlc_mutex_unlock( &p_aout->mixer_lock );
-        msg_Err( p_aout, "VolumeUp called without output module" );
-        return -1;
-    }
-
-    if ( p_aout->output.pf_volume_get( p_aout, &i_volume ) )
-    {
-        vlc_mutex_unlock( &p_aout->mixer_lock );
-        return -1;
-    }
-
-    if ( i_volume < AOUT_VOLUME_STEP * i_nb_steps )
-        i_volume = 0;
-    else
-        i_volume -= AOUT_VOLUME_STEP * i_nb_steps;
+    return __aout_VolumeUp( p_object, -i_nb_steps, pi_volume );
+}
 
-    i_result = p_aout->output.pf_volume_set( p_aout, i_volume );
+/*****************************************************************************
+ * aout_ToggleMute : Mute/un-mute the output volume
+ *****************************************************************************
+ * If pi_volume != NULL, *pi_volume will contain the volume at the end of the
+ * function (muted => 0).
+ *****************************************************************************/
+int __aout_ToggleMute( vlc_object_t * p_object, audio_volume_t * pi_volume )
+{
+    return doVolumeChanges( TOGGLE_MUTE, p_object, 1, 0, pi_volume, true );
+}
 
-    vlc_mutex_unlock( &p_aout->mixer_lock );
+/*****************************************************************************
+ * aout_IsMuted : Get the output volume mute status
+ *****************************************************************************/
+bool aout_IsMuted( vlc_object_t * p_object )
+{
+    bool b_return_val;
+    aout_instance_t * p_aout = vlc_object_find( p_object, VLC_OBJECT_AOUT,
+                                                FIND_ANYWHERE );
+    if ( p_aout ) aout_lock_volume( p_aout );
+    b_return_val = var_GetBool( p_object->p_libvlc, "volume-muted");
+    if ( p_aout ) aout_unlock_volume( p_aout );
+    return b_return_val;
+}
 
-    if ( pi_volume != NULL ) *pi_volume = i_volume;
-    return i_result;
+/*****************************************************************************
+ * aout_SetMute : Sets mute status
+ *****************************************************************************
+ * If pi_volume != NULL, *pi_volume will contain the volume at the end of the
+ * function (muted => 0).
+ *****************************************************************************/
+int aout_SetMute( vlc_object_t * p_object, audio_volume_t * pi_volume,
+                  bool b_mute )
+{
+    return doVolumeChanges( SET_MUTE, p_object, 1, 0, pi_volume, b_mute );
 }
 
 /*
@@ -215,24 +279,20 @@ void aout_VolumeSoftInit( aout_instance_t * p_aout )
 {
     int i_volume;
 
-    p_aout->output.pf_volume_infos = aout_VolumeSoftInfos;
     p_aout->output.pf_volume_get = aout_VolumeSoftGet;
     p_aout->output.pf_volume_set = aout_VolumeSoftSet;
 
     i_volume = config_GetInt( p_aout, "volume" );
-    if ( i_volume == -1 )
+    if ( i_volume < AOUT_VOLUME_MIN )
     {
         i_volume = AOUT_VOLUME_DEFAULT;
     }
+    else if ( i_volume > AOUT_VOLUME_MAX )
+    {
+        i_volume = AOUT_VOLUME_MAX;
+    }
 
-    aout_VolumeSoftSet( p_aout, i_volume );
-}
-
-/* Placeholder for pf_volume_infos(). */
-int aout_VolumeSoftInfos( aout_instance_t * p_aout, audio_volume_t * pi_soft )
-{
-    *pi_soft = 0;
-    return 0;
+    aout_VolumeSoftSet( p_aout, (audio_volume_t)i_volume );
 }
 
 /* Placeholder for pf_volume_get(). */
@@ -259,26 +319,21 @@ int aout_VolumeSoftSet( aout_instance_t * p_aout, audio_volume_t i_volume )
 /* Meant to be called by the output plug-in's Open(). */
 void aout_VolumeNoneInit( aout_instance_t * p_aout )
 {
-    p_aout->output.pf_volume_infos = aout_VolumeNoneInfos;
     p_aout->output.pf_volume_get = aout_VolumeNoneGet;
     p_aout->output.pf_volume_set = aout_VolumeNoneSet;
 }
 
-/* Placeholder for pf_volume_infos(). */
-int aout_VolumeNoneInfos( aout_instance_t * p_aout, audio_volume_t * pi_soft )
-{
-    return -1;
-}
-
 /* Placeholder for pf_volume_get(). */
 int aout_VolumeNoneGet( aout_instance_t * p_aout, audio_volume_t * pi_volume )
 {
+    (void)p_aout; (void)pi_volume;
     return -1;
 }
 
 /* Placeholder for pf_volume_set(). */
 int aout_VolumeNoneSet( aout_instance_t * p_aout, audio_volume_t i_volume )
 {
+    (void)p_aout; (void)i_volume;
     return -1;
 }
 
@@ -294,31 +349,35 @@ int aout_VolumeNoneSet( aout_instance_t * p_aout, audio_volume_t i_volume )
  * This function is used whenever the parameters of the output plug-in are
  * changed (eg. selecting S/PDIF or PCM).
  *****************************************************************************/
-int aout_Restart( aout_instance_t * p_aout )
+static int aout_Restart( aout_instance_t * p_aout )
 {
     int i;
-    vlc_bool_t b_error = 0;
+    bool b_error = 0;
 
-    vlc_mutex_lock( &p_aout->mixer_lock );
+    aout_lock_mixer( p_aout );
 
     if ( p_aout->i_nb_inputs == 0 )
     {
-        vlc_mutex_unlock( &p_aout->mixer_lock );
+        aout_unlock_mixer( p_aout );
         msg_Err( p_aout, "no decoder thread" );
         return -1;
     }
 
-    /* Lock all inputs. */
     for ( i = 0; i < p_aout->i_nb_inputs; i++ )
     {
-        vlc_mutex_lock( &p_aout->pp_inputs[i]->lock );
+        aout_lock_input( p_aout, p_aout->pp_inputs[i] );
+        aout_lock_input_fifos( p_aout );
         aout_InputDelete( p_aout, p_aout->pp_inputs[i] );
+        aout_unlock_input_fifos( p_aout );
     }
 
+    /* Lock all inputs. */
+    aout_lock_input_fifos( p_aout );
     aout_MixerDelete( p_aout );
 
     /* Re-open the output plug-in. */
     aout_OutputDelete( p_aout );
+
     if ( aout_OutputNew( p_aout, &p_aout->pp_inputs[0]->input ) == -1 )
     {
         /* Release all locks and report the error. */
@@ -326,7 +385,8 @@ int aout_Restart( aout_instance_t * p_aout )
         {
             vlc_mutex_unlock( &p_aout->pp_inputs[i]->lock );
         }
-        vlc_mutex_unlock( &p_aout->mixer_lock );
+        aout_unlock_input_fifos( p_aout );
+        aout_unlock_mixer( p_aout );
         return -1;
     }
 
@@ -337,7 +397,8 @@ int aout_Restart( aout_instance_t * p_aout )
         {
             vlc_mutex_unlock( &p_aout->pp_inputs[i]->lock );
         }
-        vlc_mutex_unlock( &p_aout->mixer_lock );
+        aout_unlock_input_fifos( p_aout );
+        aout_unlock_mixer( p_aout );
         return -1;
     }
 
@@ -345,13 +406,80 @@ int aout_Restart( aout_instance_t * p_aout )
     for ( i = 0; i < p_aout->i_nb_inputs; i++ )
     {
         aout_input_t * p_input = p_aout->pp_inputs[i];
-
-        b_error |= aout_InputNew( p_aout, p_input );
-        vlc_mutex_unlock( &p_input->lock );
+        b_error |= aout_InputNew( p_aout, p_input, &p_input->request_vout );
+        p_input->b_changed = 1;
+        aout_unlock_input( p_aout, p_input );
     }
 
-    vlc_mutex_unlock( &p_aout->mixer_lock );
+    aout_unlock_input_fifos( p_aout );
+    aout_unlock_mixer( p_aout );
 
     return b_error;
 }
 
+/*****************************************************************************
+ * aout_FindAndRestart : find the audio output instance and restart
+ *****************************************************************************
+ * This is used for callbacks of the configuration variables, and we believe
+ * that when those are changed, it is a significant change which implies
+ * rebuilding the audio-device and audio-channels variables.
+ *****************************************************************************/
+int aout_FindAndRestart( vlc_object_t * p_this, const char *psz_name,
+                         vlc_value_t oldval, vlc_value_t newval, void *p_data )
+{
+    aout_instance_t * p_aout = vlc_object_find( p_this, VLC_OBJECT_AOUT,
+                                                FIND_ANYWHERE );
+
+    (void)psz_name; (void)oldval; (void)newval; (void)p_data;
+    if ( p_aout == NULL ) return VLC_SUCCESS;
+
+    var_Destroy( p_aout, "audio-device" );
+    var_Destroy( p_aout, "audio-channels" );
+
+    aout_Restart( p_aout );
+    vlc_object_release( p_aout );
+
+    return VLC_SUCCESS;
+}
+
+/*****************************************************************************
+ * aout_ChannelsRestart : change the audio device or channels and restart
+ *****************************************************************************/
+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;
+    (void)oldval; (void)newval; (void)p_data;
+
+    if ( !strcmp( psz_variable, "audio-device" ) )
+    {
+        /* This is supposed to be a significant change and supposes
+         * rebuilding the channel choices. */
+        var_Destroy( p_aout, "audio-channels" );
+    }
+    aout_Restart( p_aout );
+    return 0;
+}
+
+#undef aout_EnableFilter
+/** Enable or disable an audio filter
+ * \param p_this a vlc object
+ * \param psz_name name of the filter
+ * \param b_add are we adding or removing the filter ?
+ */
+void aout_EnableFilter( vlc_object_t *p_this, const char *psz_name,
+                        bool b_add )
+{
+    aout_instance_t *p_aout = vlc_object_find( p_this, VLC_OBJECT_AOUT,
+                                               FIND_ANYWHERE );
+
+    if( AoutChangeFilterString( p_this, p_aout, "audio-filter", psz_name, b_add ) )
+    {
+        if( p_aout )
+            AoutInputsMarkToRestart( p_aout );
+    }
+
+    if( p_aout )
+        vlc_object_release( p_aout );
+}