From d5f315035024b5db3850e51bf1ca5f46f1ffa1a8 Mon Sep 17 00:00:00 2001
From: Pavel Rojtberg <rojtberg@gmail.com>
Date: Fri, 2 Feb 2024 19:30:30 +0100
Subject: [PATCH] Main: SimpleSpline - make coeffs static and simplify
 constructor

---
 OgreMain/include/OgreSimpleSpline.h |  6 ------
 OgreMain/src/OgreSimpleSpline.cpp   | 32 ++++++++++-------------------
 2 files changed, 11 insertions(+), 27 deletions(-)

diff --git a/OgreMain/include/OgreSimpleSpline.h b/OgreMain/include/OgreSimpleSpline.h
index 394327017d1..12e8a275b60 100644
--- a/OgreMain/include/OgreSimpleSpline.h
+++ b/OgreMain/include/OgreSimpleSpline.h
@@ -119,12 +119,6 @@ namespace Ogre {
 
         std::vector<Vector3> mPoints;
         std::vector<Vector3> mTangents;
-
-        /// Matrix of coefficients 
-        Matrix4 mCoeffs;
-
-
-
     };
 
     /** @} */
diff --git a/OgreMain/src/OgreSimpleSpline.cpp b/OgreMain/src/OgreSimpleSpline.cpp
index efecf315f36..ca606b79963 100644
--- a/OgreMain/src/OgreSimpleSpline.cpp
+++ b/OgreMain/src/OgreSimpleSpline.cpp
@@ -32,25 +32,6 @@ namespace Ogre {
     //---------------------------------------------------------------------
     SimpleSpline::SimpleSpline()
     {
-        // Set up matrix
-        // Hermite polynomial
-        mCoeffs[0][0] = 2;
-        mCoeffs[0][1] = -2;
-        mCoeffs[0][2] = 1;
-        mCoeffs[0][3] = 1;
-        mCoeffs[1][0] = -3;
-        mCoeffs[1][1] = 3;
-        mCoeffs[1][2] = -2;
-        mCoeffs[1][3] = -1;
-        mCoeffs[2][0] = 0;
-        mCoeffs[2][1] = 0;
-        mCoeffs[2][2] = 1;
-        mCoeffs[2][3] = 0;
-        mCoeffs[3][0] = 1;
-        mCoeffs[3][1] = 0;
-        mCoeffs[3][2] = 0;
-        mCoeffs[3][3] = 0;
-
         mAutoCalc = true;
     }
     //---------------------------------------------------------------------
@@ -108,6 +89,15 @@ namespace Ogre {
             return mPoints[fromIndex + 1];
         }
 
+        // Set up matrix
+        // Hermite polynomial
+        static Matrix4 hCoeffs = {
+             2, -2,  1,  1,
+            -3,  3, -2, -1,
+             0,  0,  1,  0,
+             1,  0,  0,  0
+        };
+
         // Real interpolation
         // Form a vector of powers of t
         Real t2, t3;
@@ -116,7 +106,7 @@ namespace Ogre {
         Vector4 powers(t3, t2, t, 1);
 
 
-        // Algorithm is ret = powers * mCoeffs * Matrix4(point1, point2, tangent1, tangent2)
+        // Algorithm is ret = powers * hCoeffs * Matrix4(point1, point2, tangent1, tangent2)
         const Vector3& point1 = mPoints[fromIndex];
         const Vector3& point2 = mPoints[fromIndex+1];
         const Vector3& tan1 = mTangents[fromIndex];
@@ -140,7 +130,7 @@ namespace Ogre {
         pt[3][2] = tan2.z;
         pt[3][3] = 1.0f;
 
-        Vector4 ret = powers * mCoeffs * pt;
+        Vector4 ret = powers * hCoeffs * pt;
 
 
         return Vector3(ret.x, ret.y, ret.z);