Changeset 126721 in webkit for trunk/Source/JavaScriptCore/runtime/Executable.cpp

Timestamp:

Aug 26, 2012, 8:57:02 PM (13 years ago)

Author:

[email protected]

Message:

Removed the NULL checks from visitChildren functions
​https://bugs.webkit.org/show_bug.cgi?id=95021

Reviewed by Oliver Hunt.

Source/JavaScriptCore:

As of ​http://trac.webkit.org/changeset/126624, all values are NULL-checked
during GC, so explicit NULL checks aren't needed anymore.

Source/WebCore:

As of ​http://trac.webkit.org/changeset/126624, all values are NULL-checked
during GC, so explicit NULL checks aren't needed anymore.

File:

• trunk/Source/JavaScriptCore/runtime/Executable.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/runtime/Executable.cpp

@@ -612 +612 @@
     ASSERT(thisObject->structure()->typeInfo().overridesVisitChildren());
     ScriptExecutable::visitChildren(thisObject, visitor);
-    if (thisObject->m_nameValue)
-        visitor.append(&thisObject->m_nameValue);
-    if (thisObject->m_symbolTable)
-        visitor.append(&thisObject->m_symbolTable);
+    visitor.append(&thisObject->m_nameValue);
+    visitor.append(&thisObject->m_symbolTable);
     if (thisObject->m_codeBlockForCall)
         thisObject->m_codeBlockForCall->visitAggregate(visitor);