<div dir="ltr"><div><div><div><div><div><div><div>Hi,<br></div><br>Ok. Will provide both commits.<br></div><br>Will update the name as well. Can it be "Add support for multiple video instances for Android." ?<br></div><br></div>I will check the discrepancy between drawable-vlcobject and drawable-androidobject.<br><br></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>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><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><br>Thank you for all the help.<br><br></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><br></div>Cheers,<br>Paulo<br><div><div><div><div><div><div><div><div><div><div><div class="gmail_extra"><br><div class="gmail_quote">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><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">You should do 2 commits, one that touch<br>
include/vlc/libvlc_media_player.h lib/media_player.c and src/libvlc.c.<br>
This commit should be about adding libvlc_media_player_set_androidobject<br>
into libvlc. The others commits should touch modules.<br>
<br>
The name "Add support for multiple video instances" is not good, vlc is<br>
not only about android, and support multiple video instance for other<br>
platforms since a long time.<br>
<br>
At the end, in src/libvlc.c<br>
why drawable-vlcobject and not drawable-androidobject ?<br>
<br>
You don't have to specify again  struct android_surf_value_t in vlc.<br>
You can do: typedef struct android_surf_value_t android_surf_value_t;<br>
vlc shouldn't have access to android_surf_value_t members.<br>
<br>
Some comments don't match the function name (surfacevalue vs<br>
androidobject)<br>
<div class=""><div class="h5">_______________________________________________<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>
</div></div></blockquote></div><br></div></div></div></div></div></div></div></div></div></div></div></div>