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

Parsing updated ecma262 intrinsic notation #480

Closed
ljharb opened this issue Jun 17, 2020 · 4 comments
Closed

Parsing updated ecma262 intrinsic notation #480

ljharb opened this issue Jun 17, 2020 · 4 comments

Comments

@ljharb
Copy link
Member

ljharb commented Jun 17, 2020

Hi!

In the ecma262 spec, we've added a new intrinsic notation form that will avoid unbounded growth of our list of intrinsics.

Specifically, things like %ArrayPrototype% and %ObjProto_valueOf% should now be represented as %Array.prototype% and %Object.prototype.valueOf%, respectively.

Our intention is to delete the "legacy" forms entirely in favor of the dotted forms. I wanted to file a PR on https://github.com/whatwg/html, but then realized I might need to file one on https://github.com/heycam/webidl/blob/master/index.bs#L88-L110, and then saw https://github.com/heycam/webidl#syntax-changes-should-be-filed-against-the-following-parsers and realized I might need to file one here.

I'm happy to do the work to update spec text (and for HTML, I have a branch prepared already), but I doubt I'm qualified to do the parser work. Could someone help give me pointers as to what the next steps should be?

@saschanaz
Copy link
Member

Hi, thanks for opening this!

I guess this specific one should go heycam/webidl repo, as the ecma262 syntax change does not affect IDL syntax so no change is required for this library.

Since I have no write access there unfortunately, could you open a new issue there? Thanks!

@ljharb
Copy link
Member Author

ljharb commented Jun 17, 2020

Absolutely, thanks :-) I was concerned there'd be parsing issues around %A.b% versus just %A%.

@ljharb
Copy link
Member Author

ljharb commented Jun 17, 2020

Closing in favor of whatwg/webidl#897. Thanks for the quick reply!

@ljharb ljharb closed this as completed Jun 17, 2020
@saschanaz
Copy link
Member

Thanks! I see JSON.stringify is already in that list so it shouldn't break anything.

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

2 participants