[FFmpeg-devel] [PATCH] avformat/http: remove duplicate user-agent option
u at pkh.me
Thu Sep 15 18:37:34 EEST 2016
On Thu, Sep 15, 2016 at 11:27:02PM +0800, Steven Liu wrote:
> 2016-09-15 23:03 GMT+08:00 Carl Eugen Hoyos <ceffmpeg at gmail.com>:
> > 2016-09-15 16:40 GMT+02:00 Clément Bœsch <u at pkh.me>:
> > >> Carl Eugen's mean is apply this patch after the next release?
> > >
> > > You need to deprecate the option with at least the classic #ifdefery
> > dance
> > > (git grep FF_API for examples), and ideally by also printing a warning
> > > when using the wrong option.
> > Imo, it is completely sufficient to add a line to the filter documentation
> > (and print a warning if this is easily possible).
> > Carl Eugen
> user_agent and user-agent use the same variable 'user_agent', if print a
> warning, maybe add a user variable for user_agent option is a good way?
Yes, surrounded by the #ifdefery so it will go away with the option.
Note: use av_strdup() to transfer from one field to the other
More information about the ffmpeg-devel