<div>Just to make sure: In that section of my patch you posted, which of the values do you suspect to possibly be changed after the if statement? The old meta, that is <span class="Apple-style-span" style="font-family: arial, sans-serif; font-size: 14px; background-color: rgb(255, 255, 255); ">input_item_GetNowPlaying( p_input->p->p_item ),  </span>or the new meta, that is <span class="Apple-style-span" style="font-family: arial, sans-serif; font-size: 14px; background-color: rgb(255, 255, 255); ">p_pgrm->psz_now_playing</span>?</div>
<div><br></div><div>The problem with <span class="Apple-style-span" style="font-family: arial, sans-serif; font-size: 14px; background-color: rgb(255, 255, 255); ">input_item_SetNowPlaying is, that it fires the meta changed signal, no matter whether the meta is actually changed or not. By the way, can you tell me where that function is defined? As I don't seem to find it anywhere in my git checkout.</span></div>
<br><div class="gmail_quote">2011/10/12 Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net">remi@remlab.net</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On Wed, 12 Oct 2011 14:11:14 +0200, Johannes Marbach<br>
<<a href="mailto:johannesmarbach@googlemail.com">johannesmarbach@googlemail.com</a>> wrote:<br>
> You mean the meta data could possibly be changed between the if<br>
statement<br>
> and the call to input_item_SetNowPlaying / input_item_SetPublisher? That<br>
> would imply that we'd need to lock the input item before the if check,<br>
> right?<br>
<br>
</div>It's not that easy. The lock is not recursive.<br>
<div><div></div><div class="h5"><br>
--<br>
Rémi Denis-Courmont<br>
<a href="http://www.remlab.net/" target="_blank">http://www.remlab.net/</a><br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
</div></div></blockquote></div><br>