Fixed both locations with this issue, added comment and more tests.
http://codereview.chromium.org/6335010/diff/1/src/parser.cc
File src/parser.cc (right):
http://codereview.chromium.org/6335010/diff/1/src/parser.cc#newcode3077
src/parser.cc:3077: ASSERT(!name->AsArrayIndex(&hash));
On 2011/01/24 07:57:01, Lasse Reichstein wrote:
Damnit. I fixed one such place already, but it seems there are more.
It's good
that you found it :)
It's NOT safe. The ASSERT gets hit by, e.g.:
var y = {get "1234"() {return 42; }};
Great catch. I fixed both locations and added more tests.
http://codereview.chromium.org/6335010/
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
