LGTM

http://codereview.chromium.org/141031/diff/1/2
File test/mjsunit/regress/regress-386.js (right):

http://codereview.chromium.org/141031/diff/1/2#newcode39
Line 39: c={ c1 : false, c2 : false, c3 : false, c4 : false, c5 : false
};
We can do something simpler than this.  We hit your new assert with
something like:

function A() {
   for (var i = 0; i < 13; i++) {
     this['a' + i] = i;
   }
   this.i = function(){};
};

new A();
new A();

http://codereview.chromium.org/141031

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

Reply via email to