Changeset 129827 in webkit for trunk/Source/JavaScriptCore/interpreter/CallFrame.h

Timestamp:

Sep 27, 2012, 5:50:34 PM (13 years ago)

Author:

[email protected]

Message:

Fixed CallFrameClosure::resetCallFrame() to use the valid
range of argument index values.
​https://bugs.webkit.org/show_bug.cgi?id=97836.

Reviewed by Gavin Barraclough.

• interpreter/CallFrame.h:

(ExecState):

• interpreter/CallFrameClosure.h:

(JSC::CallFrameClosure::resetCallFrame):

File:

• trunk/Source/JavaScriptCore/interpreter/CallFrame.h (modified) (1 diff)

trunk/Source/JavaScriptCore/interpreter/CallFrame.h

@@ -191 +191 @@
         static int argumentOffsetIncludingThis(int argument) { return s_thisArgumentOffset - argument; }
 
+        // In the following (argument() and setArgument()), the 'argument'
+        // parameter is the index of the arguments of the target function of
+        // this frame. The index starts at 0 for the first arg, 1 for the
+        // second, etc.
+        //
+        // The arguments (in this case) do not include the 'this' value.
+        // arguments(0) will not fetch the 'this' value. To get/set 'this',
+        // use thisValue() and setThisValue() below.
+
         JSValue argument(size_t argument)
         {