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

Mark Lee mark.lee at capricasoftware.co.uk
Sun Sep 16 11:03:13 CEST 2012


On 16 September 2012 08:42, Rémi Denis-Courmont <remi at remlab.net> wrote:

> Le samedi 15 septembre 2012 20:50:14, Rémi Denis-Courmont a écrit :
> > Again: No. Do NOT add ugly callbacks in the core code like this.
> >
> > (Volume is _not_ a good example. The UI never sets the volume variable
> > directly.)
> Fullscreen is a good example that works.

Thanks for the tip. After reviewing the fullscreen implementation it gave
me an idea for a way forward.

I have a new patch. It is not exactly the same as fullscreen, but it uses
one key aspect of it.

Do you want the previously committed patch to be reverted first, or should
I send a new patch based on what has already been committed?

> --
> 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/20120916/de02aa5f/attachment.html>

More information about the vlc-devel mailing list