[Android][Clang 3.2] Lots of "shift-op-parentheses" warnings

[ghost]

One particular line in GLM causes insane amounts of "shift-op-parentheses" warnings during compilation with Android NDK r8e/Clang 3.2:

glm-0.9.4.0/glm/core/func_integer.inl:106:95: warning: operator '<<' has lower precedence than '-'; '-' will be evaluated first [-Wshift-op-parentheses]
                        return genUType(detail::highp_int_t(1) << detail::highp_int_t(32) + detail::highp_int_t(x) - detail::highp_int_t(y));
                                                               ~~ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~
glm-0.9.4.0/glm/core/func_integer.inl:106:95: note: place parentheses around the '-' expression to silence this warning
                        return genUType(detail::highp_int_t(1) << detail::highp_int_t(32) + detail::highp_int_t(x) - detail::highp_int_t(y));

Changing that line to this helps:

return genUType(detail::highp_int_t(1) << (detail::highp_int_t(32) + detail::highp_int_t(x) - detail::highp_int_t(y)));

I guess that normally that wouldn't be a big deal but I know of a few projects that use Clang solely for static code analysis and such flood of warnings certainly doesn't help.

[ghost]

I'm sorry, I didn't check the latest version from Git repo - this is fixed already. Please close that bug, sorry for your trouble.

[Heorhiy]

I use latest version but still have lots of warnings
/glm/core/func_integer.inl:54:65: warning: shift count >= width of type [-Wshift-count-overflow]
genUType Result = genUType(Value64 % (detail::highp_uint_t(1) << detail::highp_uint_t(32)));

Apple LLVM version 5.0 (clang-500.2.75) (based on LLVM 3.3svn)
Target: x86_64-apple-darwin12.5.0

[Groovounet]

This warning was actually showing implementation bugs.

This is fixed in GLM 0.9.5 branch.

Thanks for contributing!
Christophe