http://codereview.chromium.org/6499013/diff/1/test/cctest/test-api.cc File test/cctest/test-api.cc (right):
http://codereview.chromium.org/6499013/diff/1/test/cctest/test-api.cc#newcode5640 test/cctest/test-api.cc:5640: v8::HandleScope scope1; On 2011/02/15 08:56:10, Mads Ager wrote:
No need for this since you have the handle scope above.
Done. http://codereview.chromium.org/6499013/diff/1/test/cctest/test-api.cc#newcode5648 test/cctest/test-api.cc:5648: ExpectTrue("Object.keys(other).indexOf('blocked_prop')==-1"); On 2011/02/15 08:56:10, Mads Ager wrote:
Spaces around '=='.
Done. http://codereview.chromium.org/6499013/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
