#endif
#include <vlc_common.h>
+#include <vlc_aout_intf.h>
#include <stdio.h>
#include <stdlib.h> /* calloc(), malloc(), free() */
#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 *);
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;
}
/** 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;
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");
#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)
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);
*/
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);
*/
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;
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;
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;
mute = !mute;
ret = commitVolume (obj, aout, volume, mute);
if (volp != NULL)
- *volp = mute ? AOUT_VOLUME_MIN : volume;
+ *volp = mute ? 0 : volume;
return ret;
}
bool aout_IsMuted (vlc_object_t *obj)
{
#if 0
- aout_instance_t *aout;
+ audio_output_t *aout;
bool mute;
prepareVolume (obj, &aout, NULL, &mute);
*/
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;
}
* 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);
}
* 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;
}
* 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;
}
/*****************************************************************************
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" ) )
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 ) )
{