Browse Source

Changed how storage alignment is created to eliminate warnings on msvc.

Aaron 9 years ago
parent
commit
e57615b6bb
1 changed files with 14 additions and 23 deletions
  1. 14 23
      glm/detail/type_vec.hpp

+ 14 - 23
glm/detail/type_vec.hpp

@@ -9,22 +9,6 @@
 namespace glm{
 namespace glm{
 namespace detail
 namespace detail
 {
 {
-#if GLM_COMPILER & GLM_COMPILER_VC
-#pragma warning(push)
-#pragma warning(disable : 4324) // structure was padded due to alignment specifier
-#endif
-	template<std::size_t N> struct aligned {};
-	template<> GLM_ALIGNED_STRUCT(1) aligned<1>{};
-	template<> GLM_ALIGNED_STRUCT(2) aligned<2>{};
-	template<> GLM_ALIGNED_STRUCT(4) aligned<4>{};
-	template<> GLM_ALIGNED_STRUCT(8) aligned<8>{};
-	template<> GLM_ALIGNED_STRUCT(16) aligned<16>{};
-	template<> GLM_ALIGNED_STRUCT(32) aligned<32>{};
-	template<> GLM_ALIGNED_STRUCT(64) aligned<64>{};
-#if GLM_COMPILER & GLM_COMPILER_VC
-#pragma warning(pop)
-#endif
-
 	template <typename T, std::size_t size, bool aligned>
 	template <typename T, std::size_t size, bool aligned>
 	struct storage
 	struct storage
 	{
 	{
@@ -33,15 +17,22 @@ namespace detail
 		} type;
 		} type;
 	};
 	};
 
 
-	template <typename T, std::size_t size>
-	struct storage<T, size, true>
-	{
-		struct type : aligned<size>
-		{
-			uint8 data[size];
+	#define GLM_ALIGNED_STORAGE_TYPE_STRUCT(x) \
+		template <typename T> \
+		struct storage<T, x, true> { \
+			GLM_ALIGNED_STRUCT(x) type { \
+				uint8_t data[x]; \
+			}; \
 		};
 		};
-	};
 
 
+	GLM_ALIGNED_STORAGE_TYPE_STRUCT(1)
+	GLM_ALIGNED_STORAGE_TYPE_STRUCT(2)
+	GLM_ALIGNED_STORAGE_TYPE_STRUCT(4)
+	GLM_ALIGNED_STORAGE_TYPE_STRUCT(8)
+	GLM_ALIGNED_STORAGE_TYPE_STRUCT(16)
+	GLM_ALIGNED_STORAGE_TYPE_STRUCT(32)
+	GLM_ALIGNED_STORAGE_TYPE_STRUCT(64)
+		
 #	if GLM_ARCH & GLM_ARCH_SSE2_BIT
 #	if GLM_ARCH & GLM_ARCH_SSE2_BIT
 		template <>
 		template <>
 		struct storage<float, 16, true>
 		struct storage<float, 16, true>