On Mon, May 8, 2017 at 3:14 PM, Luca Barbato <[email protected]> wrote: > On 5/8/17 6:28 PM, Vittorio Giovara wrote: >> Ambisonic channel layouts have a non-native channel ordering and all >> channels in the map are initialized to a custom ambisonic channel since >> ordering is assumed to be ACN. >> >> Signed-off-by: Vittorio Giovara <[email protected]> >> --- >> Preview patchset. >> Vittorio >> >> libavutil/channel_layout.c | 105 >> +++++++++++++++++++++++++++++++++++++++++++-- >> libavutil/channel_layout.h | 41 ++++++++++++++++++ >> 2 files changed, 143 insertions(+), 3 deletions(-) >> > > Looks like that the channel count should be enough to describe the > ambisonic channels. > > I'd consider having one ordering for canonical ambisonic that just > requires to use the channel count if it is common enough. > > If having ambisonic + some channels is common enough and has a canonical > layout as well we could use the mask field to store the number of > ambisonic channels. > > This way we'd have to use the map only for custom/special layouts as well.
I'm open to any alternative, such as adding another field to the union listing the number of ambisonic channels instead of using a map, if doable. -- Vittorio _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
