Re: [FFmpeg-devel] [PATCH v4 0/6] Add display_matrix option

2022-09-20 Thread Anton Khirnov
Quoting Nicolas George (2022-09-20 11:05:59) > Anton Khirnov (12022-09-20): > > I still see no convincing arguments in favor of adding all this > > complexity just so that we can stuff multiple options into one and get a > > WORSE user interface. > > > > Just add multiple options. > > We agree, e

Re: [FFmpeg-devel] [PATCH v4 0/6] Add display_matrix option

2022-09-20 Thread Nicolas George
Anton Khirnov (12022-09-20): > I still see no convincing arguments in favor of adding all this > complexity just so that we can stuff multiple options into one and get a > WORSE user interface. > > Just add multiple options. We agree, except on the last point. All user-visible FFmpeg types shoul

Re: [FFmpeg-devel] [PATCH v4 0/6] Add display_matrix option

2022-09-20 Thread Anton Khirnov
I still see no convincing arguments in favor of adding all this complexity just so that we can stuff multiple options into one and get a WORSE user interface. Just add multiple options. -- Anton Khirnov ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg

[FFmpeg-devel] [PATCH v4 0/6] Add display_matrix option

2022-09-19 Thread Thilo Borgmann
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