Has anyone considered if it's cool to have this copy action?<br><br>If not, here is a small patch that at least fixes the segmentation fault. Pick your choose of the three.<br><br><br>From f7e4f77d834ae77476f644bc29febbe8f46e59d4 Mon Sep 17 00:00:00 2001<br>
From: Jakob Leben <<a href="mailto:jakob.leben@gmail.com">jakob.leben@gmail.com</a>><br>Date: Thu, 13 Aug 2009 16:06:51 +0200<br>Subject: [PATCH] Qt4: fix segfault in playlist<br><br>Fix segfault when tree view is disabled and more then one items are moved<br>
from one into another top node (Playlist, Media Library...) and the receiver<br>node is empty<br>---<br> .../gui/qt4/components/playlist/playlist_model.cpp |    4 ++--<br> 1 files changed, 2 insertions(+), 2 deletions(-)<br>
<br>diff --git a/modules/gui/qt4/components/playlist/playlist_model.cpp b/modules/gui/qt4/components/playlist/playlist_model.cpp<br>index 207b0f5..c49b24c 100644<br>--- a/modules/gui/qt4/components/playlist/playlist_model.cpp<br>
+++ b/modules/gui/qt4/components/playlist/playlist_model.cpp<br>@@ -241,8 +241,8 @@ bool PLModel::dropMimeData( const QMimeData *data, Qt::DropAction action,<br>             else<br>             {<br>                 int i;<br>
-                for( i = 0 ; i< p_parent->i_children ; i++ )<br>-                    if( p_parent->pp_children[i] == p_target ) break;<br>+                for( i = 0 ; i< p_target->p_parent->i_children ; i++ )<br>
+                    if( p_target->p_parent->pp_children[i] == p_target ) break;<br>                 playlist_TreeMove( p_playlist, p_src, p_parent, i + 1 );<br>             }<br>             p_target = p_src;<br>--<br>
1.6.0.4<br>