New WebIDL types like CancelablePromise cause <types> to disappear

[arunranga]

See http://w3c.github.io/filesystem-api/Overview.html#directory-webidl and in particular, the "move" method.

Highlight the "move" method and check out the selection source.

If you remove the "Cancelable" part of "CancelablePromise" it shows the . It is supposed to look like this:

CancelablePromise move((DOMString or File or Directory) path, (DOMString or Directory or DestinationDict) dest)

but respec.js seems to shed the "" until I remove the "Cancelable" and leave it as Promise<void> in which case it shows up as intended.

[arunranga]

Whoops, some clarification needed:

Somehow, respec doesn't allow you to use CancelablePromise<void>

It strips the "<void>" leaving just CancelablePromise fu(...)

[darobin]

Ah, I believe that @tobie fixed that for Promise, I presume the same can be applied here.

[tobie]

Actually, I had fixed that in w3c/webidl2.js#15, not here.

[halindrome]

This should be fixed in version 3.2.26 - I pushed that this morning.

[arunranga]

When will the fix be reflected in http://www.w3.org/Tools/respec/respec-w3c-common.js ?

Is there any action I need to take? Sorry if I've missed something.

[halindrome]

As far as I know that is reflected now. I pushed version 3.2.26 days ago.

On Fri, Oct 31, 2014 at 10:15 AM, Arun Ranganathan <notifications@github.com

wrote:

When will the fix be reflected in
http://www.w3.org/Tools/respec/respec-w3c-common.js ?

Is there any action I need to take? Sorry if I've missed something.

—
Reply to this email directly or view it on GitHub
#352 (comment).

Shane McCarron
halindrome@gmail.com

[darobin]

On 31/10/2014 12:44 , Shane McCarron wrote:

As far as I know that is reflected now. I pushed version 3.2.26 days ago.

Nope. Not sure which crank you used but the sync hasn't happened.

Robin Berjon - http://berjon.com/ - @robinberjon

[halindrome]

Auto sync was apparently broken. This is fixed now.

On Fri, Oct 31, 2014 at 2:35 PM, Robin Berjon notifications@github.com
wrote:

On 31/10/2014 12:44 , Shane McCarron wrote:

As far as I know that is reflected now. I pushed version 3.2.26 days ago.

Nope. Not sure which crank you used but the sync hasn't happened.

Robin Berjon - http://berjon.com/ - @robinberjon

—
Reply to this email directly or view it on GitHub
#352 (comment).

Shane McCarron
halindrome@gmail.com