[FFmpeg-devel] [PATCH] RTMP seek support
Wed Apr 7 13:33:03 CEST 2010
On Tue, Apr 06, 2010 at 10:50:21PM -0700, Howard Chu wrote:
> Michael Niedermayer wrote:
>> On Thu, Apr 01, 2010 at 01:11:24PM -0700, Howard Chu wrote:
>>> Michael Niedermayer wrote:
>>>> On Wed, Mar 31, 2010 at 05:24:49PM -0700, Howard Chu wrote:
>>>>> Howard Chu wrote:
>>>>>> Michael Niedermayer wrote:
>>>>>>> removial is planed but the new API might be changed if we find the
>>>>>>> to change it still ...
>>>>> It seems to me that one thing that ought to be changed is that
>>>>> the timestamp should still be done by the caller, not in read_seek2.
>>>>> now each implementation of read_seek2() has to duplicate this code.
>>>> if rescaling is done outside then exact seeking becomes impossible
>>>> because rescaling implicates rounding
>>> But that's always true, regardless. If you sepcify a seek timestamp in
>>> nanoseconds and the stream only supports microseconds, it is going to
>>> to round anyway.
>> there are normally several streams, and their timebases normally differ.
> That cannot be true in an FLV.
so your code uses timebase even though you know its 1000, thats not optimal
also my original point is still true rounding the way you do breaks exact
seeking. if i say seek to 0.9 or before and you change this to
1.0 or before thats just a different range and can seek to a different point
thats also not just seek2 its in the remaining code too
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
It is dangerous to be right in matters on which the established authorities
are wrong. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel