Re: [streams] Merge ReadableByteStream into ReadableStream (#430)

>        },
>  
>        cancel() {
>          return fs.close(fd);
> -      }
> +      },
> +
> +      autoAllocateChunkSize: DEFAULT_CHUNK_SIZE
>      });

Oh, I see, that makes sense. I forgot that we were inside onreadable, not inside pull(). I don't think we need another auto-allocation case for this.

> We can also choose not to drain data from the kernel while there's no outstanding read requests. But it doesn't match the original concept of the example?

Right, I am assuming that the kernel buffer might overflow if we do that.

---
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/streams/pull/430/files/0562267ac9415713151bf0a34ae090c0dfb37c1a..4ad2edc8af0c82da98914eea6bd5c899618d8ec5#r57466397