[libav-devel] [PATCH 04/18] http: add encoding flag to chunksize option

Anton Khirnov anton at khirnov.net
Thu Nov 10 11:36:16 CET 2011


On Thu, 10 Nov 2011 10:55:23 +0200 (EET), Martin Storsjö <martin at martin.st> wrote:
> On Thu, 10 Nov 2011, Anton Khirnov wrote:
> 
> > ---
> > libavformat/http.c |    3 ++-
> > 1 files changed, 2 insertions(+), 1 deletions(-)
> >
> > diff --git a/libavformat/http.c b/libavformat/http.c
> > index 84300b1..78d357c 100644
> > --- a/libavformat/http.c
> > +++ b/libavformat/http.c
> > @@ -52,8 +52,9 @@ typedef struct {
> > } HTTPContext;
> >
> > #define OFFSET(x) offsetof(HTTPContext, x)
> > +#define E AV_OPT_FLAG_ENCODING_PARAM
> > static const AVOption options[] = {
> > -{"chunksize", "use chunked transfer-encoding for posts, -1 disables it, 0 enables it", OFFSET(chunksize), AV_OPT_TYPE_INT64, {.dbl = 0}, -1, 0 }, /* Default to 0, for chunked POSTs */
> > +{"chunksize", "use chunked transfer-encoding for posts, -1 disables it, 0 enables it", OFFSET(chunksize), AV_OPT_TYPE_INT64, {.dbl = 0}, -1, 0, E }, /* Default to 0, for chunked POSTs */
> > {NULL}
> > };
> > #define HTTP_CLASS(flavor)\
> > -- 
> > 1.7.7
> 
> Ok, although this has to be rebased on top of the patches I just pushed 
> (adding the "headers" avoption, which is usable for both reading and 
> writing aka D|E). Also, as discussed on IRC, I'll try to rewrite this 
> AVOption into something that makes more sense to use externally. Since 
> this patch (or the corresponding rebased one) is quite simple and doesn't 
> depend on the rest of this patchset, I guess that one can go in quite 
> soon, and my upcoming patch can be based on top of that then.
> 

I'll drop this then.

-- 
Anton Khirnov


More information about the libav-devel mailing list