typo fix (strangely VC9 doesn't consider it an error)

[wxWidgets.git] / tests / thread / atomic.cpp

diff --git a/tests/thread/atomic.cpp b/tests/thread/atomic.cpp
index 976266331bbfb3b327e6292efd6e95545052102c..9f7b01032c045d026e10bbc335c3fa09dc15a7c4 100644 (file)
--- a/tests/thread/atomic.cpp
+++ b/tests/thread/atomic.cpp
@@ -27,6 +27,18 @@
 
 WX_DEFINE_ARRAY_PTR(wxThread *, wxArrayThread);
 
 
 WX_DEFINE_ARRAY_PTR(wxThread *, wxArrayThread);
 

+// ----------------------------------------------------------------------------
+// constants
+// ----------------------------------------------------------------------------
+
+// number of times to run the loops: the code takes too long to run if we use
+// the bigger value with generic atomic operations implementation
+#ifdef wxHAS_ATOMIC_OPS
+    static const wxInt32 ITERATIONS_NUM = 10000000;
+#else
+    static const wxInt32 ITERATIONS_NUM = 1000;
+#endif
+

 // ----------------------------------------------------------------------------
 // test class
 // ----------------------------------------------------------------------------
 // ----------------------------------------------------------------------------
 // test class
 // ----------------------------------------------------------------------------


@@ -36,8 +48,6 @@ class AtomicTestCase : public CppUnit::TestCase
 public:
     AtomicTestCase() { }
 
 public:
     AtomicTestCase() { }
 

-private:
-

     enum ETestType
     {
         IncAndDecMixed,
     enum ETestType
     {
         IncAndDecMixed,


@@ -45,6 +55,7 @@ private:
         DecOnly
     };
 
         DecOnly
     };
 

+private:

     class MyThread : public wxThread
     {
     public:
     class MyThread : public wxThread
     {
     public:


@@ -61,6 +72,7 @@ private:
 
     CPPUNIT_TEST_SUITE( AtomicTestCase );
         CPPUNIT_TEST( TestNoThread );
 
     CPPUNIT_TEST_SUITE( AtomicTestCase );
         CPPUNIT_TEST( TestNoThread );

+        CPPUNIT_TEST( TestDecReturn );

         CPPUNIT_TEST( TestTwoThreadsMix );
         CPPUNIT_TEST( TestTenThreadsMix );
         CPPUNIT_TEST( TestTwoThreadsSeparate );
         CPPUNIT_TEST( TestTwoThreadsMix );
         CPPUNIT_TEST( TestTenThreadsMix );
         CPPUNIT_TEST( TestTwoThreadsSeparate );


@@ -68,6 +80,7 @@ private:
     CPPUNIT_TEST_SUITE_END();
 
     void TestNoThread();
     CPPUNIT_TEST_SUITE_END();
 
     void TestNoThread();

+    void TestDecReturn();

     void TestTenThreadsMix() { TestWithThreads(10, IncAndDecMixed); }
     void TestTwoThreadsMix() { TestWithThreads(2, IncAndDecMixed); }
     void TestTenThreadsSeparate() { TestWithThreads(10, IncOnly); }
     void TestTenThreadsMix() { TestWithThreads(10, IncAndDecMixed); }
     void TestTwoThreadsMix() { TestWithThreads(2, IncAndDecMixed); }
     void TestTenThreadsSeparate() { TestWithThreads(10, IncOnly); }


@@ -85,16 +98,28 @@ CPPUNIT_TEST_SUITE_NAMED_REGISTRATION( AtomicTestCase, "AtomicTestCase" );
 
 void AtomicTestCase::TestNoThread()
 {
 
 void AtomicTestCase::TestNoThread()
 {

-    wxAtomicInt    int1=0, int2=0;
+    wxAtomicInt int1 = 0,
+                int2 = 0;

 
 

-    for (wxInt32 i=0; i<10000000; ++i)
+    for ( wxInt32 i = 0; i < ITERATIONS_NUM; ++i )

     {
         wxAtomicInc(int1);
         wxAtomicDec(int2);
     }
 
     {
         wxAtomicInc(int1);
         wxAtomicDec(int2);
     }
 

-    CPPUNIT_ASSERT( int1 == 10000000 );
-    CPPUNIT_ASSERT( int2 == -10000000 );
+    CPPUNIT_ASSERT( int1 == ITERATIONS_NUM );
+    CPPUNIT_ASSERT( int2 == -ITERATIONS_NUM );
+}
+
+void AtomicTestCase::TestDecReturn()
+{
+    wxAtomicInt i(0);
+    wxAtomicInc(i);
+    wxAtomicInc(i);
+    CPPUNIT_ASSERT( i == 2 );
+
+    CPPUNIT_ASSERT( wxAtomicDec(i) > 0 );
+    CPPUNIT_ASSERT( wxAtomicDec(i) == 0 );

 }
 
 void AtomicTestCase::TestWithThreads(int count, ETestType testType)
 }
 
 void AtomicTestCase::TestWithThreads(int count, ETestType testType)


@@ -148,27 +173,27 @@ void *AtomicTestCase::MyThread::Entry()
 {
     wxInt32 negativeValuesSeen = 0;
 
 {
     wxInt32 negativeValuesSeen = 0;
 

-    for (wxInt32 i=0; i<10000000; ++i)
+    for ( wxInt32 i = 0; i < ITERATIONS_NUM; ++i )

     {
     {

-        switch (m_testType)
+        switch ( m_testType )

         {
         {

-        case AtomicTestCase::IncAndDecMixed:
-            wxAtomicInc(m_operateOn);
-            wxAtomicDec(m_operateOn);
+            case AtomicTestCase::IncAndDecMixed:
+                wxAtomicInc(m_operateOn);
+                wxAtomicDec(m_operateOn);

 
 

-            if (m_operateOn < 0)
-                ++negativeValuesSeen;
-            break;
+                if (m_operateOn < 0)
+                    ++negativeValuesSeen;
+                break;

 
 

-        case AtomicTestCase::IncOnly:
-            wxAtomicInc(m_operateOn);
-            break;
+            case AtomicTestCase::IncOnly:
+                wxAtomicInc(m_operateOn);
+                break;

 
 

-        case AtomicTestCase::DecOnly:
-            wxAtomicDec(m_operateOn);
-            break;
+            case AtomicTestCase::DecOnly:
+                wxAtomicDec(m_operateOn);
+                break;

         }
     }
 
         }
     }
 

-    return (wxThread::ExitCode)negativeValuesSeen;
+    return wxUIntToPtr(negativeValuesSeen);

 }
 }