Hi, this is an updated and cleaned-up version of Jan's patchset discussed in [1], now v4... I'd especially appreciate any comments on 5/6, ffmpeg_opt.c:119ff which is pretty ugly as-is.
Previous reviews were split between threads [2][3] and are merged into. Now it comes with cover letter as the topic would have changed again, keep track of revision and notget reviews on the same stuff in seperate threads. Should fix #8329 and #6370. Thanks, Thilo [1] https://lists.ffmpeg.org/pipermail/ffmpeg-devel/2022-May/296553.html [2] https://lists.ffmpeg.org/pipermail/ffmpeg-devel/2022-August/300076.html [3] https://lists.ffmpeg.org/pipermail/ffmpeg-devel/2022-August/300122.html Jan Ekström (3): fftools: Add support for dictionary options ffmpeg: Add display_matrix option ffmpeg: Deprecate display rotation override with a metadata key Thilo Borgmann (3): lavu/opt: Allow options to be arguments of other options fftools/cmdutils: Print arguments of options lavu/display: Add horizontal and vertical scaling to the display matrix doc/APIchanges | 7 ++ doc/ffmpeg.texi | 18 ++++ fftools/cmdutils.c | 22 +++++ fftools/cmdutils.h | 3 + fftools/ffmpeg.c | 2 + fftools/ffmpeg.h | 7 ++ fftools/ffmpeg_filter.c | 13 +++ fftools/ffmpeg_opt.c | 162 +++++++++++++++++++++++++++++++++++- libavutil/display.c | 35 ++++++++ libavutil/display.h | 26 ++++++ libavutil/opt.c | 14 +++- libavutil/opt.h | 8 ++ libavutil/version.h | 4 +- tests/fate/filter-video.mak | 2 +- 14 files changed, 316 insertions(+), 7 deletions(-) -- 2.20.1 (Apple Git-117) _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
