https://codereview.chromium.org/1151503002/diff/1/src/runtime.js
File src/runtime.js (right):
https://codereview.chromium.org/1151503002/diff/1/src/runtime.js#newcode932
src/runtime.js:932: function SpreadIntoArray(target, iterable) {
On 2015/05/20 13:34:14, arv wrote:
On 2015/05/20 12:57:35, rossberg wrote:
> Nit: ConcatIterableToArray
It is not to array... it adds it to an existing array. This is from my
array
spread CL where we need to add the elements to an existing array.
Hence Concat...ToArray ;)
For the rest destructuring we could skip passing an array and return a
new
array. The only question is if we want to reuse this code or not?
It's good, as long as there is a simple way to make the array strong in
strong mode.
https://codereview.chromium.org/1151503002/
--
--
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.
