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

Can SpecialOperation be more simplified? #470

Closed
saschanaz opened this issue Nov 2, 2017 · 2 comments
Closed

Can SpecialOperation be more simplified? #470

saschanaz opened this issue Nov 2, 2017 · 2 comments

Comments

@saschanaz
Copy link
Member

Currently SpecialOperation allows multiple keywords before RegularOperation:

SpecialOperation ::
    Special Specials RegularOperation
Specials ::
    Special Specials
    ε
Special ::
    getter
    setter
    deleter

This was required before #44 to allow setter creator void set(DOMString name, any value); but now it seems no IDL uses getter setter, setter deleter, etc.

So I think SpecialOperation can just be Special RegularOperation, what do you think?

@bzbarsky
Copy link
Collaborator

bzbarsky commented Nov 3, 2017

Should probably also clean up the language in https://heycam.github.io/webidl/#idl-special-operations accordingly. And replace "five" with "four" while we're there.

But yes, this cleanup would make sense to me.

Also, it's not immediately obvious why stringifier and getter/setter/deleter are handled differently in grammar terms...

@tobie
Copy link
Collaborator

tobie commented Nov 3, 2017

So now that we have CI that verifies the grammar stays LL(1), I'm much more confident to do a syntax overhaul/cleanup.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants