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

[Lynne]

On 25/05/2024 08:10, Marton Balint wrote:
> 
> 
> 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.


Thanks, changed locally.
Planning on merging this in 2 days unless there are more comments.
-------------- 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/20240526/b5a9e37d/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/20240526/b5a9e37d/attachment.sig>