<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Jun 17, 2014 at 11:46 AM, Andrea Giammarchi <span dir="ltr"><<a href="mailto:andrea.giammarchi@gmail.com" target="_blank">andrea.giammarchi@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">FWIW I think this is a non issue and "how many" should not be relevant.<div><br><div>If developers understand the issue, the fix is straight forward.</div>

</div></div></blockquote><div><br></div><div>I wish to live in your world of unicorns and magic for in that world I AM A WIZARD.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div dir="ltr"><div><div><div>instead of doing this</div>
<div><br></div><div>```javascript</div><div><font face="courier new, monospace">if (obj.entries) {</font></div><div><font face="courier new, monospace">  // do stuff with entries</font></div><div><font face="courier new, monospace">} else if(Array.isArray(obj)) {</font></div>


<div><font face="courier new, monospace">  // do stuff with obj</font></div><div><font face="courier new, monospace">}</font></div><div>```<br></div><div><br></div><div>or this</div><div><br></div><div><font face="courier new, monospace">`var entries = obj.entries || obj;`</font></div>


<div><br></div><div>we should just advocate this:</div><div><br></div><div>```javascript</div><div><font face="courier new, monospace">if (Array.isArray(obj)) {</font></div><div><font face="courier new, monospace">  // do stuff with obj</font></div>


<div><font face="courier new, monospace">} else if(obj.entries) {</font></div><div><font face="courier new, monospace">  // do stuff with entry property</font></div><div><font face="courier new, monospace">}</font></div>

<div>
```<br></div><div><br></div><div>or using `<font face="courier new, monospace">instanceof</font>` when the realm is not a problem (if performance is a concern and <font face="courier new, monospace">Array.isArray</font> is the bottleneck which is rarely the case)</div>


<div><br></div><div>This also seems to be an API only related problem where both `<font face="courier new, monospace">Array`</font> and `<font face="courier new, monospace">Object`</font> could be passed as parameter but yeah, objects with properties named as `<font face="courier new, monospace">list</font>`, `<font face="courier new, monospace">items</font>`, or `<font face="courier new, monospace">entries</font>` are quite common but I personally prefer a future proof approach/small refactoring than a stopper for new specs.<br>


</div><div><br></div><div>my 2 cents</div><div><br></div></div></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jun 17, 2014 at 11:18 AM, Alex Russell <span dir="ltr"><<a href="mailto:slightlyoff@google.com" target="_blank">slightlyoff@google.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Right. Would love to know the size/traffic of the "number of sites" affected.</div><div>
<div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jun 17, 2014 at 10:45 AM, Rick Waldron <span dir="ltr"><<a href="mailto:waldron.rick@gmail.com" target="_blank">waldron.rick@gmail.com</a>></span> wrote:<br>




<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote"><div>On Mon, Jun 16, 2014 at 11:11 PM, Brendan Eich <span dir="ltr"><<a href="mailto:brendan@mozilla.org" target="_blank">brendan@mozilla.org</a>></span> wrote:<br>






<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Would .items fare better, I wonder.<br></blockquote><div><br></div></div><div>Or outreach to sites the break?</div><span><font color="#888888"><div>




<br>
</div>
<div>Rick</div></font></span><div><div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
/be<br>
<br>
Oliver Hunt wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
It turns out there are a number of sites (such as <a href="http://mobile.twitter.com" target="_blank">mobile.twitter.com</a> <<a href="http://mobile.twitter.com" target="_blank">http://mobile.twitter.com</a>>) that are property detecting .entries on objects, and that means that they're breaking when Array.prototype.entries is provided.<div>






<br>
<br>
We're removing it from JSC now, until we can find a way to expose it without causing site breakage, although this does seem like fixing it would require bringing back the awful "pretend that you're undefined" horror.<br>







<br>
--Oliver<br>
<br></div><div>
______________________________<u></u>_________________<br>
es-discuss mailing list<br>
<a href="mailto:es-discuss@mozilla.org" target="_blank">es-discuss@mozilla.org</a><br>
<a href="https://mail.mozilla.org/listinfo/es-discuss" target="_blank">https://mail.mozilla.org/<u></u>listinfo/es-discuss</a><br>
</div></blockquote><div><div>
______________________________<u></u>_________________<br>
es-discuss mailing list<br>
<a href="mailto:es-discuss@mozilla.org" target="_blank">es-discuss@mozilla.org</a><br>
<a href="https://mail.mozilla.org/listinfo/es-discuss" target="_blank">https://mail.mozilla.org/<u></u>listinfo/es-discuss</a><br>
</div></div></blockquote></div></div></div><br></div></div>
<br>_______________________________________________<br>
es-discuss mailing list<br>
<a href="mailto:es-discuss@mozilla.org" target="_blank">es-discuss@mozilla.org</a><br>
<a href="https://mail.mozilla.org/listinfo/es-discuss" target="_blank">https://mail.mozilla.org/listinfo/es-discuss</a><br>
<br></blockquote></div><br></div>
</div></div><br>_______________________________________________<br>
es-discuss mailing list<br>
<a href="mailto:es-discuss@mozilla.org" target="_blank">es-discuss@mozilla.org</a><br>
<a href="https://mail.mozilla.org/listinfo/es-discuss" target="_blank">https://mail.mozilla.org/listinfo/es-discuss</a><br>
<br></blockquote></div><br></div>
</div></div></blockquote></div><br></div></div>