illegal storage class in quaternion.hpp

[Vectrobe]

as the title suggests, if said file is included in any way GLM will not compile, couple examples being glm\ext.hpp and glm\gtc\type_ptr.hpp, either files have to be modified to not include the header

error C2071: 'glm::tquat<T,P>::operator glm::tmat3x3<T,P>' : illegal storage class glm\gtc\quaternion.hpp 116
error C2071: 'glm::tquat<T,P>::operator glm::tmat4x4<T,P>' : illegal storage class glm\gtc\quaternion.hpp 117

[Groovounet]

With wish compile do you have thus issue?

[Vectrobe]

VS110 C++11 x86-64

[plasmacel]

Explicit conversion operators are an unsupported C++11 feature in MSVC11 (Visual Studio 2012).
See http://msdn.microsoft.com/en-us/library/hh567368.aspx

Sorry at the time of the commit I didn't try it with previous MSVC versions. However it is a core C++11 feature, it seems Microsoft only adapted it in Visual Studio 2013.

After a short research: it is supported by and above Visual C++ 2013 (MSVC12), GCC 4.5, Clang 3.0 and Intel C++ Composer XE 13.0.

We should release something like a 0.9.6.1 hotfix and enable this feature with some macro predefinition like GLM_FORCE_CPP11 for the present.

[Groovounet]

Thanks for the research!

I have a GLM_HAS_{CPP_FEATURE} mechanism that I am going to us for this feature to resolve the problem for GLM 0.9.6.1.

Thanks for contributing,
Christophe

[Vectrobe]

general rule of thumb is to not use experimental language components without at least researching it's support for compatibility, if you want or need to have such features it's a good idea to have them in a second branch of which you could merge to the main when it has adequate support.