<div dir="ltr"><div><div>About the different patch, can it be applied after the multiple video instance patch or before ? If before, I will take a little longer to get that done.<br><br></div>Inside VideoPlayerActivity::onPause method the libvlc stop function is called and the video's last position is saved. After the patch you will be able to use the libvlc pause function. <br><br></div>Thanks,<br>Paulo<br><div><br><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2014-12-02 11:45 GMT-02:00 Thomas Guillem <span dir="ltr"><<a href="mailto:thomas@gllm.fr" target="_blank">thomas@gllm.fr</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><u></u>




<div><span class=""><div> </div>
<div>On Tue, Dec 2, 2014, at 14:04, Paulo Vitor Magacho da Silva wrote:<br></div>
<blockquote type="cite"><div dir="ltr"><div><div><div><div><div><div><div>Hi,<br></div>
<div> </div>
<div>Ok. Will provide both commits.<br></div>
</div>
<div> </div>
<div>Will update the name as well. Can it be "Add support for multiple video instances for Android." ?<br></div>
</div>
<div> </div>
</div>
<div>I will check the discrepancy between drawable-vlcobject and drawable-androidobject.<br></div>
<div> </div>
</div>
<div>I am using the android_surf_value_t members to check if the surface was destroyed so I it can be re-attached.<br></div>
</div>
<div>To fix this, I will create a method in the android libvlc JNI code to check that. This way I don't have to use the struct member. Is that ok ? If yes, I will make a new commit for Android code and VLC. Should do it later today.<br></div>
<div>This functionality is interesting, because if you hide the SurfaceView and show it again, the video can continue to be displayed in the new Surface object. While the SurfaceView is hidden, no image is sent to the display, since the Surface object was destroyed.<br></div>
</div>
</div>
</blockquote><div> </div>
</span><div>Ok, It might be interesting, but you have to do that in a separate commit. I missed that code when I reviewed your patch.<br></div><span class="">
<div> </div>
<blockquote type="cite"><div dir="ltr"><div>Thank you for all the help.<br></div>
<div> </div>
<div>After all this is ok, I will work on changing the onPause/onResume methods to use the pause function, instead of the stop, if that is ok.<br></div>
</div>
</blockquote></span><div>I'm curious, In which case you don't want to stop the video when you hit onPause ?<br></div><div><div class="h5">
<div> </div>
<blockquote type="cite"><div dir="ltr"><div> </div>
<div>Cheers,<br></div>
<div>Paulo<br></div>
<div><div><div><div><div><div><div><div><div><div><div><div> </div>
<div><div>2014-12-02 10:50 GMT-02:00 Thomas Guillem <span dir="ltr"><<a href="mailto:thomas@gllm.fr" target="_blank">thomas@gllm.fr</a>></span>:<br></div>
<blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>You should do 2 commits, one that touch<br></div>
<div>include/vlc/libvlc_media_player.h lib/media_player.c and src/libvlc.c.<br></div>
<div>This commit should be about adding libvlc_media_player_set_androidobject<br></div>
<div>into libvlc. The others commits should touch modules.<br></div>
<div> </div>
<div>The name "Add support for multiple video instances" is not good, vlc is<br></div>
<div>not only about android, and support multiple video instance for other<br></div>
<div>platforms since a long time.<br></div>
<div> </div>
<div>At the end, in src/libvlc.c<br></div>
<div>why drawable-vlcobject and not drawable-androidobject ?<br></div>
<div> </div>
<div>You don't have to specify again  struct android_surf_value_t in vlc.<br></div>
<div>You can do: typedef struct android_surf_value_t android_surf_value_t;<br></div>
<div>vlc shouldn't have access to android_surf_value_t members.<br></div>
<div> </div>
<div>Some comments don't match the function name (surfacevalue vs<br></div>
<div>androidobject)<br></div>
<div><div>_______________________________________________<br></div>
<div>Android mailing list<br></div>
<div><a href="mailto:Android@videolan.org" target="_blank">Android@videolan.org</a><br></div>
<div><a href="https://mailman.videolan.org/listinfo/android" target="_blank">https://mailman.videolan.org/listinfo/android</a><br></div>
</div>
</blockquote></div>
<div> </div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
<div><u>_______________________________________________</u><br></div>
<div>Android mailing list<br></div>
<div><a href="mailto:Android@videolan.org" target="_blank">Android@videolan.org</a><br></div>
<div><a href="https://mailman.videolan.org/listinfo/android" target="_blank">https://mailman.videolan.org/listinfo/android</a><br></div>
</blockquote><div> </div>
</div></div></div>

<br>_______________________________________________<br>
Android mailing list<br>
<a href="mailto:Android@videolan.org">Android@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/android" target="_blank">https://mailman.videolan.org/listinfo/android</a><br>
<br></blockquote></div><br></div>