[opus] [PATCH] Add missing HAVE_CONFIG_H guards

Michael Bradshaw mjbshaw at google.com
Wed Feb 15 19:32:25 UTC 2017


On Wed, Feb 15, 2017 at 10:45 AM, Jean-Marc Valin <jmvalin at jmvalin.ca>
wrote:

> Hi Michael,
>
> I guess the main issue here is that config.h is never actually supposed
> to be included from a header file in the first place. So it might be
> best to fix that directly rather than adding ifdefs.


Thanks for the response. Attached patch removes the config.h include. I
tested by building Ne10 and libopus for both armv7 and arm64.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.xiph.org/pipermail/opus/attachments/20170215/29d44947/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Remove-include-config.h-from-ARM-Ne10-headers.patch
Type: application/octet-stream
Size: 925 bytes
Desc: not available
URL: <http://lists.xiph.org/pipermail/opus/attachments/20170215/29d44947/attachment.obj>


More information about the opus mailing list