Added new patch set. Landed.
http://codereview.chromium.org/8390047/diff/1/test/cctest/test-dictionary.cc File test/cctest/test-dictionary.cc (right): http://codereview.chromium.org/8390047/diff/1/test/cctest/test-dictionary.cc#newcode79 test/cctest/test-dictionary.cc:79: CHECK_NE(key->GetIdentityHash(OMIT_CREATION), HEAP->undefined_value()); On 2011/10/26 12:19:48, rossberg wrote:
Why not check for isSmi?
Done. http://codereview.chromium.org/8390047/diff/1/test/cctest/test-dictionary.cc#newcode87 test/cctest/test-dictionary.cc:87: CHECK_NE(key->GetIdentityHash(OMIT_CREATION), HEAP->undefined_value()); On 2011/10/26 12:19:48, rossberg wrote:
Same here.
Done. http://codereview.chromium.org/8390047/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
