[FFmpeg-devel] [PATCH] lavf/matroskaenc: sort options by name

Stefano Sabatini stefasab at gmail.com
Wed Apr 10 12:07:32 EEST 2024


On date Sunday 2024-04-07 16:01:27 +0800, Zhao Zhili wrote:
> 
> > On Apr 7, 2024, at 14:16, Anton Khirnov <anton at khirnov.net> wrote:
> > 
> > Quoting Andreas Rheinhardt (2024-04-06 13:25:49)
> >> See https://ffmpeg.org/pipermail/ffmpeg-devel/2024-February/320849.html
> >> Additionally I do not agree that sorting options by name is the best
> >> way; it should be sorted by what are (believed to be) the most commonly
> >> used options.
> > 
> > +1
> 
> https://patchwork.ffmpeg.org/project/ffmpeg/patch/20240106165246.274472-1-stefasab@gmail.com/
> 
> I have the same consideration in another patch. Maybe group related
> options together than sort whole options.

This hardly works for the aforementioned reasons, no objective
criteria means that there will be never a consistent way to sort the
options. What happens in practice is that options are added more or
less randomly, which doesn't help readability and discoverability
especially when there are many options.


More information about the ffmpeg-devel mailing list