<div dir="ltr">2014-11-12 20:53 GMT+01:00 Rick Waldron <span dir="ltr"><<a href="mailto:waldron.rick@gmail.com" target="_blank">waldron.rick@gmail.com</a>></span>:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><div class="h5"><div><br></div></div></div><div>Done <a href="https://github.com/tc39/agendas/blob/master/2014/11.md" target="_blank">https://github.com/tc39/agendas/blob/master/2014/11.md</a></div></div></div></div>
</blockquote></div><br></div><div class="gmail_extra">Thanks. A related point: my harmony-reflect shim also patches Array.prototype.concat such that it recognizes proxies-for-arrays and will splice their elements (rather than adding the proxy as a single element to the target array).</div><div class="gmail_extra"><br></div><div class="gmail_extra">In ES6, this may not be necessary since a proxy could implement @@isConcatSpreadable (see <<a href="http://people.mozilla.org/~jorendorff/es6-draft.html#sec-array.prototype.concat">http://people.mozilla.org/~jorendorff/es6-draft.html#sec-array.prototype.concat</a>> step 9.b)</div><div class="gmail_extra"><br></div><div class="gmail_extra">However, looking at IsConcatSpreadable <<a href="http://people.mozilla.org/~jorendorff/es6-draft.html#sec-isconcatspreadable">http://people.mozilla.org/~jorendorff/es6-draft.html#sec-isconcatspreadable</a>> step 5, you'll notice it explicitly tests for Arrays. *If* we decide that Array.isArray should return `true` for proxies-for-arrays, *then* I think we should relax this step 5 to also return `true` for proxies-for-arrays, on the grounds of consistency and principle-of-least-surprise.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Cheers,</div><div class="gmail_extra">Tom</div></div>