[FFmpeg-devel] [RFC] Removal of the "x264opts" libx264 wrapper AVOption
Lou Logan
lou at lrcd.com
Tue Mar 7 02:19:02 EET 2017
On Tue, 7 Mar 2017 00:20:19 +0200, Jan Ekström wrote:
> Hi,
>
> I am not sure what the correct order of things to remove an AVOption is,
> but I would like to propose that the "x264opts" AVOption be removed, as it
> duplicates "x264-params", which has a matching pair over at the libx265
> wrapper.
>
> Also, this way we would get rid of one in a group of similar features we
> have a duplicate of in our code base.
>
> Jan Ekström (2):
> libx264: remove the "x264opts" AVOption
> doc/encoders: remove mentions of the x264opts AVOption
>
> doc/encoders.texi | 25 ++++++-------------------
> libavcodec/libx264.c | 29 -----------------------------
> 2 files changed, 6 insertions(+), 48 deletions(-)
>
Instead of updating the -x264opts example that Marton pointed out you
can just delete it because an existing -x264-params example is
immediately below it.
The rest of the patch series LGTM.
More information about the ffmpeg-devel
mailing list