[MPlayer-dev-eng] [PATCH] RTMP seek support

[Howard Chu]

Reimar Döffinger wrote:
> On Mon, Mar 29, 2010 at 05:48:00PM -0700, Howard Chu wrote:
>> Reimar Döffinger wrote:
>>> On Mon, Mar 29, 2010 at 07:32:35PM +0200, Reimar Döffinger wrote:
>>>> On Sun, Mar 28, 2010 at 08:02:16PM -0700, Howard Chu wrote:
>>>>> As a followon to this patch
>>>>>   https://lists.mplayerhq.hu/pipermail/ffmpeg-devel/2010-March/086051.html
>>>>>
>>>>> this patch is needed for mplayer to support rtmp seek. (The ffmpeg
>>>>> patch just gets it working for ffplay.)
>>>>
>>>> You will have to make sure the stream is actually lavf,
>>
>> Since rtmp support is only provided by ffmpeg, that seems to be a
>> foregone conclusion.
>
> But you're assuming that the stream->priv field is always a URLContext
> when the lavf _demuxer_ is used, and the lavf demuxer sure is used for
> other things than rtmp!

OK. This patch is functionally the same, but avoids the unsafe assumptions.

>>>> and even then
>>>> I'd like to think some more if this is not too hackish.
>>
>>> Now I remember why it felt so wrong: look at the existing
>>> STREAM_CTRL_SEEK_TO_TIME stuff for DVDs and try to reuse it if at
>>> all possible.

-- 
   -- 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...
Name: dif.txt
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20100330/cd9ff670/attachment.txt>