<p dir="ltr">Sounds good, I'll make those changes.</p>
<p dir="ltr">Thanks for pointing out the make differences. Do you mind if I add an #ifndef, #define to the top of the file for the experiment flag? The code becomes significantly more nasty with #ifdefs everywhere and it would only get worse in subsequent patches.</p>
<div class="gmail_quote">On May 23, 2016 21:28, "Jean-Marc Valin" <<a href="mailto:jmvalin@jmvalin.ca">jmvalin@jmvalin.ca</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 05/23/2016 10:57 PM, Michael Graczyk wrote:<br>
> Since they correspond to mapping family values, I figured it would be<br>
> wise to make them match so that the correspondence would be clear. If<br>
> you would rather that correspondence not be explicit I will remove the<br>
> explicit assignments.<br>
<br>
I'd rather have them be different, since they're not really the same<br>
thing. The idea is that we may have more than one family map to the same<br>
type (e.g. currently both families 0 and 255 map to TYPE_NONE).<br>
<br>
One last thing I just noticed, please don't rely on<br>
ENABLE_EXPERIMENTAL_AMBISONICS actually being defined to zero when your<br>
patch is disabled. Some people don't use the autotools build (e.g. they<br>
use the plain Makefile or Visual Studio, or custom scripts), so your<br>
patch would break their build. Instead, please rely on #ifdef's.<br>
<br>
Cheers,<br>
<br>
        Jean-Marc<br>
</blockquote></div>