[FFmpeg-devel] [PATCH] http Transfer-Encoding chunked
Peter Holik
holik
Mon May 25 17:00:10 CEST 2009
>> I reworked my patch with no limitation and set filesize to -1.
> Thanks! Minor nitpicks now:
> - please use uint64_t for chunksize
i've taken int64_t like filesize
> - strncmp(p, "chunked", 7) -> please use strncasecmp(), I've seen
> "Chunked" and "chunked" be used
fixed
> - (optional) you're now using 2 variables for chunks: chunked
> (boolean) and chunksize. You could consider setting chunksize to -1
> (0xFFFFFFFFFFFFFFFFULL) by default and setting it to 0 if chunked.
done
> Michael might disagree, if he does, follow whatever he says (and be
> sure to document the variable appropriately if you do this)
> - document chunk_size and its relation to file_size please, and if
> yuou keep chunked, please document that also
where should i document the variable?
cu Peter
More information about the ffmpeg-devel
mailing list