Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Transfer buffers passed to ReadableStreamBYOBReader.read(view) #136

Merged
merged 7 commits into from
Jan 4, 2024
Merged

Transfer buffers passed to ReadableStreamBYOBReader.read(view) #136

merged 7 commits into from
Jan 4, 2024

Conversation

MattiasBuelens
Copy link
Owner

We now have ArrayBuffer.prototype.transfer() and structuredClone(), so we can finally properly transfer the buffers passed to read(view) like the specification wants! 🎉

@MattiasBuelens MattiasBuelens added enhancement domain: compliance Issues related to compliance with the streams standard labels Jan 4, 2024
@MattiasBuelens MattiasBuelens added this to the v3.3.0 milestone Jan 4, 2024
@MattiasBuelens MattiasBuelens merged commit 41516dc into master Jan 4, 2024
5 checks passed
@MattiasBuelens MattiasBuelens deleted the transfer-using-structured-clone branch January 4, 2024 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LeonardDrs LeonardDrs LeonardDrs left review comments

Assignees
No one assigned
Labels
domain: compliance Issues related to compliance with the streams standard enhancement
Projects
None yet
Milestone
v3.3.0
Development

Successfully merging this pull request may close these issues.
2 participants
@MattiasBuelens @LeonardDrs