Changeset 37040 in webkit for trunk/JavaScriptCore

Timestamp:

Sep 28, 2008, 11:56:50 AM (17 years ago)

Author:

Simon Hausmann

Message:

2008-09-28 Simon Hausmann <Simon Hausmann>

Reviewed by David Hyatt.

In Qt's initializeThreading re-use an existing thread identifier for the main
thread if it exists.

currentThread() implicitly creates new identifiers and it could be that
it is called before initializeThreading().

Location:

trunk/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• wtf/ThreadingQt.cpp (modified) (1 diff)

trunk/JavaScriptCore/ChangeLog

@@ +1 @@
+2008-09-28  Simon Hausmann  <[email protected]>
+
+        Reviewed by David Hyatt.
+
+        In Qt's initializeThreading re-use an existing thread identifier for the main
+        thread if it exists.
+
+        currentThread() implicitly creates new identifiers and it could be that
+        it is called before initializeThreading().
+
+        * wtf/ThreadingQt.cpp:
+        (WTF::initializeThreading):
+
 2008-09-27  Keishi Hattori  <[email protected]>
 

trunk/JavaScriptCore/wtf/ThreadingQt.cpp

@@ -128 +128 @@
         wtf_random_init();
         QThread* mainThread = QCoreApplication::instance()->thread();
-        mainThreadIdentifier = establishIdentifierForThread(mainThread);
+        mainThreadIdentifier = identifierByQthreadHandle(mainThread);
+        if (!mainThreadIdentifier)
+            mainThreadIdentifier = establishIdentifierForThread(mainThread);
         initializeMainThread();
     }