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

[Lynne]

On 31/05/2024 15:39, Jan Ekström wrote:
> On Thu, May 30, 2024 at 5:39 AM Lynne via ffmpeg-devel
> <ffmpeg-devel at ffmpeg.org> wrote:
>>
>> apichanges will be updated upon merging, as well as a version bump.
>> ---
>>   libavutil/channel_layout.c | 4 ++++
>>   libavutil/channel_layout.h | 8 ++++++++
>>   2 files changed, 12 insertions(+)
>>
>> diff --git a/libavutil/channel_layout.c b/libavutil/channel_layout.c
>> index 98839b7250..2d6963b6df 100644
>> --- a/libavutil/channel_layout.c
>> +++ b/libavutil/channel_layout.c
>> @@ -75,6 +75,10 @@ static const struct channel_name channel_names[] = {
>>       [AV_CHAN_BOTTOM_FRONT_CENTER  ] = { "BFC",       "bottom front center"   },
>>       [AV_CHAN_BOTTOM_FRONT_LEFT    ] = { "BFL",       "bottom front left"     },
>>       [AV_CHAN_BOTTOM_FRONT_RIGHT   ] = { "BFR",       "bottom front right"    },
>> +    [AV_CHAN_SIDE_SURROUND_LEFT   ] = { "SSL",       "side surround left"    },
>> +    [AV_CHAN_SIDE_SURROUND_RIGHT  ] = { "SSR",       "side surround right"   },
>> +    [AV_CHAN_TOP_SURROUND_LEFT    ] = { "TTL",       "top surround left"     },
>> +    [AV_CHAN_TOP_SURROUND_RIGHT   ] = { "TTR",       "top surround right"    },
> 
> Just noticed for these two top ones, is there a connection being "TTL"
> and "top surround left" that I somehow missed, or is this a typo of
> "TSL"?

TSL and TSR are already taken:

 > [AV_CHAN_TOP_SIDE_LEFT        ] = { "TSL",       "top side left" },
 > [AV_CHAN_TOP_SIDE_RIGHT       ] = { "TSR",       "top side right" },

I tried using "Beside" instead of side, which would have resulted in 
"TBR" and "TBL", but those are taken by Top Back Left/Right.

So I went with the short form of "Top Top Left" and "Top Top Right" 
without putting too much thought.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_0xA2FEA5F03F034464.asc
Type: application/pgp-keys
Size: 624 bytes
Desc: OpenPGP public key
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20240531/55803919/attachment.key>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature.asc
Type: application/pgp-signature
Size: 236 bytes
Desc: OpenPGP digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20240531/55803919/attachment.sig>