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

[Nicolas George]

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.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20130429/03eba7cf/attachment.asc>