You're right, that was the original problem, and this is also addressed by the patch I posted. While being at it, I thought we could also improve the meta data signalling by limiting the signal emission to cases, in which the meta data has actually been changed. But I definitely see you're point. So if we cannot lock the meta data, I guess it would be best to strip out these if checks? That way we at least have the correct meta ID for each signal emission, although the associated meta field might not necessarily been changed at the time of emission.<br>
<br><div class="gmail_quote">2011/10/13 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 16:20:21 +0200, Johannes Marbach<br>
<<a href="mailto:johannesmarbach@googlemail.com">johannesmarbach@googlemail.com</a>> wrote:<br>
> Just to make sure: In that section of my patch you posted, which of the<br>
> values do you suspect to possibly be changed after the if statement?<br>
<br>
</div>Any meta value can change while the lock is not held, though it's unlikely<br>
to happen.<br>
<br>
Also, I thought the real problem was the incorrect meta ID in the event,<br>
not the lack of events as such.<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>