Hello.<br><br><div class="gmail_quote">On 20 September 2012 19:06, Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net" target="_blank">remi@remlab.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


Le mardi 18 septembre 2012 20:11:34, Mark Lee a écrit :<br>
<div><div>> Rather than using callbacks in the core, the audio equalizer is simply<br>
> directly applied (when enabled) to the aout:<br>
><br>
>  1. each time an aout is created;<br>
>  2. each time the equalizer is set on the media player.<br>
><br>
> This is similar, in part, to how libvlc_set_fullscreen is implemented.<br>
> ---<br>
>  lib/media_player.c        |   31 +++++++++++++++++++++++++++----<br>
>  src/audio_output/common.c |   15 ---------------<br>
>  2 files changed, 27 insertions(+), 19 deletions(-)<br>
><br>
> diff --git a/lib/media_player.c b/lib/media_player.c<br>
> index b9b2a11..b468f96 100644<br>
> --- a/lib/media_player.c<br>
> +++ b/lib/media_player.c<br>
> @@ -32,6 +32,7 @@<br>
><br>
>  #include <vlc_demux.h><br>
>  #include <vlc_input.h><br>
> +#include <vlc_aout.h><br>
>  #include <vlc_vout.h><br>
>  #include <vlc_keys.h><br>
>  #include <vlc_aout_intf.h><br>
> @@ -191,6 +192,21 @@ static void set_state( libvlc_media_player_t *p_mi,<br>
> libvlc_state_t state, }<br>
>  }<br>
><br>
> +static void apply_equalizer( input_thread_t *p_input,<br>
> libvlc_media_player_t *p_mi ) +{<br>
> +    audio_output_t *p_aout;<br>
> +    if( input_Control( p_input, INPUT_GET_AOUT, &p_aout ) == VLC_SUCCESS )<br>
> +    {<br>
> +        var_SetString( p_aout, "equalizer-bands", var_GetString( p_mi,<br>
> "equalizer-bands" ) ); +        var_SetFloat( p_aout, "equalizer-preamp",<br>
> var_GetFloat( p_mi, "equalizer-preamp" ) ); +<br>
> +        if( strlen( var_GetString( p_mi, "equalizer-bands" ) ) > 0 )<br>
> +            aout_EnableFilter( p_mi, "equalizer", true );<br>
<br>
</div></div>Memory leak and potential NULL dereference. </blockquote><div><br>OK.<br>
 </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Also in the old code that should
not have been merged.<br></blockquote><div><br>That ship has sailed. I'm doing what I can to resolve it.<br>
</div><br>Thanks.<br></div>