[FFmpeg-devel] [PATCH 001/289 v6] Add a new channel layout API

Andreas Rheinhardt andreas.rheinhardt at outlook.com
Wed Jan 19 20:07:12 EET 2022


James Almer:
> From: Anton Khirnov <anton at khirnov.net>
> 
> The new API is more extensible and allows for custom layouts.
> More accurate information is exported, eg for decoders that do not
> set a channel layout, lavc will not make one up for them.
> 
> Deprecate the old API working with just uint64_t bitmasks.
> 
> Expanded and completed by Vittorio Giovara <vittorio.giovara at gmail.com>
> and James Almer <jamrial at gmail.com>.
> Signed-off-by: Vittorio Giovara <vittorio.giovara at gmail.com>
> Signed-off-by: James Almer <jamrial at gmail.com>
> ---
> Changes since last version:
> 
> *av_channel_layout_from_string() and av_channel_layout_describe() now support
>  a "designation at name" syntax, effectively making both functions reciprocal
>  when there are custom names in some or all channels.
>  It's the syntax suggested by Marton and is both extensible if required and
>  not too ulgy in human readable output if the string is printed verbatim.
> 
> *av_channel_layout_index_from_string() and
>  av_channel_layout_channel_from_string() also support this syntax now.
> 
> I plan to push this version soon. Any extension to the syntax supported by these
> helpers can be written by whoever needs it.

You forgot the examples and the tools; and ffplay. And do you really
intend for the deprecation warnings in ffmpeg_opt.c to stay?

- Andreas


More information about the ffmpeg-devel mailing list