[FFmpeg-devel] [PATCH 001/279 v2] Add a new channel layout API

James Almer jamrial at gmail.com
Thu Dec 16 21:14:25 EET 2021



On 12/16/2021 3:31 PM, Paul B Mahol wrote:
> On Thu, Dec 16, 2021 at 7:28 PM James Almer <jamrial at gmail.com> wrote:
> 
>>
>>
>> On 12/16/2021 2:20 PM, Paul B Mahol wrote:
>>> You can not use '|' as separator for channel layout definition.
>>> Use '+' instead.
>>
>> Is there a technical reason, or just a preference?
>>
> 
> Only for cases when you need to provide multiple channel layouts at once as
> string.
> How it is currently done with this set?

Mmh, guess you're right. aformat uses | to split layouts, so it would 
break if you pass it a combination of channels instead of a mask or a 
standard layout name. So we may need to use +.

> 
> 
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>> To unsubscribe, visit link above, or email
>> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
>>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".


More information about the ffmpeg-devel mailing list