Hello,<br><br><div class="gmail_quote">On 6 September 2012 17:49, 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">
<div class="im">
> The latest version of the patch does set values on the aout, but it uses an<br>
> additional variable on the media player to track whether or not the<br>
> equalizer should automatically be enabled when playing new media.<br>

</div>><br>Yes but the way to do that is to set both, not add ugly callbacks in the core<br>
code.<br></blockquote><div><br>Is it OK to do something like this (in aout_New() in src/audio_output/common.c):<br><br>if (var_Type (p_parent, "equalizer-bands"))<br>{<br>     var_Create (aout, "equalizer-preamp", VLC_VAR_FLOAT | VLC_VAR_DOINHERIT);<br>
     var_Create (aout, "equalizer-bands", VLC_VAR_STRING | VLC_VAR_DOINHERIT);<br>     var_AddCallback (p_parent, "equalizer-preamp", var_Copy, aout);<br>     var_AddCallback (p_parent, "equalizer-bands", var_Copy, aout);<br>
}<br><br>Or am I not supposed to change anything in the core, and I must instead add the callbacks from the libvlc side?<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im HOEnZb">
--<br>
Rémi Denis-Courmont<br>
<a href="http://www.remlab.net/" target="_blank">http://www.remlab.net/</a><br>
</div><div class="HOEnZb"><div class="h5">_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
</div></div></blockquote></div><br>