[FFmpeg-devel] [PATCH] http Transfer-Encoding chunked

Ronald S. Bultje rsbultje
Sat May 23 14:25:58 CEST 2009


Hi Peter,

On Sat, May 23, 2009 at 5:55 AM, Peter Holik <peter at holik.at> wrote:
>>> The only drawback of my patch is, that the filesize is limited (i've taken 512 kByte)
>
>>Just set filesize to -1 then, i.e. unlimited. Don't make up some
>>arbitrary number. It should just work.
>
> Sorry, but this doesn't work for "image2"

Ah. img2.c uses AVFMT_NOFILE + local variable is_pipe for this and
then assumes url_fsize() gives valid output if AVFMT_NOFILE is not
set. A better behaviour would be to take into account the return value
of url_fsize(), and if it is -1, then revert back to the pipe-like
behaviour anyway.

This part of your patch feels like a hack to me, I'd prefer if you'd
fix this more "cleanly".

> I can't imagine a need for "multiple chunks", please give me an example.

Imagine that we want to write a dav:// module, or extend the http://
module for DAV support (since dav is essentially just a http
extension). Anyway, I agree this is outside the scope of this patch,
so please ignore. :-).

Ronald



More information about the ffmpeg-devel mailing list