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.

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

Normative: add ArrayBuffer.prototype.{detached,transfer,transferToFixedLength} #3175

Merged
merged 1 commit into from
Feb 15, 2024

Conversation

ljharb
Copy link
Member

@ljharb ljharb commented Sep 15, 2023

@ljharb ljharb added normative change Affects behavior required to correctly evaluate some ECMAScript source text pending stage 4 This proposal has not yet achieved stage 4, but may otherwise be ready to merge. proposal This is related to a specific proposal, and will be closed/merged when the proposal reaches stage 4. labels Sep 15, 2023
@ljharb ljharb requested a review from a team September 15, 2023 17:36
@ljharb ljharb marked this pull request as draft September 15, 2023 17:36
@syg
Copy link
Contributor

syg commented Sep 18, 2023

This should be sequenced after #3116, as it uses AOs introduced in that PR which have been renamed and updated since the spec draft.

@ljharb
Copy link
Member Author

ljharb commented Sep 18, 2023

ah ok, i didn't realize the proposal repo was missing some of the changes in the spec PR, and i based it off of the proposal repo. I'll update that as well. (it'd be nice to pull the 262 changes into the proposal repo also, before it's archived)

@ljharb
Copy link
Member Author

ljharb commented Oct 25, 2023

@bakkot as discussed, heads up that this is failing lint because DetachArrayBuffer went from 0 to 2 internal callsites (that assert it's a normal completion), but it has external callsites that require it to be a completion.

@ljharb ljharb marked this pull request as ready for review October 25, 2023 22:06
Copy link
Contributor

@syg syg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

spec.html Outdated Show resolved Hide resolved
@ljharb ljharb requested a review from a team January 26, 2024 16:32
@ljharb ljharb added has stage 4 This PR represents a proposal that has achieved stage 4, and is ready to merge. and removed pending stage 4 This proposal has not yet achieved stage 4, but may otherwise be ready to merge. labels Feb 6, 2024
@ljharb ljharb added the es2024 to be landed before es2024 label Feb 8, 2024
spec.html Outdated Show resolved Hide resolved
@bakkot bakkot added the ready to merge Editors believe this PR needs no further reviews, and is ready to land. label Feb 15, 2024
@ljharb ljharb merged commit 44bb0e0 into tc39:main Feb 15, 2024
8 checks passed
@ljharb ljharb deleted the transfer branch February 15, 2024 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
es2024 to be landed before es2024 has stage 4 This PR represents a proposal that has achieved stage 4, and is ready to merge. has test262 tests normative change Affects behavior required to correctly evaluate some ECMAScript source text proposal This is related to a specific proposal, and will be closed/merged when the proposal reaches stage 4. ready to merge Editors believe this PR needs no further reviews, and is ready to land.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants