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

[css-layout-api] VoidFunction is also defined in another spec #335

Closed
bzbarsky opened this issue Dec 19, 2016 · 5 comments
Closed

[css-layout-api] VoidFunction is also defined in another spec #335

bzbarsky opened this issue Dec 19, 2016 · 5 comments

Comments

@bzbarsky
Copy link

https://drafts.css-houdini.org/css-layout-api/#registering-a-layout has:

callback VoidFunction = void ();

but VoidFunction is also defined in https://webaudio.github.io/web-audio-api/#AudioWorkletGlobalScope and a conformant IDL parser will error out on the duplicate definition of the name.

We could change IDL to allow this sort of thing: identical callback definition sharing a name. Or we could change these two specs to reference VoidFunction as defined in some one place.

@tabatkins
Copy link
Member

Like, say, the Infra spec over in WHATWG?

@bzbarsky
Copy link
Author

That would be a reasonable place for something like this, yes.

@annevk
Copy link
Member

annevk commented Jan 5, 2017

From @foolip:

https://heycam.github.io/webidl/#common already has VoidFunction.

@annevk
Copy link
Member

annevk commented Jan 5, 2017

Filed WebAudio/web-audio-api#1132 on Web Audio.

@bfgeek
Copy link
Contributor

bfgeek commented Jan 5, 2017

I've removed this from both the paint and layout specs.

@bfgeek bfgeek closed this as completed Jan 5, 2017
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

4 participants