<p dir="ltr"><br>
On May 20, 2016 18:32, "Jean-Marc Valin" <<a href="mailto:jmvalin@jmvalin.ca">jmvalin@jmvalin.ca</a>> wrote:<br>
><br>
> Is it intended that this patch never sets ALLOCATION_MODE_AMBISONICS?<br>
> I'm having a hard time figuring out what it does in its current state.<br>
That was intended. I wanted to "reserve" the number 2 in this enum to make it clear that 2 would mean ambisonics in the future.</p>
<p dir="ltr">> Also, I think the ambisonics function should be completely disabled<br>
> (hidden behind EXPERIMENTAL_AMBISONICS/--enable-experimental-ambisonics<br>
> flags) until there's actually an approved spec to avoid problems in the<br>
> future.<br>
I believe I did that already. It should be the case that without that flag, this patch is a noop. Did I miss something?</p>
<p dir="ltr">>As for the mapping family to use, I'm not yet sure whether it's<br>
> best to use 2 or 255 for now -- Tim, any opinion?<br>
Isn't 255 best reserved to mean "no mapping" as it does now?<br>
</p>