[vlc-devel] [PATCH] Qt/ML: Enable Panels to display SQL ML

Jean-Baptiste Kempf jb at videolan.org
Mon Jan 24 18:43:31 CET 2011


Hello,

On Mon, Jan 24, 2011 at 10:18:19PM +0530, Srikanth Raju wrote :
> This patch will bring in a new view for the ML.
> It's not complete, and I'm yet to iron a few things out. But it
> works-ish. So, here's a quick patch to check it out.
Ok. Most of the code seems quite ok, but:

> +    rightPLPanel = new StandardPLPanel( this, p_intf, p_root, selector, plmodel );
> +    rightMLPanel = new StandardPLPanel( this, p_intf, NULL, selector, mlmodel );
Why do we need 2 different PLPanel instance? This would take quite some
RAM.

Best Regards,

-- 
Jean-Baptiste Kempf
http://www.jbkempf.com/
+33 672 704 734



More information about the vlc-devel mailing list