Changeset 35533 in webkit for trunk/JavaScriptCore/VM/CodeBlock.cpp

Timestamp:

Aug 3, 2008, 2:58:21 AM (17 years ago)

Author:

[email protected]

Message:

Bug 19359: JavaScriptCore behaves differently from FF2/3 and IE when handling context in catch statement
<​https://bugs.webkit.org/show_bug.cgi?id=19359>

Reviewed by Cameron Zwarich

Make our catch behave like Firefox and IE, we do this by using a StaticScopeObject
instead of a generic JSObject for the scope node. We still don't make use of the
fact that we have a static scope inside the catch block, so the internal performance
of the catch block is not improved, even though technically it would be possible to
do so.

File:

• trunk/JavaScriptCore/VM/CodeBlock.cpp (modified) (1 diff)

trunk/JavaScriptCore/VM/CodeBlock.cpp

@@ -634 +634 @@
             break;
         }
+        case op_push_new_scope: {
+            int r0 = (++it)->u.operand;
+            int id0 = (++it)->u.operand;
+            int r1 = (++it)->u.operand;
+            printf("[%4d] push_new_scope \t%s, %s, %s\n", location, registerName(r0).c_str(), idName(id0, identifiers[id0]).c_str(), registerName(r1).c_str());
+            break;
+        }
         case op_jmp_scopes: {
             int scopeDelta = (++it)->u.operand;