Human gcmole driveby :-)

http://codereview.chromium.org/7247013/diff/1/src/objects.cc
File src/objects.cc (right):

http://codereview.chromium.org/7247013/diff/1/src/objects.cc#newcode8445
src/objects.cc:8445: SetFastElementsCapacityAndLength(elms_length,
length);
Shouldn't value be in an handle here, SetFastElementsCapacityAndLength
can (actually, always do) allocation. In addition, we return value
several places below where we could potentially have had allocation in
between.

http://codereview.chromium.org/7247013/

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

Reply via email to