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.

Sign up for GitHub

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

Jump to bottom

Comment support in WebIDL #160

Closed
dirkschulze opened this issue Mar 10, 2013 · 4 comments
Closed

Comment support in WebIDL #160

dirkschulze opened this issue Mar 10, 2013 · 4 comments
Milestone
3.2

Comments

@dirkschulze
Copy link
Contributor

Support for comments in interfaces and dictionaries is missing.

I made some modifications to adapt the old behavior of respec v1 but need to figure out how to build the script :P
@marcoscaceres
Copy link
Member

Relates to: https://github.com/darobin/respec/pull/163

@dirkschulze
Copy link
Contributor Author

The only missing part seems to be multiline comment support such as:

/* This is a comment over
   two lines */

@darobin
Copy link
Member

darobin commented Dec 23, 2014

Note that this can be added to the current work on the new WebIDL module. It's not supported in the existing pull request, but basically the AST from webidl2.js has all the white space and comments, and so can actually be used to reproduce the original formatting with full fidelity.

@darobin darobin modified the milestone: 3.2 Dec 23, 2014
@marcoscaceres
Copy link
Member

This WFM using contiguous webidl.

screenshot 2015-09-12 19 18 16

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
3.2
Development

No branches or pull requests
3 participants
@darobin @marcoscaceres @dirkschulze