[vlc-devel] [PATCH] Qt/ML: New Models for Playlist UI

Jean-Baptiste Kempf jb at videolan.org
Sat Jan 8 15:08:15 CET 2011


Hello,

On Sat, Jan 08, 2011 at 12:12:12AM +0530, Srikanth Raju wrote :
> > I believe this should be merged with other functions that do mostly the
> > same, in the future.
> Can't really do anything except macro it.
I disagree, but well, this'll do for now.

> It's there. I could also virtualize popupDel() popupAdd() etc, but
> those are specific to the models and get called by popup(). It's
> better to just have only the necessary interfaces in VLCModel instead
> of pulling everything out that can be pulled out.
I don't see why. Popup menus should be consistent.

Best Regards,

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



More information about the vlc-devel mailing list