[FFmpeg-devel] [PATCH] RTMP seek support
Wed Apr 7 07:50:21 CEST 2010
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 need
>>>>>> to change it still ...
>>>> It seems to me that one thing that ought to be changed is that rescaling
>>>> 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 have
>> to round anyway.
> there are normally several streams, and their timebases normally differ.
That cannot be true in an FLV.
Anyway, I give up. I have no idea what you expect a real read_seek2()
implementation to do in this context. If these patches are wrong, whatever.
The functionality is broken right now and I'm tired of trying to guess what
other problem is going to crop up.
-- Howard Chu
CTO, Symas Corp. http://www.symas.com
Director, Highland Sun http://highlandsun.com/hyc/
Chief Architect, OpenLDAP http://www.openldap.org/project/
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
More information about the ffmpeg-devel