Bug 215514 – [WebIDL] 'void' type is changing to 'undefined'

• Status: RESOLVED
• Resolution: FIXED
• Component: Bindings

[Tab Atkins Jr.]

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 https://github.com/heycam/webidl/pull/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). You may wish to have a transitional period where you recognize 'void' and 'undefined' as synonyms, as specifications are moved over to the new syntax.

[Sam Weinig]

*** Bug 215652 has been marked as a duplicate of this bug. ***

[Radar WebKit Bug Importer]

<rdar://problem/67566201>

[Sam Weinig]

Created attachment 407481 [details]
Patch

[Darin Adler]

Comment on attachment 407481 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=407481&action=review

> Source/WebCore/ChangeLog:10
> +        Also updates the parser to better match current WebIDl grammar, removing the special 

WebIDL

> Source/WebCore/bindings/scripts/test/JS/JSTestIterable.cpp:200
> -    using KeyType = void;
> +    using KeyType = undefined;

Looks like this change in expected results is wrong.

> Source/WebCore/bindings/scripts/test/JS/JSTestNode.cpp:388
> -    using KeyType = void;
> +    using KeyType = undefined;

Ditto.

[Sam Weinig]

(In reply to Darin Adler from comment #4)
> Comment on attachment 407481 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=407481&action=review
> 
> > Source/WebCore/ChangeLog:10
> > +        Also updates the parser to better match current WebIDl grammar, removing the special 
> 
> WebIDL

Fixing.

> 
> > Source/WebCore/bindings/scripts/test/JS/JSTestIterable.cpp:200
> > -    using KeyType = void;
> > +    using KeyType = undefined;
> 
> Looks like this change in expected results is wrong.
> 
> > Source/WebCore/bindings/scripts/test/JS/JSTestNode.cpp:388
> > -    using KeyType = void;
> > +    using KeyType = undefined;
> 
> Ditto.

Oh, yeah, didn't run run-bindings-test --reset-results after I fixed that. Fixing.

[Sam Weinig]

Created attachment 407484 [details]
Patch

[EWS]

Tools/Scripts/svn-apply failed to apply attachment 407484 [details] to trunk.
Please resolve the conflicts and upload a new patch.

[Sam Weinig]

Created attachment 407507 [details]
Patch

[EWS]

Committed r266311: <https://trac.webkit.org/changeset/266311>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 407507 [details].