[vlc-devel] [PATCH] Add new API to libvlc for persistent equalizer settings.

Mark Lee mark.lee at capricasoftware.co.uk
Thu Sep 6 21:06:53 CEST 2012


On 6 September 2012 17:49, Rémi Denis-Courmont <remi at remlab.net> wrote:

> > The latest version of the patch does set values on the aout, but it uses
> an
> > additional variable on the media player to track whether or not the
> > equalizer should automatically be enabled when playing new media.
> >
> Yes but the way to do that is to set both, not add ugly callbacks in the
> core
> code.

Is it OK to do something like this (in aout_New() in

if (var_Type (p_parent, "equalizer-bands"))
     var_Create (aout, "equalizer-preamp", VLC_VAR_FLOAT |
     var_Create (aout, "equalizer-bands", VLC_VAR_STRING |
     var_AddCallback (p_parent, "equalizer-preamp", var_Copy, aout);
     var_AddCallback (p_parent, "equalizer-bands", var_Copy, aout);

Or am I not supposed to change anything in the core, and I must instead add
the callbacks from the libvlc side?

>  --
> Rémi Denis-Courmont
> http://www.remlab.net/
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> http://mailman.videolan.org/listinfo/vlc-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20120906/430aab04/attachment.html>

More information about the vlc-devel mailing list