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. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
