06:31 | <Jesse> | oops, yes, I meant June, sorry |
20:45 | <shu> | one unfortunate about grouping being static methods is that the fast path got more tricky to reason about |
20:47 | <shu> | because of the iterables thing |
20:49 | <bakkot> | shu: would it get easier if we froze the next method on ArrayIteratorPrototype |
20:49 | <bakkot> | if so can we just do that |
20:50 | <bakkot> | (and also the Symbol.iterator method on Array.prototype I guess) |
20:51 | <shu> | well we already cache an IteratorRecord so mutations to the iterator or the prototype during iteration won't be reflected |
20:52 | <shu> | i don't think we need to do anything, i'm just venting |
21:13 | <ljharb> | We wouldn’t have to freeze it if we made IsArray things that weren’t proxies or subclasses always call the intrinsic next/Symbol.iterator :-p |
22:46 | <littledan> | We wouldn’t have to freeze it if we made IsArray things that weren’t proxies or subclasses always call the intrinsic next/Symbol.iterator :-p |
22:47 | <littledan> | Except I don’t understand the “that aren’t proxies or subclasses” part, what the line would be |
22:47 | <littledan> | No idea if any change there is web compatible |
22:53 | <ljharb> | i mean like, a proxy to an array would still need to generate the Gets, as would a subclass, to be web compatible |