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

[idlharness] 'void' type is changing to 'undefined' #25015

Closed
tabatkins opened this issue Aug 14, 2020 · 1 comment
Closed

[idlharness] 'void' type is changing to 'undefined' #25015

tabatkins opened this issue Aug 14, 2020 · 1 comment

Comments

@tabatkins
Copy link
Contributor

@tabatkins tabatkins commented Aug 14, 2020

As part of efforts to finally add an 'undefined' type to WebIDL (so methods/etc don't have to generalize their type to 'any'), it was decided to actually add 'undefined' as the type name, and so the previous function return signature 'void' is being removed for redundancy.

See heycam/webidl#906 for details on the change.

This change will be merged shortly; this bug is a heads-up to adjust your IDL parser (see the PR for the changes to the grammar; it's quite small).

@stephenmcgruer
Copy link
Contributor

@stephenmcgruer stephenmcgruer commented Feb 22, 2021

I don't think there is any work to do here; we've landed many PRs that use undefined in the IDL and it's gone fine so I assume webidl2 supports it fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants