Reviewers: Mads Ager,

Description:
Added mutation of locals to debugger.

The locals can now be mutated in the debugger. After a debugger
evaluation the content of the scope created for the evaluation is
written back to where it was originally picked from (context extension,
context, stack locals, stack parameters and possibly adaptor frame stack
parameters).

Fixed an issue of evaluating arguments through arguments[n] in the
debugger in some senarios. If n was larger than the functions formal
parameters count and the arguments object was not used by the function
arguments[n] used to resolve to undefined, as the precense of an
arguments adaptor frame was not taken into account. Now all the passed
parameters are added to the arguments object which is sometimes created
for the evaluation function. If there is an arguments adaptor frame the
arguments are picked from there.

BUG=212
TEST=test\mjsunit\debug-evaluate.js


Please review this at http://codereview.chromium.org/125115

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
   M     src/frames.h
   M     src/frames.cc
   M     src/runtime.cc
   M     test/mjsunit/debug-evaluate.js



--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to