https://codereview.chromium.org/527963002/diff/1/src/ia32/full-codegen-ia32.cc
File src/ia32/full-codegen-ia32.cc (right):

https://codereview.chromium.org/527963002/diff/1/src/ia32/full-codegen-ia32.cc#newcode1299
src/ia32/full-codegen-ia32.cc:1299: __
push(Immediate(isolate()->factory()->empty_string()));
Can we do a better error message than an empty string?

https://codereview.chromium.org/527963002/diff/20001/test/mjsunit/harmony/super.js
File test/mjsunit/harmony/super.js (right):

https://codereview.chromium.org/527963002/diff/20001/test/mjsunit/harmony/super.js#newcode87
test/mjsunit/harmony/super.js:87: }.toMethod(Derived.prototype);
Still wrong indentation. function bodies indent 2 spaces, even when used
as part of another expression/statement.

Derived.prototype.testGetter = function() {
  return super.x;
}.toMethod(Derived.prototype);

https://codereview.chromium.org/527963002/

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to