<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body><div><div style="font-family: Calibri,sans-serif; font-size: 11pt;">How do I use use the android window vout ?<br>Thanks</div></div><div dir="ltr"><hr><span style="font-family: Calibri,sans-serif; font-size: 11pt; font-weight: bold;">De: </span><span style="font-family: Calibri,sans-serif; font-size: 11pt;"><a href="mailto:thomas@gllm.fr">Thomas Guillem</a></span><br><span style="font-family: Calibri,sans-serif; font-size: 11pt; font-weight: bold;">Enviada em: </span><span style="font-family: Calibri,sans-serif; font-size: 11pt;">26/11/2014 18:29</span><br><span style="font-family: Calibri,sans-serif; font-size: 11pt; font-weight: bold;">Para: </span><span style="font-family: Calibri,sans-serif; font-size: 11pt;"><a href="mailto:android@videolan.org">android@videolan.org</a></span><br><span style="font-family: Calibri,sans-serif; font-size: 11pt; font-weight: bold;">Assunto: </span><span style="font-family: Calibri,sans-serif; font-size: 11pt;">Re: [Android] VideoPlayerActivity onPause changes</span><br><br></div><br>On Wed, Nov 26, 2014, at 21:21, Edward Wang wrote:<br>> On Wed, Nov 26, 2014 at 11:02 AM, Paulo Vitor Magacho da Silva<br>> <pvmagacho@gmail.com> wrote:<br>> > Hi,<br>> ><br>> > In this class there is the following comment in onPause method.<br>> ><br>> > /*<br>> > * Pausing here generates errors because the vout is constantly<br>> > * trying to refresh itself every 80ms while the surface is not<br>> > * accessible anymore.<br>> > * To workaround that, we keep the last known position in the playlist<br>> > * in savedIndexPosition to be able to restore it during onResume().<br>> > */<br><br>Hum, I don't think this hack is needed anyore after android 2.3 when<br>using the new android_window vout.<br>I'll chech tomorrow.<br><br>> ><br>> > Instead of doing this, I propose another solution, where the surface is<br>> > detached and re-attached on resume (using surfaceCreate and surfaceDestroy<br>> > methods) and the pause method of LibVLC is used (instead of stop). It<br>> > wouldn't be necessary to save the current position and time anymore.<br>> ><br>> > There are also some minor changes needed in the VLC library to support this<br>> > - need to stop refreshing the surface and wait for a new one to be created<br>> > again.<br>> ><br>> > However, I would like to introduce it along with the multiple video<br>> > instances patch. At this point, it would take me a very long time to apply<br>> > this change to the current code.<br>> ><br>> > Can I submit both changes together ?<br>> <br>> <br>> I think that it would be cleaner to do them in two separate patches,<br>> since these are two separate functional changes.<br>> <br>> Regards,<br>> Edward<br>> _______________________________________________<br>> Android mailing list<br>> Android@videolan.org<br>> https://mailman.videolan.org/listinfo/android<br>_______________________________________________<br>Android mailing list<br>Android@videolan.org<br>https://mailman.videolan.org/listinfo/android<br></body></html>