<div class="gmail_quote">On Thu, Aug 13, 2009 at 2:48 PM, git version control <span dir="ltr"><<a href="mailto:git@videolan.org">git@videolan.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div id=":wz" class="ii gt">-void PLItem::init( int _i_id, int _i_input_id, bool _is_node, PLItem *parent, PLModel *m, QSettings *settings )<br>
+void PLItem::init( playlist_item_t *_playlist_item, PLItem *parent, PLModel *m, QSettings *settings )</div></blockquote></div><br>Also, now you are not only counting on input item pointer to be valid, but also on playlist item pointer. Are you sure that PLItem functions like this are always called within PL_LOCK? At least it should be documented that PL_LOCK must be held upon entering the function!<br>