Comments addressed
https://codereview.chromium.org/1130623004/diff/80001/src/pattern-rewriter.cc
File src/pattern-rewriter.cc (right):
https://codereview.chromium.org/1130623004/diff/80001/src/pattern-rewriter.cc#newcode222
src/pattern-rewriter.cc:222: // TODO(dslomov): computed property names.
On 2015/05/11 14:25:16, arv wrote:
Also:
TODO(dslomov): Add support for initializer
let {x: y = e1} = e2;
=>
t1 = e2;
let y = (t2 = t1.x) === undefined ? e1 : t2;
I think this is covered by VisitAssignment
https://codereview.chromium.org/1130623004/diff/80001/src/pattern-rewriter.cc#newcode255
src/pattern-rewriter.cc:255: NOT_A_PATTERN(Property)
On 2015/05/11 14:25:16, arv wrote:
I think this one still belongs with the implement ones above to make
sure we
remember to handle it.
I'd rather not (it really does not matter at this point)
https://codereview.chromium.org/1130623004/diff/80001/src/pattern-rewriter.cc#newcode260
src/pattern-rewriter.cc:260: NOT_A_PATTERN(BreakStatement)
On 2015/05/11 14:25:16, arv wrote:
Sort these...
Done.
https://codereview.chromium.org/1130623004/diff/80001/src/pattern-rewriter.cc#newcode295
src/pattern-rewriter.cc:295: }
On 2015/05/11 14:25:16, arv wrote:
#undef NOT_A_PATTERN
Done.
https://codereview.chromium.org/1130623004/diff/80001/test/mjsunit/harmony/destructuring.js
File test/mjsunit/harmony/destructuring.js (right):
https://codereview.chromium.org/1130623004/diff/80001/test/mjsunit/harmony/destructuring.js#newcode12
test/mjsunit/harmony/destructuring.js:12: var {z} = { z : 3 };
On 2015/05/11 14:25:16, arv wrote:
Maybe add let and const tests too and add TDZ check.
let {x, y} = {x: 1, y};
^
TDZ
This CL is just a rudimentary implementation of destructuring, for
proof-of-concept. I'll expand on implemenattion and tests in subsequent
CLs
https://codereview.chromium.org/1130623004/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
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 v8-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
