[Ffmpeg-cvslog] Re: r5468 - trunk/libavformat/mov.c

[Luca Barbato]

Baptiste Coudurier wrote:
> Hi
> 
> Michael Niedermayer wrote:
>> [...]
>>
>> well, for the AVSEEK_FLAG_BACKWARD case id say that if the user wants
>> timestamp t then we at least should seek so that the user will receive
>> a keyframe in every stream before or at t ...
>>
>> for a forward seek on stream s id say that we should at least ensure that
>> the user gets a keyframe on every stream prior to the keyframe in s we
>> select
>>
>> these are just the rules which seems to make most sense when thinking
>> about forward and backward seeking in a normal player, i doubt many demuxers
>> in ffmpeg actually conform to them ... and maybe iam missing something
>> and the rules actually arent ideal ...
>>
>> also note that when we designed nut, rich and others suggested that seeking
>> should actually be based on a user specified set of streams instead of
>> streamless (stream_id == -1) or based on a single stream so that in case of
>> very few keyframes on one stream we wouldnt have "problems" with the
>> remaining streams
>>
> 
> I understand, and I experienced that. That version of mov_read_seek,
> seeks in other streams according to sample found timestamp. Is that
> version is preferred ?
> 

Works quite better than the current code, please commit.

lu

PS: we should either fix mplayer mov demuxer or start using ffmpeg one...

-- 

Luca Barbato

Gentoo/linux Gentoo/PPC
http://dev.gentoo.org/~lu_zero