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

Nicolas George george at nsup.org
Tue Sep 20 12:05:59 EEST 2022


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 should have a way for the user to input
them, self-contained and convenient with regard with the logic of the
type, and a function to print them in a way that can be parsed back.

Channels layouts do not have multiple options, they have
av_channel_layout_from_string(). Rationals are not entered with
-framerate_num 24000 -framerate_den 1001, they are entered as a ratio,
and so on. It should be systematic.

Regards,

-- 
  Nicolas George


More information about the ffmpeg-devel mailing list