[MPlayer-dev-eng] [PATCH] dvdnav

[Ötvös Attila]

2006. január 2. 16.33 dátummal Diego Biurrun ezt írta:
> On Fri, Dec 23, 2005 at 03:10:55AM +0100, Diego Biurrun wrote:
> > On Fri, Dec 23, 2005 at 03:07:04AM +0100, Diego Biurrun wrote:
> > > On Mon, Dec 19, 2005 at 02:46:50AM +0100, ?tv?s Attila wrote:
> > > > I am created a patch mplayer and gmplayer to using dvd menu.
> > > >
> > > > Implemented:
> > > > - still frame menu (partially)
> > > > - spu menu buttons (partially)
> > > > - can be used mouse with menu selection (partially)
> > > > - simple light box menu button (new OSDTYPE)
> > > > - time length of vts domain stream
> > > > - can be selection the subtitle language with "vobsub_lang" command
> > > > ('j') - can be dvd menu trought set audio and subtitle languages
> > > > - new menu item in gmplayer (in gtk menu)
> > > > - can be set dvd menu languages
> > >
> > > Wow, this looks like quite a feat!
> > >
> > > I've patched it into my test tree to try it out (I had to
> > > --disable-live at configure time, but that's minor for the moment) and
> > > indeed a DVD menu comes up..  But how do I interact with it?
> >
> > OK, works with the mouse when using GMPlayer.  Still frames still seem
> > to be problematic, though.
>
> Do you plan on fixing these issues?
>
> Diego
>
> _______________________________________________
> MPlayer-dev-eng mailing list
> MPlayer-dev-eng at mplayerhq.hu
> http://mplayerhq.hu/mailman/listinfo/mplayer-dev-eng

The dvdnav handler is bad in mplayer.c and in several case it passes in the 
dvdnav event in the course of processing of the demuxer_seek(). 

I found out DVDNAV_STOP event error, that it happenes if at the given program 
entry hasn't command table present.

The new dvdnav patch:

http://members.chello.hu/dc/mplayer/

Main modifications (compared with old patch):

- libdvdnav & dvdnav_stream were packed in the stream_dvdnav (so that cab be 
used: --disable-mpdvdkit2 options)
- dvdnav handle is an error and I corrected it in mplayer.c (demuxer_seek())
- dvdnav STOP_EVENT is an error and I corrected it
- remove demuxer->event structure

(I am sorry to say that my English knowledge isn't good.)

Best regards!

Attila