From af2a12f1dcec9fc30160a016cdd21d79d78beac6 Mon Sep 17 00:00:00 2001 From: Christophe Riccio Date: Sat, 7 May 2011 16:00:07 +0100 Subject: [PATCH] Fixed assert --- glm/core/func_integer.inl | 92 +++++++++++++++++++-------------------- 1 file changed, 46 insertions(+), 46 deletions(-) diff --git a/glm/core/func_integer.inl b/glm/core/func_integer.inl index c5e6fe25..bf6ac2e4 100644 --- a/glm/core/func_integer.inl +++ b/glm/core/func_integer.inl @@ -20,7 +20,7 @@ namespace glm { // uaddCarry template - inline genUType uaddCarry + GLM_FUNC_QUALIFIER genUType uaddCarry ( genUType const & x, genUType const & y, @@ -34,7 +34,7 @@ namespace glm } template - inline detail::tvec2 uaddCarry + GLM_FUNC_QUALIFIER detail::tvec2 uaddCarry ( detail::tvec2 const & x, detail::tvec2 const & y, @@ -47,7 +47,7 @@ namespace glm } template - inline detail::tvec3 uaddCarry + GLM_FUNC_QUALIFIER detail::tvec3 uaddCarry ( detail::tvec3 const & x, detail::tvec3 const & y, @@ -61,7 +61,7 @@ namespace glm } template - inline detail::tvec4 uaddCarry + GLM_FUNC_QUALIFIER detail::tvec4 uaddCarry ( detail::tvec4 const & x, detail::tvec4 const & y, @@ -77,7 +77,7 @@ namespace glm // usubBorrow template - inline genUType usubBorrow + GLM_FUNC_QUALIFIER genUType usubBorrow ( genUType const & x, genUType const & y, @@ -92,7 +92,7 @@ namespace glm } template - inline detail::tvec2 usubBorrow + GLM_FUNC_QUALIFIER detail::tvec2 usubBorrow ( detail::tvec2 const & x, detail::tvec2 const & y, @@ -105,7 +105,7 @@ namespace glm } template - inline detail::tvec3 usubBorrow + GLM_FUNC_QUALIFIER detail::tvec3 usubBorrow ( detail::tvec3 const & x, detail::tvec3 const & y, @@ -119,7 +119,7 @@ namespace glm } template - inline detail::tvec4 usubBorrow + GLM_FUNC_QUALIFIER detail::tvec4 usubBorrow ( detail::tvec4 const & x, detail::tvec4 const & y, @@ -135,7 +135,7 @@ namespace glm // umulExtended template - inline void umulExtended + GLM_FUNC_QUALIFIER void umulExtended ( genUType const & x, genUType const & y, @@ -151,7 +151,7 @@ namespace glm } template - inline detail::tvec2 umulExtended + GLM_FUNC_QUALIFIER detail::tvec2 umulExtended ( detail::tvec2 const & x, detail::tvec2 const & y, @@ -165,7 +165,7 @@ namespace glm } template - inline detail::tvec3 umulExtended + GLM_FUNC_QUALIFIER detail::tvec3 umulExtended ( detail::tvec3 const & x, detail::tvec3 const & y, @@ -180,7 +180,7 @@ namespace glm } template - inline detail::tvec4 umulExtended + GLM_FUNC_QUALIFIER detail::tvec4 umulExtended ( detail::tvec4 const & x, detail::tvec4 const & y, @@ -213,7 +213,7 @@ namespace glm } template - inline detail::tvec2 imulExtended + GLM_FUNC_QUALIFIER detail::tvec2 imulExtended ( detail::tvec2 const & x, detail::tvec2 const & y, @@ -227,7 +227,7 @@ namespace glm } template - inline detail::tvec3 imulExtended + GLM_FUNC_QUALIFIER detail::tvec3 imulExtended ( detail::tvec3 const & x, detail::tvec3 const & y, @@ -242,7 +242,7 @@ namespace glm } template - inline detail::tvec4 imulExtended + GLM_FUNC_QUALIFIER detail::tvec4 imulExtended ( detail::tvec4 const & x, detail::tvec4 const & y, @@ -267,21 +267,21 @@ namespace glm ) { GLM_STATIC_ASSERT(std::numeric_limits::is_integer, "'bitfieldExtract' only accept integer values"); - assert(Offset + Bits <= sizeof(genIUType)); + assert(Offset + Bits <= (sizeof(genIUType) << 3)); - genIUType Result = 0; + genIUType Result(0); if(std::numeric_limits::is_signed) - Result |= (1 << (sizeof(genIUType) * 8 - 1)) & (1 << (Offset + Bits - 1)); + Result |= (genIUType(1) << (sizeof(genIUType) * genIUType(8) - genIUType(1))) & (genIUType(1) << (Offset + Bits - genIUType(1))); - genIUType Mask = 0; + genIUType Mask(0); for(int Bit = Offset; Bit < Bits; ++Bit) - Mask |= (1 << Bit); + Mask |= (genIUType(1) << Bit); return Result | ((Mask & Value) >> Offset); } template - inline detail::tvec2 bitfieldExtract + GLM_FUNC_QUALIFIER detail::tvec2 bitfieldExtract ( detail::tvec2 const & Value, int const & Offset, @@ -294,7 +294,7 @@ namespace glm } template - inline detail::tvec3 bitfieldExtract + GLM_FUNC_QUALIFIER detail::tvec3 bitfieldExtract ( detail::tvec3 const & Value, int const & Offset, @@ -308,7 +308,7 @@ namespace glm } template - inline detail::tvec4 bitfieldExtract + GLM_FUNC_QUALIFIER detail::tvec4 bitfieldExtract ( detail::tvec4 const & Value, int const & Offset, @@ -324,7 +324,7 @@ namespace glm // bitfieldInsert template - inline genIUType bitfieldInsert + GLM_FUNC_QUALIFIER genIUType bitfieldInsert ( genIUType const & Base, genIUType const & Insert, @@ -340,13 +340,13 @@ namespace glm genIUType Mask = 0; for(int Bit = Offset; Bit < Offset + Bits; ++Bit) - Mask |= (1 << Bit); + Mask |= (genIUType(1) << Bit); return (Base & ~Mask) | (Insert & Mask); } template - inline detail::tvec2 bitfieldInsert + GLM_FUNC_QUALIFIER detail::tvec2 bitfieldInsert ( detail::tvec2 const & Base, detail::tvec2 const & Insert, @@ -360,7 +360,7 @@ namespace glm } template - inline detail::tvec3 bitfieldInsert + GLM_FUNC_QUALIFIER detail::tvec3 bitfieldInsert ( detail::tvec3 const & Base, detail::tvec3 const & Insert, @@ -375,7 +375,7 @@ namespace glm } template - inline detail::tvec4 bitfieldInsert + GLM_FUNC_QUALIFIER detail::tvec4 bitfieldInsert ( detail::tvec4 const & Base, detail::tvec4 const & Insert, @@ -392,19 +392,19 @@ namespace glm // bitfieldReverse template - inline genIUType bitfieldReverse(genIUType const & Value) + GLM_FUNC_QUALIFIER genIUType bitfieldReverse(genIUType const & Value) { GLM_STATIC_ASSERT(std::numeric_limits::is_integer, "'bitfieldReverse' only accept integer values"); genIUType Result = 0; for(std::size_t i = 0; i < sizeof(genIUType) * std::size_t(8); ++i) - if(Value & (1 << i)) - Result |= (genIUType(1) << (sizeof(genIUType) * std::size_t(8)) - genIUType(1) - i); + if(Value & (genIUType(1) << genIUType(i))) + Result |= (genIUType(1) << (genIUType(sizeof(genIUType)) * genIUType(8)) - genIUType(1) - genIUType(i)); return Result; } template - inline detail::tvec2 bitfieldReverse + GLM_FUNC_QUALIFIER detail::tvec2 bitfieldReverse ( detail::tvec2 const & value ) @@ -415,7 +415,7 @@ namespace glm } template - inline detail::tvec3 bitfieldReverse + GLM_FUNC_QUALIFIER detail::tvec3 bitfieldReverse ( detail::tvec3 const & value ) @@ -427,7 +427,7 @@ namespace glm } template - inline detail::tvec4 bitfieldReverse + GLM_FUNC_QUALIFIER detail::tvec4 bitfieldReverse ( detail::tvec4 const & value ) @@ -448,14 +448,14 @@ namespace glm int Count = 0; for(std::size_t i = 0; i < sizeof(genIUType) * std::size_t(8); ++i) { - if(Value & (1 << i)) + if(Value & (genIUType(1) << i)) ++Count; } return Count; } template - inline detail::tvec2 bitCount + GLM_FUNC_QUALIFIER detail::tvec2 bitCount ( detail::tvec2 const & value ) @@ -466,7 +466,7 @@ namespace glm } template - inline detail::tvec3 bitCount + GLM_FUNC_QUALIFIER detail::tvec3 bitCount ( detail::tvec3 const & value ) @@ -478,7 +478,7 @@ namespace glm } template - inline detail::tvec4 bitCount + GLM_FUNC_QUALIFIER detail::tvec4 bitCount ( detail::tvec4 const & value ) @@ -492,7 +492,7 @@ namespace glm // findLSB template - inline int findLSB + GLM_FUNC_QUALIFIER int findLSB ( genIUType const & Value ) @@ -507,7 +507,7 @@ namespace glm } template - inline detail::tvec2 findLSB + GLM_FUNC_QUALIFIER detail::tvec2 findLSB ( detail::tvec2 const & value ) @@ -518,7 +518,7 @@ namespace glm } template - inline detail::tvec3 findLSB + GLM_FUNC_QUALIFIER detail::tvec3 findLSB ( detail::tvec3 const & value ) @@ -530,7 +530,7 @@ namespace glm } template - inline detail::tvec4 findLSB + GLM_FUNC_QUALIFIER detail::tvec4 findLSB ( detail::tvec4 const & value ) @@ -544,7 +544,7 @@ namespace glm // findMSB template - inline int findMSB + GLM_FUNC_QUALIFIER int findMSB ( genIUType const & Value ) @@ -559,7 +559,7 @@ namespace glm } template - inline detail::tvec2 findMSB + GLM_FUNC_QUALIFIER detail::tvec2 findMSB ( detail::tvec2 const & value ) @@ -570,7 +570,7 @@ namespace glm } template - inline detail::tvec3 findMSB + GLM_FUNC_QUALIFIER detail::tvec3 findMSB ( detail::tvec3 const & value ) @@ -582,7 +582,7 @@ namespace glm } template - inline detail::tvec4 findMSB + GLM_FUNC_QUALIFIER detail::tvec4 findMSB ( detail::tvec4 const & value )