Corrections to copy assignment operators for Quaternion, and matrix c…

…lasses

morph/Matrix33.h

@@ -41,7 +41,7 @@ namespace morph {
         //! User-declared copy assignment constructor
         Matrix33<Flt>& operator= (Matrix33<Flt>& other)
         {
-            std::swap (mat, other.mat);
+            std::copy (other.mat.begin(), other.mat.end(), mat.begin());
             return *this;
         }
         //! Explicitly defaulted  move constructor

morph/Quaternion.h

@@ -50,10 +50,10 @@ namespace morph {
         //! User-declared copy assignment constructor
         Quaternion<Flt>& operator= (Quaternion<Flt>& other)
         {
-            std::swap (w, other.w);
-            std::swap (x, other.x);
-            std::swap (y, other.y);
-            std::swap (z, other.z);
+            w = other.w;
+            x = other.x;
+            y = other.y;
+            z = other.z;
             return *this;
         }
         //! Explicitly defaulted  move constructor

morph/TransformMatrix.h

@@ -51,7 +51,7 @@ namespace morph {
         //! User-declared copy assignment constructor
         TransformMatrix<Flt>& operator= (TransformMatrix<Flt>& other)
         {
-            std::swap (mat, other.mat);
+            std::copy (other.mat.begin(), other.mat.end(), mat.begin());
             return *this;
         }
         //! Explicitly defaulted  move constructor

tests/testTransformMatrix.cpp

@@ -37,7 +37,13 @@ int main()
             ++rtn;
         }
     }
-
+    tm2 = static_cast<morph::TransformMatrix<float>>(tm1);
+    std::cout << "After second assignment:\n" << tm2 << std::endl;
+    for (unsigned int i = 0; i<16; ++i) {
+        if (tm2.mat[i] != (float)i) {
+            ++rtn;
+        }
+    }
     // Test multiplication
     morph::TransformMatrix<float> mult1;
     setMatrixSequence (mult1);