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

replace complex spec values with spec records #1742

Open
michaelficarra opened this issue Oct 17, 2019 · 4 comments
Open

replace complex spec values with spec records #1742

michaelficarra opened this issue Oct 17, 2019 · 4 comments

Comments

@michaelficarra
Copy link
Member

The spec currently refers to the components of execution contexts in a couple different ways. They are referred to in at least the following ways:

  • "the ComponentName of variableName"
  • "the ComponentName component of variableName"
  • "variableName's ComponentName"
  • "variableName's ComponentName component" (only one use, I will PR it away)

We should choose one and normalise the format. I'm also not sure why execution contexts are not just spec records like basically everything else.

@michaelficarra michaelficarra changed the title inconsistent references to evaluation context components inconsistent references to execution context components Oct 17, 2019
@jmdyck
Copy link
Collaborator

jmdyck commented Oct 17, 2019

I'm also not sure why execution contexts are not just spec records like basically everything else.

About the only reason I can think of is that the code evaluation state component is implementation-dependent, which raises the question of what you would put for its value in a record 'literal'. But that can be handled one way or another.

I think modeling Execution Context as a Record would be an improvement.

ljharb pushed a commit to michaelficarra/ecma262 that referenced this issue Oct 17, 2019
@michaelficarra
Copy link
Member Author

michaelficarra commented Apr 20, 2020

We should convert References to spec records as well.

edit: See #2085.

@michaelficarra
Copy link
Member Author

A State in the Pattern Semantics section contains an ordered pair. We should use spec records there as well.

jmdyck added a commit to jmdyck/ecma262 that referenced this issue Dec 8, 2020
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Jan 16, 2021
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Jan 28, 2021
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Jan 29, 2021
@michaelficarra michaelficarra changed the title inconsistent references to execution context components replace complex spec values with spec records Feb 4, 2021
@ExE-Boss
Copy link
Contributor

ExE-Boss commented Feb 4, 2021

jmdyck added a commit to jmdyck/ecma262 that referenced this issue Feb 5, 2021
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Apr 21, 2021
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Apr 21, 2021
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue May 13, 2021
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Jul 9, 2021
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Jul 18, 2021
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Aug 17, 2021
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Aug 17, 2021
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Aug 25, 2021
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Sep 14, 2021
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Jan 10, 2022
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Jan 21, 2022
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Feb 28, 2022
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Mar 18, 2022
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Apr 22, 2022
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Jun 2, 2022
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Aug 6, 2022
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Aug 16, 2022
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Aug 30, 2022
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Sep 3, 2022
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Nov 4, 2022
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Nov 17, 2022
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Nov 19, 2022
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Jan 15, 2023
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Feb 10, 2023
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Mar 1, 2023
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Apr 21, 2023
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Aug 17, 2023
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Sep 1, 2023
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Oct 13, 2023
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Jan 11, 2024
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Feb 25, 2024
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue May 23, 2024
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Jun 5, 2024
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Jul 11, 2024
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Aug 10, 2024
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Aug 14, 2024
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Oct 25, 2024
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Oct 25, 2024
Resolves the first comment in issue tc39#1742
jmdyck added a commit to jmdyck/ecma262 that referenced this issue Nov 13, 2024
Resolves the first comment in issue tc39#1742
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants