diff --git a/src/Magnum/Audio/Audio.cpp b/src/Magnum/Audio/Audio.cpp index 4384c776f..7758c1a63 100644 --- a/src/Magnum/Audio/Audio.cpp +++ b/src/Magnum/Audio/Audio.cpp @@ -33,12 +33,9 @@ namespace Magnum { namespace Audio { /* Verify types */ static_assert(std::is_same::value, "ALubyte is not the same as UnsignedByte"); -/** @todo Why `ALbyte` is defined as `char` and not `signed char` on OSX? */ -#ifndef CORRADE_TARGET_APPLE -static_assert(std::is_same::value, "ALbyte is not the same as Byte"); -#else -static_assert(std::is_signed::value && sizeof(ALbyte) == 1, "ALbyte does not have the same characteristics as Byte"); -#endif +/* ALbyte is defined as `signed char` in OpenAL Soft and `char` in the official + SDK from Creative and on Apple. Both are okay. */ +static_assert(std::is_same::value || std::is_signed::value && sizeof(ALbyte) == 1, "ALbyte does not have the characteristics as Byte"); static_assert(std::is_same::value, "ALushort is not the same as UnsignedShort"); static_assert(std::is_same::value, "ALshort is not the same as Short"); static_assert(std::is_same::value, "ALuint is not the same as UnsignedInt");