Reviewers: Michael Starzinger,
Description:
Fix sequence point bug.
[email protected]
BUG=
Please review this at https://codereview.chromium.org/11308090/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files:
M src/runtime.cc
Index: src/runtime.cc
diff --git a/src/runtime.cc b/src/runtime.cc
index
b16c6855974a59660f07d854677a0fc7a7485beb..8fde642bcc313e717199b70a6835d5378018e8ec
100644
--- a/src/runtime.cc
+++ b/src/runtime.cc
@@ -1109,7 +1109,8 @@ static MaybeObject* GetOwnProperty(Isolate* isolate,
if (accessors == NULL) {
elms->set(WRITABLE_INDEX, heap->ToBoolean((attrs & READ_ONLY) == 0));
// GetProperty does access check.
- elms->set(VALUE_INDEX, *GetProperty(obj, name));
+ Handle<Object> value = GetProperty(obj, name);
+ elms->set(VALUE_INDEX, *value);
} else {
// Access checks are performed for both accessors separately.
// When they fail, the respective field is not set in the descriptor.
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
