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

Normative: Add class static initialization blocks #2440

Merged
merged 1 commit into from
Sep 8, 2021
Merged

Normative: Add class static initialization blocks #2440

merged 1 commit into from
Sep 8, 2021

Conversation

rbuckton
Copy link
Contributor

This adds the specification text from https://github.com/tc39/proposal-class-static-block, currently at Stage 3.

@ljharb ljharb added needs test262 tests The proposal should specify how to test an implementation. Ideally via github.com/tc39/test262 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 Jun 23, 2021
jmdyck
jmdyck requested changes Jun 23, 2021
View reviewed changes
Copy link
Collaborator

@jmdyck jmdyck left a comment

Choose a reason for hiding this comment

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

Also, I believe the following SDO definitions are needed:

  • EvaluateBody for ClassStaticBlockBody
  • PrivateBoundIdentifiers for ClassStaticBlock
  • ComputedPropertyContainsAwait for ClassStaticBlock

spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
@rbuckton rbuckton force-pushed the class-static-block branch 2 times, most recently from 1875a2e to 5319cfe Compare June 24, 2021 20:19
@rbuckton
Copy link
Contributor Author

Thanks @jmdyck. I believe I've addressed your feedback, though I chose to use [[BodyFunction]] over [[Function]] and _bodyFunction_ over _func_ to further improve clarity.

This was referenced Jun 24, 2021
Merged
Merged
Closed
jmdyck
jmdyck previously requested changes Jun 25, 2021
View reviewed changes
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
@ljharb ljharb force-pushed the master branch 3 times, most recently from 3d0c24c to 7a79833 Compare June 29, 2021 02:21
@rbuckton
Copy link
Contributor Author

@tc39/ecma262-editors would you be able to review this?

@ljharb ljharb requested review from syg, michaelficarra, bakkot and a team August 12, 2021 22:07
jmdyck
jmdyck reviewed Aug 13, 2021
View reviewed changes
spec.html Outdated Show resolved Hide resolved
jmdyck
jmdyck reviewed Aug 13, 2021
View reviewed changes
spec.html Outdated Show resolved Hide resolved
@rbuckton
Copy link
Contributor Author

SpiderMonkey intends to ship this unflagged in 93, and it is already shipping in V8. I'd like to seek Stage 4 at the upcoming meeting, assuming an editor review is complete and approved prior to the agenda submission deadline at the end of the week.

@bakkot bakkot added editor call to be discussed in the next editor call has test262 tests and removed needs test262 tests The proposal should specify how to test an implementation. Ideally via github.com/tc39/test262 labels Aug 18, 2021
jmdyck
jmdyck reviewed Aug 18, 2021
View reviewed changes
spec.html Outdated Show resolved Hide resolved
syg
syg approved these changes Aug 24, 2021
View reviewed changes
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
bakkot
bakkot reviewed Aug 24, 2021
View reviewed changes
spec.html Show resolved Hide resolved
bakkot
bakkot approved these changes Aug 24, 2021
View reviewed changes
Copy link
Contributor

@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 apart from open comments.

@bakkot bakkot removed the editor call to be discussed in the next editor call label Aug 25, 2021
@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 Aug 31, 2021
@erik-krogh erik-krogh mentioned this pull request Sep 1, 2021
Merged
@aladdin-add aladdin-add mentioned this pull request Sep 8, 2021
Closed
36 tasks
@bakkot bakkot added editor call to be discussed in the next editor call ready to merge Editors believe this PR needs no further reviews, and is ready to land. and removed editor call to be discussed in the next editor call labels Sep 8, 2021
@ljharb ljharb dismissed jmdyck’s stale review September 8, 2021 22:07

changes addressed

ljharb pushed a commit to rbuckton/ecma262 that referenced this pull request Sep 8, 2021
@rbuckton @ljharb
Normative: add class static initialization blocks (tc39#2440)
b51a822
@ljharb ljharb merged commit 51e28c7 into tc39:master Sep 8, 2021
mathiasbynens pushed a commit to mathiasbynens/ecma262 that referenced this pull request Oct 18, 2021
@rbuckton @mathiasbynens
Normative: add class static initialization blocks (tc39#2440)
3cb09d6
@bakkot bakkot mentioned this pull request Feb 22, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@syg syg syg approved these changes

@jmdyck jmdyck jmdyck left review comments

@bakkot bakkot bakkot approved these changes

@michaelficarra michaelficarra Awaiting requested review from michaelficarra

Assignees
No one assigned
Labels
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
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@rbuckton @syg @bakkot @jmdyck @ljharb