<div class="gmail_quote">On Wed, Jul 29, 2009 at 6:03 PM,  <span dir="ltr"><<a href="mailto:jpd@videolan.org">jpd@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 class="im">On Wed, Jul 29, 2009 at 11:38:29AM +0200, Jakob Leben wrote:<br>
> I would like to know your opinion about making the process of moving<br>
> items atomic, meaning that PL_LOCK and PL_UNLOCK is called only once<br>
> before and after the process as a whole.<br>
<br>
</div>I'm not familiar with that part of the code, but I'd recommend checking<br>
the code inside the locks doesn't do things like calling other parts<br>
that also lock other things, or that run for a really long time. Other<br>
than that, no objection.<br></blockquote></div><br>That should not be a problem as I am not using in the new code any other functions that were not already used before inside the locks.<br><br>However, I noticed another weird thing: if the option "Display playlist tree" in "advanced" settings / Playlist is checked, then playlist_TreeMove() moves always to index 0, no matter what index you pass it.<br>
<br>Does anybody know what the above mentioned option is supposed to do?<br><br>And can somebody who knows better the core code (playlist_TreeMove() function) please check why the problem is happening? (maybe it's not a problem?)<br>
<br>Jakob<br>