2009/7/30 Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net" target="_blank">remi@remlab.net</a>></span>><br><blockquote style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;" class="gmail_quote">
+                int i;<br><div>+                for( i = 0 ; i< p_parent->i_children ; i++ )<br>
<br>
</div>You could move i into the loop, I guess.<br></blockquote>
<br>Nope, because:<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>playlist_TreeMove( p_playlist, p_src, p_parent, <b>i </b>+ 1 ); // !!!<br>
<br><br>However, there was a ugly bug in the patch that I sent, sorry. Forgot to include the change of name of an argument to dropMimeData().<br><br>Attached is the corrected patch.<br>