<div dir="ltr">The spec for Object.assign, <a href="http://people.mozilla.org/~jorendorff/es6-draft.html#sec-object.assign">http://people.mozilla.org/~jorendorff/es6-draft.html#sec-object.assign</a>, seems to have some issues.<br>

<br><i>gotAllNames</i> is never read.<br><br>What is the intention regarding exceptions? Is the intention to continue assigning properties after an exception is thrown. Why is that the desired behavior?<br><br>On Tue, Oct 22, 2013 at 6:41 AM, André Bargull <<a href="mailto:andre.bargull@udo.edu">andre.bargull@udo.edu</a>> wrote:<br>

...<br><div>> A more conformant implementation should be this one:<br>><br>> function assign(target, source) {<br>>   function ToObject(o) {<br>>     if (o == null) {<br>>       throw new TypeError();<br>

>     }<br>>     return Object(o);<br>>   }<br>>   var to = ToObject(target);<br>>   var from = ToObject(source);<br>>   var keys = $getOwnPropertyNames(from);<br>>   var pendingException = null;<br>
>   for (var i = 0, length = keys.length; i < length; ++i) {<br>
>     var nextKey = keys[i];<br>>     try {<br>>       var desc = $getOwnPropertyDescriptor(from, nextKey);<br>>       if (desc !== undefined && desc.enumerable) {<br>>         to[nextKey] = from[nextKey];<br>

>       }<br>>     } catch (e) {<br>>       if (pendingException !== null) {<br>>         pendingException = e;<br>>       }<br>>     }<br>>   }<br>>   if (pendingException !== null) {<br>>     throw pendingException;<br>

>   }<br>>   return to;<br>> }<br>><br>><br>> - André<br>><br>> _______________________________________________<br>> es-discuss mailing list<br>> <a href="mailto:es-discuss@mozilla.org">es-discuss@mozilla.org</a><br>

> <a href="https://mail.mozilla.org/listinfo/es-discuss">https://mail.mozilla.org/listinfo/es-discuss</a><br><br><br>-- <br>erik</div></div>