[MPlayer-dev-eng] [PATCH] fix seeking in demux_lavf

Aurelien Jacobs aurel at gnuage.org
Wed Aug 13 01:53:35 CEST 2008


Michael Niedermayer wrote:

> On Mon, Aug 11, 2008 at 01:48:24AM +0200, Aurelien Jacobs wrote:
> > Reimar Doeffinger wrote:
> > 
> > > On Wed, Aug 06, 2008 at 11:44:28PM +0200, Aurelien Jacobs wrote:
> > > > So here is a better patch that just reset stream cache to it's
> > > > previous state after a stream_seek() failure, thus making
> > > > mp_seek meet url_fseek() expectation.
> > > 
> > > It might be even better to even improve stream_seek to match the expectations
> > > instead,
> > 
> > I've thought about this too. But this would be quite hard to do such
> > a major change to stream_seek, testing all the possible corner case,
> > without breaking anything.
> > So I think it's not worth.
> > 
> > > but this patch is good enough for me (though I do not maintain
> > > demux_lavf).
> > 
> > Michael ? I will commit this in a few days if I see no complaint.
> 
> you can commit it as soon as you cast your vote to anyone you like to
> become mplayer leader.

Well. I think you're not interested in the task so I won't vote for you.
Anyway, I prefer seeing you concentrating on FFmpeg.
So the only person left who I think could handle the task decently is
Reimar. So if he volunteers for this position, he gets my vote.

And now, I will just commit the patch.

Aurel



More information about the MPlayer-dev-eng mailing list