Re: [FFmpeg-devel] [PATCH 001/289 v6] Add a new channel layout API

[Andreas Rheinhardt <andreas.rheinhardt@outlook.com>]

James Almer:
> From: Anton Khirnov <anton@khirnov.net>
> 
> The new API is more extensible and allows for custom layouts.
> More accurate information is exported, eg for decoders that do not
> set a channel layout, lavc will not make one up for them.
> 
> Deprecate the old API working with just uint64_t bitmasks.
> 
> Expanded and completed by Vittorio Giovara <vittorio.giovara@gmail.com>
> and James Almer <jamrial@gmail.com>.
> Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
> Signed-off-by: James Almer <jamrial@gmail.com>
> ---
> Changes since last version:
> 
> *av_channel_layout_from_string() and av_channel_layout_describe() now support
>  a "designation@name" syntax, effectively making both functions reciprocal
>  when there are custom names in some or all channels.
>  It's the syntax suggested by Marton and is both extensible if required and
>  not too ulgy in human readable output if the string is printed verbatim.
> 
> *av_channel_layout_index_from_string() and
>  av_channel_layout_channel_from_string() also support this syntax now.
> 
> I plan to push this version soon. Any extension to the syntax supported by these
> helpers can be written by whoever needs it.

You forgot the examples and the tools; and ffplay. And do you really
intend for the deprecation warnings in ffmpeg_opt.c to stay?

- Andreas
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".