http://codereview.chromium.org/6450003/diff/1/src/runtime.cc
File src/runtime.cc (right):

http://codereview.chromium.org/6450003/diff/1/src/runtime.cc#newcode881
src/runtime.cc:881: static MaybeObject*
Runtime_PreventExtensions(Arguments args) {
Do you need it here too?

http://codereview.chromium.org/6450003/diff/1/src/runtime.cc#newcode892
src/runtime.cc:892: if (proto->IsNull()) return obj;
Return false? Otherwise this will be interpreted as true.

http://codereview.chromium.org/6450003/diff/1/src/runtime.cc#newcode896
src/runtime.cc:896: return obj->map()->is_extensible() ?
Heap::true_value()
Remove extra space.

http://codereview.chromium.org/6450003/

--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev

Reply via email to