From: Thilo Borgmann <thilo.borgmann@mail.de> To: ffmpeg-devel@ffmpeg.org Subject: [FFmpeg-devel] [PATCH v4 0/6] Add display_matrix option Date: Mon, 19 Sep 2022 11:45:58 +0200 Message-ID: <20220919094604.4645-1-thilo.borgmann@mail.de> (raw) 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-request@ffmpeg.org with subject "unsubscribe".
next reply other threads:[~2022-09-19 9:46 UTC|newest] Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-09-19 9:45 Thilo Borgmann [this message] 2022-09-19 9:45 ` [FFmpeg-devel] [PATCH v4 1/6] lavu/opt: Allow options to be arguments of other options Thilo Borgmann 2022-09-19 9:46 ` [FFmpeg-devel] [PATCH v4 2/6] fftools/cmdutils: Print arguments of options Thilo Borgmann 2022-09-19 9:46 ` [FFmpeg-devel] [PATCH v4 3/6] fftools: Add support for dictionary options Thilo Borgmann 2022-09-19 9:46 ` [FFmpeg-devel] [PATCH v4 4/6] lavu/display: Add horizontal and vertical scaling to the display matrix Thilo Borgmann 2022-09-19 13:37 ` Andreas Rheinhardt 2022-09-19 9:46 ` [FFmpeg-devel] [PATCH v4 5/6] ffmpeg: Add display_matrix option Thilo Borgmann 2022-09-19 9:46 ` [FFmpeg-devel] [PATCH v4 6/6] ffmpeg: Deprecate display rotation override with a metadata key Thilo Borgmann 2022-09-20 8:36 ` [FFmpeg-devel] [PATCH v4 0/6] Add display_matrix option Anton Khirnov 2022-09-20 9:05 ` Nicolas George 2022-09-20 9:23 ` Anton Khirnov
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20220919094604.4645-1-thilo.borgmann@mail.de \ --to=thilo.borgmann@mail.de \ --cc=ffmpeg-devel@ffmpeg.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel This inbox may be cloned and mirrored by anyone: git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \ ffmpegdev@gitmailbox.com public-inbox-index ffmpegdev Example config snippet for mirrors. AGPL code for this site: git clone https://public-inbox.org/public-inbox.git