Hello,<br><br><div class="gmail_quote">On 16 September 2012 08:42, Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net" target="_blank">remi@remlab.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Le samedi 15 septembre 2012 20:50:14, Rémi Denis-Courmont a écrit :<br>
<div class="im">> Again: No. Do NOT add ugly callbacks in the core code like this.<br>
><br>
> (Volume is _not_ a good example. The UI never sets the volume variable<br>
> directly.)<br>
<br>
</div>Fullscreen is a good example that works.<br></blockquote><div><br>Thanks for the tip. After reviewing the fullscreen implementation it gave me an idea for a way forward. <br><br>I have a new patch. It is not exactly the same as fullscreen, but it uses one key aspect of it.<br>
<br>Do you want the previously committed patch to be reverted first, or should I send a new patch based on what has already been committed?<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="HOEnZb"><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><br>