Changeset 20733 in webkit

Timestamp:

Apr 5, 2007 3:33:48 PM (17 years ago)

Author:

kjk

Message:

Reviewed by darin.

Coverity fix: in single-threaded case currentThreadIsMainThread is always true
so the code in if (!currentThreadIsMainThread) cannot possibly be reached
and Coverity complains about dead code.

• kjs/collector.cpp: (KJS::Collector::collect):

Location:

trunk/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• kjs/collector.cpp (modified) (1 diff)

trunk/JavaScriptCore/ChangeLog

@@ +1 @@
+2007-04-05  Krzysztof Kowalczyk  <kkowalczyk@gmail.com>
+
+        Reviewed by darin.
+
+        Coverity fix: in single-threaded case currentThreadIsMainThread is always true
+        so the code in if (!currentThreadIsMainThread) cannot possibly be reached
+        and Coverity complains about dead code.
+
+        * kjs/collector.cpp:
+        (KJS::Collector::collect):
+
 2007-04-03  Kevin McCullough  <kmccullough@apple.com>
 

trunk/JavaScriptCore/kjs/collector.cpp

@@ -745 +745 @@
   markProtectedObjects();
   List::markProtectedLists();
+#if USE(MULTIPLE_THREADS)
   if (!currentThreadIsMainThread)
     markMainThreadOnlyObjects();
+#endif
 
 #ifndef NDEBUG