<br><br><div class="gmail_quote">2010/3/8 Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net">remi@remlab.net</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im"><br>
On Mon,  8 Mar 2010 12:02:53 +0100 (CET), <a href="mailto:git@videolan.org">git@videolan.org</a> wrote:<br>
> vlc | branch: master | Jakob Leben <<a href="mailto:jleben@videolan.org">jleben@videolan.org</a>> | Mon Mar  8<br>
> 11:56:45 2010 +0100| [4bd42ef61b011eeb2f016354df9767bd75a5e1df] |<br>
> committer: Jakob Leben<br>
><br>
> Qt: Playlist drag and drop corrections<br>
><br>
> - Use item ids instead of pointers for drag data to prevent usage after<br>
> freeing.<br>
<br>
</div>No way. ID are deprecated and broken by design. Pierre and I have spent<br>
enough hours getting rid of them. Please revert this.<br></blockquote><div><br>Previously, the drag data consisted of PLItem *, which is the worst option, since they might be freed while dragging.<br>Do you agree then on using playlist_item_t *? Those are never freed until the playlist is, but we can check if they are still in the playlist_t->all_items array.<br>
Unfortunately, we cannot use input_item_t*, because if we incref them, I don't see any way in Qt to make sure they are decrefed after drag operation.<br></div></div>