Changeset 38101 in webkit for trunk/JavaScriptCore/wtf/MainThread.cpp

Timestamp:

Nov 4, 2008, 10:57:01 AM (17 years ago)

Author:

Darin Adler

Message:

2008-11-03 Darin Adler <Darin Adler>

Reviewed by Tim Hatcher.

• ​https://bugs.webkit.org/show_bug.cgi?id=22061 create script to check for exit-time destructors

• JavaScriptCore.exp: Changed to export functions rather than a global for the atomically initialized static mutex.

• JavaScriptCore.xcodeproj/project.pbxproj: Added a script phase that runs the check-for-exit-time-destructors script.

• wtf/MainThread.cpp: (WTF::mainThreadFunctionQueueMutex): Changed to leak an object rather than using an exit time destructor. (WTF::functionQueue): Ditto.
• wtf/unicode/icu/CollatorICU.cpp: (WTF::cachedCollatorMutex): Ditto.

• wtf/Threading.h: Changed other platforms to share the Windows approach where the mutex is internal and the functions are exported.
• wtf/ThreadingGtk.cpp: (WTF::lockAtomicallyInitializedStaticMutex): Ditto. (WTF::unlockAtomicallyInitializedStaticMutex): Ditto.
• wtf/ThreadingNone.cpp: (WTF::lockAtomicallyInitializedStaticMutex): Ditto. (WTF::unlockAtomicallyInitializedStaticMutex): Ditto.
• wtf/ThreadingPthreads.cpp: (WTF::threadMapMutex): Changed to leak an object rather than using an exit time destructor. (WTF::lockAtomicallyInitializedStaticMutex): Mutex change. (WTF::unlockAtomicallyInitializedStaticMutex): Ditto. (WTF::threadMap): Changed to leak an object rather than using an exit time destructor.
• wtf/ThreadingQt.cpp: (WTF::lockAtomicallyInitializedStaticMutex): Mutex change. (WTF::unlockAtomicallyInitializedStaticMutex): Ditto.
• wtf/ThreadingWin.cpp: (WTF::lockAtomicallyInitializedStaticMutex): Added an assertion.

File:

• trunk/JavaScriptCore/wtf/MainThread.cpp (modified) (2 diffs)

trunk/JavaScriptCore/wtf/MainThread.cpp

@@ -50 +50 @@
 typedef Vector<FunctionWithContext> FunctionQueue;
 
-static bool callbacksPaused; // This global varialble is only accessed from main thread.
+static bool callbacksPaused; // This global variable is only accessed from main thread.
 
 Mutex& mainThreadFunctionQueueMutex()
 {
-    static Mutex staticMutex;
+    static Mutex& staticMutex = *new Mutex;
     return staticMutex;
 }
@@ -60 +60 @@
 static FunctionQueue& functionQueue()
 {
-    static FunctionQueue staticFunctionQueue;
+    static FunctionQueue& staticFunctionQueue = *new FunctionQueue;
     return staticFunctionQueue;
 }