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

Remove exceptions support #65

Closed
markandrus opened this issue Apr 15, 2017 · 1 comment
Closed

Remove exceptions support #65

markandrus opened this issue Apr 15, 2017 · 1 comment

Comments

@markandrus
Copy link
Contributor

webidl2.js includes a function for parsing exceptions (I even went so far as to type these as IDLException in my typed branch); however, upon closer inspection, I don't believe WebIDL allows exceptions to be declared. See here:

An exception is a type of object that represents an error and which can be thrown or treated as a first class value by implementations. Web IDL does not allow exceptions to be defined, but instead has a number of pre-defined exceptions that specifications can reference and throw in their definition of operations, attributes, and so on.

I also don't see any reference to "exception" in the grammar, so I think this code should be removed (originally introduced here).

@marcoscaceres
Copy link
Member

yeah, you used to be able to define exceptions many years ago. That's all dead code likely.

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