[FFmpeg-devel] [PATCH v2] avformat/hlsenc: deprecate hls_wrap option

Steven Liu lingjiujianke at gmail.com
Thu Feb 9 13:04:04 EET 2017


2017-02-09 18:53 GMT+08:00 Carl Eugen Hoyos <ceffmpeg at gmail.com>:

> 2017-02-09 11:52 GMT+01:00 Steven Liu <lingjiujianke at gmail.com>:
> > 2017-02-09 18:46 GMT+08:00 Carl Eugen Hoyos <ceffmpeg at gmail.com>:
> >
> >> 2017-02-09 8:58 GMT+01:00 Steven Liu <lq at chinaffmpeg.org>:
> >>
> >> > +#if FF_API_HLS_WRAP
> >> >      {"hls_wrap",      "set number after which the index wraps",
> >> OFFSET(wrap),    AV_OPT_TYPE_INT,    {.i64 = 0},     0, INT_MAX, E},
> >> > +#endif
> >>
> >> (Even if this is not done for other deprecated options:)
> >> Imo, either add the word deprecated, or even replace the whole
> >> description string with the word "deprecated".
> >>
> > Your mean: s/set number after which the index wraps/ this option is
> > deprecated/g
>
> I would suggest this, yes.
>
Apply your good suggest :)

> (Without having looked at other deprecated options)
>
> Carl Eugen
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>


More information about the ffmpeg-devel mailing list