[MPlayer-dev-eng] Re: [PATCH] Make all subtitles availiable

[Guillaume POIRIER]

On 12/1/06, Sturle Sunde <sturles at ifi.uio.no> wrote:
> Lehel Bernadt <lehel at pmc-services.hu> writes:
> > I have a more simple (and correct) patch for this, which i posted to
> > the list twice already, but it seems like the dvd stuff is
> > maintained by nobody, so no one was willing to apply it until now.
> > The patch to today's svn is attached, please see if it works for
> > you.
>
> I have reverted my own patch and tested yours after a svn update.  It
> works fine.

Did you test with the change suggested by Nico?

Which is:

 +  return d->subtitles[d->nr_of_subtitles-1].id;

instead of

+  return d->subtitles[d->nr_of_subtitles].id;

in Lehel Bernadt's patch

Guillaume
-- 
An association of men who will not quarrel with one another is a thing
which has never yet existed, from the greatest confederacy of nations
down to a town meeting or a vestry.
 -- Thomas Jefferson
(when interviewed about MPlayer ML flamewars)
http://www.brainyquote.com/quotes/quotes/t/thomasjeff157207.html