<div dir="ltr">Alright, I've made the changes to the enum and flag usage.<br><br>I'm don't see the difference with respect to "disabling blocks of code", since the compiler should completely elide the blocks that are conditioned on a constant 0. If there were compiler errors in the experimental code there could be a difference, but I'm hoping that is not the case on any platform! Either way I changed them to #ifdef everywhere.<br><br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 24, 2016 at 8:26 AM, Jean-Marc Valin <span dir="ltr"><<a href="mailto:jmvalin@jmvalin.ca" target="_blank">jmvalin@jmvalin.ca</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Michael,<br>
<span class=""><br>
On 05/24/2016 01:14 AM, Michael Graczyk wrote:<br>
> Thanks for pointing out the make differences. Do you mind if I add an<br>
> #ifndef, #define to the top of the file for the experiment flag? The<br>
> code becomes significantly more nasty with #ifdefs everywhere and it<br>
> would only get worse in subsequent patches.<br>
<br>
</span>Unless it makes the code really bad, I'd rather use #ifdefs directly<br>
since we'll want to use it to disable blocks of code and mostly because<br>
everything else already uses that (it would be confusing).<br>
<br>
Thanks,<br>
<br>
        Jean-Marc<br>
<span class="im HOEnZb"><br>
<br>
> On May 23, 2016 21:28, "Jean-Marc Valin" <<a href="mailto:jmvalin@jmvalin.ca">jmvalin@jmvalin.ca</a><br>
</span><div class="HOEnZb"><div class="h5">> <mailto:<a href="mailto:jmvalin@jmvalin.ca">jmvalin@jmvalin.ca</a>>> wrote:<br>
><br>
>     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>
><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><br><div>Thanks,</div><div>Michael Graczyk</div></div></div>
</div>