No, I was confused. I was just failing w/ unnamed classes but I fixed that.
Now I still think that this makes sense as an step forward; we don't yet
produce
errors for methods inside object literals inside computed property names
(bug
filed) but the overall state is saner (stronger?) after this CL.
https://codereview.chromium.org/968263002/diff/200001/test/mjsunit/strong/declaration-after-use.js
File test/mjsunit/strong/declaration-after-use.js (right):
https://codereview.chromium.org/968263002/diff/200001/test/mjsunit/strong/declaration-after-use.js#newcode146
test/mjsunit/strong/declaration-after-use.js:146: ReferenceError);
On 2015/03/06 14:48:01, Dmitry Lomov (chromium) wrote:
Add a test that tests the wrong behavior:
class C {
[ { m() { ... C ... } }; "x"] { .... }
}
with a bug reference
Done.
https://codereview.chromium.org/968263002/diff/200001/test/mjsunit/strong/declaration-after-use.js#newcode218
test/mjsunit/strong/declaration-after-use.js:218: let C8 = class C9 {
static method() { C9; } }; new C8();
On 2015/03/06 14:48:01, Dmitry Lomov (chromium) wrote:
Add a test for inner class referencing outer class:
class COuter {
m() {
class CInner {
n() {
COuter();
}
}
}
}
Also the same for object literal method
Done (not sure what you meant w/ object literal method here, did you
mean object literal method inside a computed property name? ptal if the
test I added matches w/ what you meant.
https://codereview.chromium.org/968263002/
--
--
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.
