[FFmpeg-devel] [PATCH v3 01/10] channel_layout: add new channel positions supported by xHE-AAC

[Marton Balint]

On Sat, 25 May 2024, Lynne via ffmpeg-devel wrote:

> apichanges will be updated upon merging, as well as a version bump.
> ---
> libavutil/channel_layout.h | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/libavutil/channel_layout.h b/libavutil/channel_layout.h
> index 8a078d1601..4e19bbbd9e 100644
> --- a/libavutil/channel_layout.h
> +++ b/libavutil/channel_layout.h
> @@ -79,6 +79,10 @@ enum AVChannel {
>     AV_CHAN_BOTTOM_FRONT_CENTER,
>     AV_CHAN_BOTTOM_FRONT_LEFT,
>     AV_CHAN_BOTTOM_FRONT_RIGHT,
> +    AV_CHAN_SURROUND_LEFT,
> +    AV_CHAN_SURROUND_RIGHT,

You want to add a channel ID for Surround or Side Surround? Because based 
on the subsequent AAC patch you want to add it for side surround, but then 
the AV_CHAN_SURROUND name is confusing, since we are mapping Surround to 
AV_CHAN_SIDE. So I suggest using AV_CHAN_SIDE_SURROUND_LEFT/RIGHT instead.

> +    AV_CHAN_TOP_SURROUND_LEFT,
> +    AV_CHAN_TOP_SURROUND_RIGHT,

You will need to extend the channel_names[] array in channel_layout.c with 
the newly added channel IDs.

Regards,
Marton