[MPlayer-dev-eng] Re: [PATCH] Seek in http-streams

[Valentine Zaretsky]

Hi, Eviv!

Thank you for helping me with learning the reason of this breakage!

eviv bulgroz wrote:

>--- Attila Kinali <attila at kinali.ch> wrote:
>  
>
>>On Mon, 23 Feb 2004 12:21:59 -0500
>>D Richard Felker III <dalias at aerifal.cx> wrote:
>>
>>    
>>
>>>It's probably non-interleaved. In which case there's no way to play
>>>the file from stdin without 600 megs of cache.
>>>      
>>>
>>It's not and i made sure it played with mplayer before
>>the seeking patch was applied.
>>
>>    
>>
>
>Valentine, at least part of your patch problem is probably in the following code since s->seek is
>set when reading from stdin for avi streams. seekability should be check here too.
>  
>

I think that if seek callback is available we should use it, otherwise 
why is it set at all???
It seems that setting seek_forward as callback for stdin is not very useful.

>Better would be to sort ot this whole seekable/STREAM_SEEK_FW/STREAM_SEEK_BW mess on non seekable
>stream. Why is STREAM_SEEK_FW set on an non seekable stream ?
>
Yes, I agree with you that it's rather strange.
Albeu, what do you think?

-- 
Best regards, Valentine