[opus] [PATCH 1/1] Updated opus_types.h to correctly support 8 and 64 bit types.
Simon Jackson
si at sonocent.com
Wed Jun 3 09:50:10 PDT 2015
Indentation looked ok to me, its probably due to my editor. I will fix it in vim and re-submit it.
Didn’t know that, will fix.
No they haven’t all been tested, however I consulted documentation available for each of the platforms on the internet and copied the ogg headers when in doubt.
> On 3 Jun 2015, at 17:43, Mark Harris <mark.hsj at gmail.com> wrote:
>
> Simon Jackson wrote:
>> + typedef char opus_int8;
>
> ISO C allows the "char" type to be either signed or unsigned. It is
> even a compiler switch on some compilers. Changing this from "signed
> char" to plain "char" breaks when "char" is unsigned.
>
> Several of the changed lines introduce inconsistent indentation or
> spaces at the end of the line, which is somewhat unfitting for a patch
> whose purpose is to improve consistency.
>
> Have each of the affected #elif sections been tested on the platforms
> where that section is compiled?
>
> - Mark
More information about the opus
mailing list