feat: support constructor as a member #300
Conversation
test/syntax/idl/constructor.widl
Outdated
| interface Circle { | ||
| constructor(); | ||
| constructor(float radius); |
There was a problem hiding this comment.
Wonder if we should add our own cases too with other argument things... probably not necessary given everything uses the same code path.
There was a problem hiding this comment.
..e.g. variadic, optional, defaults, etc.
There was a problem hiding this comment.
I think it's not strictly needed but also does not harm. I'll add them as it's simple enough.
whatwg/webidl#700 (review) says the vise-versa thing, wouldn't it be a deadlock? I'm not sure. |
|
Ok, I think we should let them know we are ready on our side. We are a small piece of the solution: getting Gecko, Blink, etc. to update their binding layers will take some time. We could also add ReSpec support, which will also help move things along a little. |
|
Should be merged only when we are confident that browser implementors are on board. |
|
Yay! Looks like this can finally land... we should make a plan for what to do about specs that are using the old syntax. |
We can add an autofix, but I feel like I should wait until at least some of the spec actually start to use the new syntax. |
|
I'd like to use it for PaymentRequest :) |
For whatwg/webidl#700.