Changeset 2230 for trunk/testTweet

Timestamp:

Jan 17, 2013, 11:19:30 PM (7 years ago)

Author:

sam

Message:

base: fix a nasty double free issue in Array::Remove().

File:

• trunk/test/unit/array.cpp (modified) (2 diffs)

trunk/test/unit/array.cpp

@@ -18 +18 @@
 namespace lol
 {
+
+struct TrackedObj
+{
+    static int m_ctor, m_dtor;
+
+    TrackedObj() { m_ctor++; }
+    TrackedObj(TrackedObj const &) { m_ctor++; }
+    ~TrackedObj() { m_dtor++; }
+};
+
+int TrackedObj::m_ctor = 0;
+int TrackedObj::m_dtor = 0;
 
 LOLUNIT_FIXTURE(ArrayTest)
@@ -120 +132 @@
         LOLUNIT_ASSERT_EQUAL(b[3], 3);
     }
+
+    LOLUNIT_TEST(ElementCtorDtor)
+    {
+        /* Ensure array elements get created and destroyed the proper
+         * number of times. */
+        TrackedObj::m_ctor = 0;
+        TrackedObj::m_dtor = 0;
+        {
+            Array<TrackedObj> a;
+
+            a.Push(TrackedObj());
+        }
+        LOLUNIT_ASSERT_EQUAL(TrackedObj::m_ctor, TrackedObj::m_dtor);
+
+        TrackedObj::m_ctor = 0;
+        TrackedObj::m_dtor = 0;
+        {
+            Array<TrackedObj> a;
+
+            a.Resize(2);
+            a.Resize(4);
+            a.Resize(1);
+        }
+        LOLUNIT_ASSERT_EQUAL(TrackedObj::m_ctor, TrackedObj::m_dtor);
+    }
 };