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

James Almer jamrial at gmail.com
Mon Jan 17 15:54:32 EET 2022



On 1/17/2022 10:53 AM, Nicolas George wrote:
> Marton Balint (12022-01-16):
>> Should the serializaton and unserialization functions store/parse both the
>> channel label and the channel designation? As far as I see right now it is
>> kind of inconsistent: av_channel_layout_describe_print() prints the label
>> (if exists), not the designation, but av_channel_layout_from_string()
>> expects the designations only, never the custom label.
> 
> Oh, excellent point. av_channel_layout_from_string() should be able to
> parse the output of av_channel_layout_describe_bprint() in all cases.
> And it should be covered by FATE.

Yes, which is why I'll make describe() stop looking at the name field.

> 
> Regards,
> 
> 
> _______________________________________________
> 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