[MPlayer-dev-eng] [PATCH] demux_lavf: add an option to use the new seek API.

[Reimar Döffinger]

On Mon, Apr 29, 2013 at 05:21:17PM +0200, Nicolas George wrote:
> Le decadi 20 germinal, an CCXXI, Reimar Döffinger a écrit :
> > What is the point of that?
> 
> A few formats implement read_seek2 but not read_seek: seeking works very
> badly for these formats: only forward, slowly and with glitches.
> 
> > With those settings I believe seeking will work very badly (in particular,
> > in files with few keyframes seeking forward will actually end up seeking
> > backwards and then get stuck).
> 
> > Or in other words, at the very least for relative seeks you should set
> > either min or max to the reference time stamp (possibly with a small
> > offset to make sure we never get stuck).
> 
> > Then, to keep old behaviour you should try a seek with neither min or max
> > set if that fails.
> 
> I am probably missing something, but I believe that is exactly what the API
> is supposed to do internally. Otherwise, I do not see the point of having
> all four parameters: target ts, min and max ts, and flag indicating
> preferred direction.

Of which you set neither min or max, that was my point...