<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">2014-07-07 3:13 GMT+02:00 Jean-Baptiste Kempf <span dir="ltr"><<a href="mailto:jb@videolan.org" target="_blank">jb@videolan.org</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">On 07 Jul, Felix Abecassis wrote :<br>
<div><div class="h5">> Le 7 juil. 2014 00:04, "Jean-Baptiste Kempf" <<a href="mailto:jb@videolan.org">jb@videolan.org</a>> a écrit :<br>
> ><br>
> > On 04 Jul, Felix Abecassis wrote :<br>
> > > vlc-ports/android | branch: master | Felix Abecassis <<br>
> <a href="mailto:felix.abecassis@gmail.com">felix.abecassis@gmail.com</a>> | Fri Jul  4 15:48:31 2014 +0200|<br>
> [c5190950a94cad33cb155de58d9c15ecf82245e3] | committer: Felix Abecassis<br>
> > ><br>
> > > VideoPlayerActivity: handle navigable media (DVD, Blu-Ray)<br>
> > ><br>
> > > There are three different cases:<br>
> > > - Item is not navigable: same as before.<br>
> > > - Item is navigable and we are on a menu: keep overlay hidden, touch<br>
> > >   events will be immediately forwarded to the navigation module<br>
> > >   instead of showing the overlay.<br>
> > > - Item is navigable but we are not on a menu: add a "return to menu"<br>
> > >   button to the overlay<br>
> ><br>
> > We have an important issue with the 0.9.7 release, maybe related to this<br>
> > one commit:<br>
> >  - "This latest update has lost the interface controls, now not able to<br>
> >  pause, or access the screen size options."<br>
> >  - "Yup. Also having 'no control interface'-issue when playing videos;<br>
> >  can't even pause it! "<br>
> >  - "Videos have no menu. Thus not pausierbar, nothing adjustable"<br>
> >  - "Nexus7 I can't seek and see video menu options... Please fix"<br>
> >  - "After the upgrade panel lost rewind, lock button and a status bar."<br>
> ><br>
> > If someone has an urgent idea to fix, please share :)<br>
><br>
> Yes it's probably caused by this commit. You can temporarily set mIsNavMenu<br>
> to false and it should avoid hiding the overlay while I investigate the<br>
> issue (tomorrow morning).<br>
<br>
</div></div>I think the DEFAULT should be displayed ON, and disabled ONLY when DVD<br>
are played back.<br></blockquote><div><br></div><div>That was my goal. However I don't think there is a nice way to detect DVD playback currently.<br><br></div><div>Videos can have chapters indeed, but the store version is also broken for videos without chapters. This is apparently caused by a bug in the core, libvlc_get_chapter_count_for_title() returns random garbage for some samples (e.g. 1976853078 chapters for one sample). This is caused by a lack of error checking in libvlc_media_player_get_chapter_count_for_title(), the return value of var_Change is not tested and thus we return an uninitialized value if var_Change failed (for instance in case of VLC_ENOVAR). <br>

</div><div><br></div></div></div><div class="gmail_extra"><br>-- <br>Félix Abecassis<div><a href="http://felix.abecassis.me" target="_blank">http://felix.abecassis.me</a></div>
</div></div>