Changes inspired by cppckeck

Author: robertosfield

include/vsg/maths/box.h

@@ -42,7 +42,7 @@ namespace vsg
 
         constexpr t_box& operator=(const t_box&) = default;
 
-        constexpr std::size_t size() const { return 6; }
+        constexpr static std::size_t size() { return 6; }
 
         value_type& operator[](std::size_t i) { return data()[i]; }
         value_type operator[](std::size_t i) const { return data()[i]; }

include/vsg/maths/mat2.h

@@ -59,9 +59,9 @@ namespace vsg
             value[1] = rhs[1];
         }
 
-        constexpr std::size_t size() const { return 4; }
-        constexpr std::size_t columns() const { return 2; }
-        constexpr std::size_t rows() const { return 2; }
+        constexpr static std::size_t size() { return 4; }
+        constexpr static std::size_t columns() { return 2; }
+        constexpr static std::size_t rows() { return 2; }
 
         column_type& operator[](std::size_t c) { return value[c]; }
         const column_type& operator[](std::size_t c) const { return value[c]; }

include/vsg/maths/mat3.h

@@ -66,9 +66,9 @@ namespace vsg
             value[2] = rhs[2];
         }
 
-        constexpr std::size_t size() const { return 9; }
-        constexpr std::size_t columns() const { return 3; }
-        constexpr std::size_t rows() const { return 3; }
+        constexpr static std::size_t size() { return 9; }
+        constexpr static std::size_t columns() { return 3; }
+        constexpr static std::size_t rows() { return 3; }
 
         column_type& operator[](std::size_t c) { return value[c]; }
         const column_type& operator[](std::size_t c) const { return value[c]; }

include/vsg/maths/mat4.h

@@ -75,9 +75,9 @@ namespace vsg
             value[3] = rhs[3];
         }
 
-        constexpr std::size_t size() const { return 16; }
-        constexpr std::size_t columns() const { return 4; }
-        constexpr std::size_t rows() const { return 4; }
+        constexpr static std::size_t size() { return 16; }
+        constexpr static std::size_t columns() { return 4; }
+        constexpr static std::size_t rows() { return 4; }
 
         column_type& operator[](std::size_t c) { return value[c]; }
         const column_type& operator[](std::size_t c) const { return value[c]; }

include/vsg/maths/plane.h

@@ -77,7 +77,7 @@ namespace vsg
         constexpr explicit t_plane(const t_vec4<T>& v) :
             value{v[0], v[1], v[2], v[3]} {}
 
-        constexpr std::size_t size() const { return 4; }
+        constexpr static std::size_t size() { return 4; }
 
         value_type& operator[](std::size_t i) { return value[i]; }
         value_type operator[](std::size_t i) const { return value[i]; }

include/vsg/maths/quat.h

@@ -65,7 +65,7 @@ namespace vsg
 
         constexpr t_quat& operator=(const t_quat&) = default;
 
-        constexpr std::size_t size() const { return 4; }
+        constexpr static std::size_t size() { return 4; }
 
         value_type& operator[](std::size_t i) { return value[i]; }
         value_type operator[](std::size_t i) const { return value[i]; }

include/vsg/maths/sphere.h

@@ -74,7 +74,7 @@ namespace vsg
         constexpr t_sphere(R sx, R sy, R sz, R sd) :
             value{sx, sy, sz, sd} {}
 
-        constexpr std::size_t size() const { return 4; }
+        constexpr static std::size_t size() { return 4; }
 
         value_type& operator[](std::size_t i) { return value[i]; }
         value_type operator[](std::size_t i) const { return value[i]; }

include/vsg/maths/vec2.h

@@ -68,7 +68,7 @@ namespace vsg
         constexpr explicit t_vec2(const t_vec2<R>& v) :
             value{static_cast<T>(v.x), static_cast<T>(v.y)} {}
 
-        constexpr std::size_t size() const { return 2; }
+        constexpr static std::size_t size() { return 2; }
 
         value_type& operator[](std::size_t i) { return value[i]; }
         value_type operator[](std::size_t i) const { return value[i]; }

include/vsg/maths/vec3.h

@@ -67,7 +67,7 @@ namespace vsg
         constexpr explicit t_vec3(const t_vec3<R>& v) :
             value{static_cast<T>(v.x), static_cast<T>(v.y), static_cast<T>(v.z)} {}
 
-        constexpr std::size_t size() const { return 3; }
+        constexpr static std::size_t size() { return 3; }
 
         value_type& operator[](std::size_t i) { return value[i]; }
         value_type operator[](std::size_t i) const { return value[i]; }

include/vsg/maths/vec4.h

@@ -78,7 +78,7 @@ namespace vsg
         constexpr t_vec4(const t_vec3<R>& v, value_type in_w) :
             value{static_cast<T>(v.x), static_cast<T>(v.y), static_cast<T>(v.z), in_w} {}
 
-        constexpr std::size_t size() const { return 4; }
+        constexpr static std::size_t size() { return 4; }
 
         value_type& operator[](std::size_t i) { return value[i]; }
         value_type operator[](std::size_t i) const { return value[i]; }