Skip to content

Editorial: TypedArrayCreateSameType take length as parameter#3570

Merged
ljharb merged 1 commit intotc39:mainfrom
aapoalas:feat/typed-array-create-same-type-length
Aug 26, 2025
Merged

Editorial: TypedArrayCreateSameType take length as parameter#3570
ljharb merged 1 commit intotc39:mainfrom
aapoalas:feat/typed-array-create-same-type-length

Conversation

@aapoalas
Copy link
Contributor

@aapoalas aapoalas commented Apr 20, 2025

This method is only called with an explicit length but because it can theoretically take any arguments list, the call to TypedArray construction could trigger user code: this is not really possible in the current spec.

@ljharb ljharb changed the title feat: TypedArrayCreateSameType take length as parameter Editorial: TypedArrayCreateSameType take length as parameter Apr 21, 2025
Copy link
Member

@bakkot bakkot left a comment

Choose a reason for hiding this comment

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

lgtm other than comment (which we can address when landing if you don't get to it)

@aapoalas
Copy link
Contributor Author

lgtm other than comment (which we can address when landing if you don't get to it)

Sorry this took a while, merged the comment. Thanks! <3

@michaelficarra michaelficarra added the ready to merge Editors believe this PR needs no further reviews, and is ready to land. label Aug 25, 2025
@ljharb ljharb force-pushed the feat/typed-array-create-same-type-length branch from 00f75c7 to 3864092 Compare August 25, 2025 23:33
@ljharb ljharb merged commit 3864092 into tc39:main Aug 26, 2025
7 checks passed
Jack-Works added a commit to engine262/engine262 that referenced this pull request Aug 31, 2025
Jack-Works added a commit to engine262/engine262 that referenced this pull request Oct 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

editorial change 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