[FFmpeg-devel] [PATCH v1 1/3] avformat/http: Add option to disable send-100-continue

Jun Li junli1026 at gmail.com
Wed Mar 20 00:55:58 EET 2019


On Tue, Mar 19, 2019 at 3:21 PM Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:

> 2019-03-19 22:58 GMT+01:00, Jun Li <junli1026 at gmail.com>:
> > The current setting for send-100-continue is either
> > applicable or enabled, no option to disable the header.
> > This change is to expand the option setting to provide
> > more flexibility, which is useful for rstp case.
> > ---
> >  libavformat/http.c | 15 +++++++++------
> >  1 file changed, 9 insertions(+), 6 deletions(-)
> >
> > diff --git a/libavformat/http.c b/libavformat/http.c
> > index ed0eb1c875..7e74719450 100644
> > --- a/libavformat/http.c
> > +++ b/libavformat/http.c
> > @@ -113,7 +113,7 @@ typedef struct HTTPContext {
> >      uint8_t *inflate_buffer;
> >  #endif /* CONFIG_ZLIB */
> >      AVDictionary *chained_options;
> > -    int send_expect_100;
> > +    // -1 = try to send if applicable, 0 = always disabled, 1 = always
> > enabled
> >      int send_expect_100;
> >      char *method;
> >      int reconnect;
> >      int reconnect_at_eof;
>
> It seems to me that something is broken in this hunk.
>
> Carl Eugen
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Yes Carl, Thanks for review.
I updated the version: https://patchwork.ffmpeg.org/patch/12359/


More information about the ffmpeg-devel mailing list