<div dir="ltr">Thanks Jean-Marc!<div><br></div><div>I'm alittle under the weather today but I'll address your comments tomorrow. Feel free to wait on my patch for libopusenc until tomorrow as well, as I will have a new patch sent out then.</div><div><br></div><div>Cheers,</div><div>Drew</div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Cheers,<div>Drew</div></div></div></div>
<br><div class="gmail_quote">On Mon, Mar 12, 2018 at 10:06 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 Drew,<br>
<br>
I think the idea of returning OPUS_UNIMPLEMENTED for all projection<br>
encoder/decoder calls is pretty reasonable. I haven't looked in detail,<br>
but I think some of the calls are missing. I think all calls should have<br>
that behaviour and not just the ones used by libopusenc.<br>
<br>
As for the includes, how about leaving<br>
OPUS_PROJECTION_*_REQUEST in opus_projection.h, but defining<br>
OPUS_HAVE_OPUS_PROJECTION_H (or something like that) in opus.h?<br>
<br>
I still haven't looked at your libopusenc changes, that's my next step.<br>
<br>
Cheers,<br>
<br>
        Jean-Marc<br>
<span class=""><br>
On 03/08/2018 06:43 PM, Drew Allen wrote:<br>
> Hello Jean-Marc et all,<br>
><br>
> I've attached an updated patch which I feel addresses all 3 cases.<br>
><br>
> PTAL, Thanks!<br>
><br>
> Cheers,<br>
> Drew<br>
><br>
> On Wed, Mar 7, 2018 at 8:39 PM Jean-Marc Valin <<a href="mailto:jmvalin@jmvalin.ca">jmvalin@jmvalin.ca</a><br>
</span><span class="">> <mailto:<a href="mailto:jmvalin@jmvalin.ca">jmvalin@jmvalin.ca</a>>> wrote:<br>
><br>
>     Make sure the modified libraries work in all of these three cases:<br>
>     1) libopus 1.2.1<br>
>     2) libopus master (or 1.3-beta) with --enable-ambisonics<br>
>     3) libopus master (or 1.3-beta) with --disable-ambisonics<br>
><br>
>     I suspect with the current set of patches you might have problems with<br>
>     case 3) since you will find the symbols in the header file, but the<br>
>     functionality won't be there.<br>
><br>
>     Cheers,<br>
><br>
>             Jean-Marc<br>
><br>
>     On 03/07/2018 02:40 PM, Drew Allen wrote:<br>
>     > Move demixing matrix defines to opus_define to better determine<br>
>     > availability of Projection API.<br>
>     ><br>
>     > Allows libopusenc, opusfile and opus-tools to much more easily<br>
>     determine<br>
>     > availability of Projection API.<br>
>     ><br>
>     > Cheers,<br>
>     > Drew<br>
>     ><br>
>     ><br>
>     > ______________________________<wbr>_________________<br>
>     > opus mailing list<br>
</span>>     > <a href="mailto:opus@xiph.org">opus@xiph.org</a> <mailto:<a href="mailto:opus@xiph.org">opus@xiph.org</a>><br>
>     > <a href="http://lists.xiph.org/mailman/listinfo/opus" rel="noreferrer" target="_blank">http://lists.xiph.org/mailman/<wbr>listinfo/opus</a><br>
>     ><br>
><br>
</blockquote></div><br></div>