Reviewed by darin.
authorkjk <kjk@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 5 Apr 2007 22:33:48 +0000 (22:33 +0000)
committerkjk <kjk@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 5 Apr 2007 22:33:48 +0000 (22:33 +0000)
        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):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20733 268f45cc-cd09-0410-ab3c-d52691b4dbfc

JavaScriptCore/ChangeLog
JavaScriptCore/kjs/collector.cpp

index 6c766d1..d1285a2 100644 (file)
@@ -1,3 +1,14 @@
+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>
 
         Reviewed by Adam.
index f9b4a75..a5c7806 100644 (file)
@@ -744,8 +744,10 @@ bool Collector::collect()
   markStackObjectsConservatively();
   markProtectedObjects();
   List::markProtectedLists();
+#if USE(MULTIPLE_THREADS)
   if (!currentThreadIsMainThread)
     markMainThreadOnlyObjects();
+#endif
 
 #ifndef NDEBUG
   fastMallocAllow();