Assigning ComponentWrapper should always mean copying components

sideeffects · Oct 30, 2015 · 12e31c2 · 12e31c2
1 parent d114c3e
commit 12e31c2
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/types.h b/types.h
@@ -122,11 +122,9 @@ class ComponentWrapper
 
         Type &operator =(const Type &o)
         {
-            myArray = o.myArray;
-            myIndex = o.myIndex;
-            myComponent = o.myComponent;
-
-            assert(myComponent == StartComponent);
+            // Assignment should always mean copying components
+            assert(&myArray != &o.myArray);
+            return this->operator=<Type>(o);
         }
 
         template<typename O>