Improve subsystem manager's timing statistics

[simgear.git] / simgear / structure / SGAtomic.hxx

diff --git a/simgear/structure/SGAtomic.hxx b/simgear/structure/SGAtomic.hxx
index a6202a869132c4372be5bf50dd0d658eb90e7362..d4611d11c078c0e8c5ef3cbb6332d68047d7ffad 100644 (file)
--- a/simgear/structure/SGAtomic.hxx
+++ b/simgear/structure/SGAtomic.hxx
@@ -28,7 +28,7 @@
 #elif defined(__sgi) && defined(_COMPILER_VERSION) && (_COMPILER_VERSION>=730)
 // No need to include something. Is a Compiler API ...
 # define SGATOMIC_USE_MIPSPRO_BUILTINS
-#elif defined(WIN32)  && !defined ( __CYGWIN__ )
+#elif defined(_WIN32)
 # include <windows.h>
 # define SGATOMIC_USE_WIN32_INTERLOCKED
 #else
@@ -113,85 +113,32 @@ private:
   unsigned mValue;
 };
 
-// Value that can be atomically compared and swapped.
-class SGSwappable
-{
-public:
-    typedef unsigned long value_type;
-    SGSwappable(unsigned long value = 0) : mValue(value) {}
-    operator unsigned long() const
-    {
-#if defined(SGATOMIC_USE_GCC4_BUILTINS)
-        __sync_synchronize();
-        return mValue;
-#elif defined(SGATOMIC_USE_MIPOSPRO_BUILTINS)
-        __synchronize();
-        return mValue;
-#elif defined(SGATOMIC_USE_WIN32_INTERLOCKED)
-        return static_cast<long const volatile &>(mValue);
-#else
-        SGGuard<SGMutex> lock(mMutex);
-        return mValue;
-#endif
-    }
-
-    bool compareAndSwap(unsigned long oldVal, unsigned long newVal)
-    {
-#if defined(SGATOMIC_USE_GCC4_BUILTINS)
-        return __sync_bool_compare_and_swap(&mValue, oldVal, newVal);
-#elif defined(SGATOMIC_USE_MIPOSPRO_BUILTINS)
-        return __compare_and_swap(&mValue, oldVal, newVal);
-#elif defined(SGATOMIC_USE_WIN32_INTERLOCKED)
-        long previous
-            = InterlockedCompareExchange(reinterpret_cast<long volatile*>(&mValue),
-                                         (long)newVal,
-                                         (long)oldVal);
-        return previous == (long)oldVal;
-#else
-        SGGuard<SGMutex> lock(mMutex);
-        if (oldVal == mValue) {
-            mValue = newVal;
-            return true;
-        } else {
-            return false;
-        }
-#endif
-    }
-
-private:
-    SGSwappable(const SGAtomic&);
-    SGSwappable& operator=(const SGAtomic&);
-
-#if !defined(SGATOMIC_USE_GCC4_BUILTINS)        \
-    && !defined(SGATOMIC_USE_MIPOSPRO_BUILTINS) \
-    && !defined(SGATOMIC_USE_WIN32_INTERLOCKED)
-    mutable SGMutex mMutex;
-#endif
-#ifdef SGATOMIC_USE_WIN32_INTERLOCKED
-    __declspec(align(32))
-#endif
-    value_type mValue;
-
-};
-
 namespace simgear
 {
 // Typesafe wrapper around SGSwappable
 template <typename T>
-class Swappable : private SGSwappable
+class Swappable : private SGAtomic
 {
 public:
-    Swappable(const T& value) : SGSwappable(static_cast<value_type>(value))
+    Swappable(const T& value) : SGAtomic(static_cast<unsigned>(value))
     {
     }
     T operator() () const
     {
-        return static_cast<T>(SGSwappable::operator unsigned long ());
+        return static_cast<T>(SGAtomic::operator unsigned ());
+    }
+    Swappable& operator=(const Swappable& rhs)
+    {
+        for (unsigned oldval = unsigned(*this);
+             !compareAndExchange(oldval, unsigned(rhs));
+             oldval = unsigned(*this))
+            ;
+        return *this;
     }
     bool compareAndSwap(const T& oldVal, const T& newVal)
     {
-        return SGSwappable::compareAndSwap(static_cast<value_type>(oldVal),
-                                           static_cast<value_type>(newVal));
+        return SGAtomic::compareAndExchange(static_cast<unsigned>(oldVal),
+                                           static_cast<unsigned>(newVal));
     }
 };
 }