- From: Raphael Kubo da Costa <notifications@github.com>
- Date: Tue, 19 Oct 2021 14:59:23 -0700
- To: whatwg/webidl <webidl@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Tuesday, 19 October 2021 21:59:35 UTC
With my implementer hat on, I guess what's confusing is the (potential) signed/unsigned mixture in the algorithm (the current Blink implementation special-cases the empty [[BackingList]] case for this reason). From the spec it's possible to know `uint32Len`'s type/range, but in `oldLen`'s case it's not that clear -- it represents a list size, which I thought would be unsigned, but if `indexToDelete` were to become -1 then I guess it's not the case. Also, if it's a mathematical number in the ECMAScript sense, isn't `indexToDelete >= uint32Len` applying an operator to mixed-type operands (mathematical value and Number, respectively), which is not allowed? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/whatwg/webidl/issues/1049#issuecomment-947137447
Received on Tuesday, 19 October 2021 21:59:35 UTC