<div class="gmail_quote">On Thu, Aug 13, 2009 at 4:21 PM, Ilkka Ollakka <span dir="ltr"><<a href="mailto:ilkka.ollakka%2Bvlc@oamk.fi">ilkka.ollakka+vlc@oamk.fi</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=":14i" class="ii gt">Actually it counted that playlist_item_t pointer was valid in earlier<br>
constructor too,</div></blockquote></div><br>Oh, that's right.<br><br>One other thing:<br>I suppose the PLItem( int, int, bool, PLItem *parent , PLModel * ) constructor should be removed from playlist_item.hpp as it does not have it's definition now.<br>
<br>Nice work. By the way I'm working currently also on trying to remove PLItem completely. However, I found out there is two prerequisites from the core to do before:<br>- adding lock and refcounting to playlist items as well (playlist lock is not good enough for that, because we will need to store playlist item * right inside the QModelIndex which is handed out to views, where we have no control on it's usage)<br>
- for the sake of Qt4 model/view architecture we should have a callback like "playlist-item-about-to-be-deleted", because we have to notify views in advance.<br><br>Just so to mention the tasks I see coming, if somebody wants to take on it...<br>
<br>Cheers,<br>Jakob<br>