[vlc-devel] [PATCH 2/2] DnD from Internet Explorer to VLC and support of links (*.lnk)

Jean-Baptiste Kempf jb at videolan.org
Fri Feb 8 12:02:23 CET 2013


On 08 Feb, Mario Speiß wrote :
> On 07.02.2013 17:41, Jean-Baptiste Kempf wrote:
> >On 06 Jan, Mario Speiß wrote :
> >>  modules/gui/qt4/dialogs/playlist.cpp |   17 +++++++++++++----
> >>  modules/gui/qt4/main_interface.cpp   |   21 +++++++++++++++++----
> >>  2 files changed, 30 insertions(+), 8 deletions(-)
> >
> >Too much duplication for same feature. Please factorize.
> What do you mean? The DnD handling is in the playlist widget _AND_
> in the main interface also. One DnD-handling method for both?

Or at least a common function instead of copy/paste.

Best regards,

-- 
Jean-Baptiste Kempf
http://www.jbkempf.com/ - +33 672 704 734
Sent from my Electronic Device



More information about the vlc-devel mailing list