Status: Assigned
Owner: [email protected]
Labels: Type-Bug Priority-Medium Harmony

New issue 4308 by [email protected]: Lots of Array methods need to call ToLength, not ToUint32
https://code.google.com/p/v8/issues/detail?id=4308

These used to call ToUint32 in ES5, and were changed in ES6. If we don't think this is a good idea, let's bring it up at TC39. This only affects non-array Array-like things, as the "length" property of an array is always within Uint32 range.

I started making a list, but when it comes down to it, there's just no occurrence of $toLength in src/array.js at all, so every single one needs to be changed. Here's to get started:
- Array.prototype.indexOf
- Array.prototype.join
- Array.prototype.reverse
- Array.prototype.shift
- Array.prototype.toLocaleString
- Array.prototype.unshift


--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
--
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.

Reply via email to