Re: [whatwg/streams] Commit pull-into descriptors after filling from queue (PR #1326)

@MattiasBuelens commented on this pull request.



> +    if (SafeCopyDataBlockBytes(pullIntoDescriptor.buffer, destStart, headOfQueue.buffer, headOfQueue.byteOffset,
+                               bytesToCopy) === false) {
+      // This should never happen. Please report an issue if it does! https://github.com/whatwg/streams/issues
+      const e = new TypeError('Invalid buffer');
+      ReadableByteStreamControllerError(controller, e);
+      return false;
+    }

I changed it to an assertion with a big warning.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/streams/pull/1326#discussion_r1761992636
You are receiving this because you are subscribed to this thread.

Message ID: <whatwg/streams/pull/1326/review/2307888858@github.com>