Changeset 54732 in webkit for trunk/JavaScriptCore

Timestamp:

Feb 12, 2010, 1:00:15 PM (15 years ago)

Author:

[email protected]

Message:

Removed an unnecessary data dependency from my last patch.

Reviewed by Darin Adler.

• runtime/SmallStrings.cpp:

(JSC::SmallStrings::markChildren): Since isAnyStringMarked being false
is a condition of entering the loop, we can just use '=' instead of '|='.

Location:

trunk/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• runtime/SmallStrings.cpp (modified) (1 diff)

trunk/JavaScriptCore/ChangeLog

@@ +1 @@
+2010-02-12  Geoffrey Garen  <[email protected]>
+
+        Reviewed by Darin Adler.
+
+        Removed an unnecessary data dependency from my last patch.
+
+        * runtime/SmallStrings.cpp:
+        (JSC::SmallStrings::markChildren): Since isAnyStringMarked being false
+        is a condition of entering the loop, we can just use '=' instead of '|='.
+
 2010-02-12  Janne Koskinen  <[email protected]>
 

trunk/JavaScriptCore/runtime/SmallStrings.cpp

@@ -84 +84 @@
     bool isAnyStringMarked = isMarked(m_emptyString);
     for (unsigned i = 0; i < numCharactersToStore && !isAnyStringMarked; ++i)
-        isAnyStringMarked |= isMarked(m_singleCharacterStrings[i]);
+        isAnyStringMarked = isMarked(m_singleCharacterStrings[i]);
     
     if (!isAnyStringMarked) {