[MPlayer-dev-eng] [PATCH] Fixing dvd:// support
Nico Sabbi
nicola.sabbi at poste.it
Tue May 11 15:38:56 CEST 2010
Il martedì 11 maggio 2010 15:28:53 Olivier Rolland ha scritto:
> On Sun, May 2, 2010 at 1:09 PM, Olivier Rolland
>
> <billl at users.sourceforge.net> wrote:
> > Hi,
> >
> > Here are 3 patches to fix some issues in stream_dvd.c :
> >
> > - The first one fixes the incorrect assumption that the number of
> > chapters of a DVD title is equal to the number of cells.
> >
> > - The second one improves seeking by chapters. The current code
> > seeks to the starting position of the chapter. From this
> > position, it then tries to figure out the starting cell. This is
> > completely suboptimal and error prone since the starting cell can
> > be directly deduced from the chapter.
> >
> > - The third one improves seeking by position. The current code
> > takes the angle into account. This is a mistake since the
> > position is independent of the angle.
> >
> > Please, apply the patches in order.
> >
> > Regards,
> >
> > Olivier
>
> ping !
I gave them a quick look few days ago and they seemed to be correct;
yet I remember that I implemented that seeking mechanism for some
reason that -guess what?- I don't remember.
I didn't test the code (sorry, but my work is taking away all my time)
but my heart says it's correct, so if it works I approve the commit.
More information about the MPlayer-dev-eng
mailing list