This is an archived snapshot of W3C's public bugzilla bug tracker, decommissioned in April 2019. Please see the home page for more details.

Bug 20481 - Grammar Fixes needed: Dominique's feedback + Microsoft's feedback
Summary: Grammar Fixes needed: Dominique's feedback + Microsoft's feedback
Status: CLOSED FIXED
Alias: None
Product: WebAppsWG
Classification: Unclassified
Component: WebIDL (show other bugs)
Version: unspecified
Hardware: PC Windows NT
: P2 normal
Target Milestone: ---
Assignee: Cameron McCormack
QA Contact: public-webapps-bugzilla
URL:
Whiteboard: [v1]
Keywords:
Depends on:
Blocks:
 
Reported: 2012-12-21 18:35 UTC by Travis Leithead [MSFT]
Modified: 2013-08-08 19:09 UTC (History)
2 users (show)

See Also:


Attachments

Description Travis Leithead [MSFT] 2012-12-21 18:35:56 UTC
DOM Recently wrote to the public-script-coord list about the current WebIDL grammar having errors:

http://lists.w3.org/Archives/Public/public-script-coord/2012OctDec/0398.html

In addition, some developers on the IE team have raised similar complaints about errors in the grammar:

"Other includes "other" which eats the ] <-- So you can't break out of an attribute.

"we [should] remove "other" as an option and fully specify those values we expect to see in Other. For instance, we already added things like "-" and ".", so just continue adding as many as are necessary. Also, to simplify Other we should remove redundancy.  There are 6 basic tokens:

"integer, float, identifier, string, whitespace, and other. If we remove other and whitespace is already eaten between any tokens implicitly, then the correct production is:

"Other = float / integer / identifier / string / ... { list of other tokens we want to allow }
Comment 1 Cameron McCormack 2013-08-04 06:26:30 UTC
(In reply to comment #0)
> DOM Recently wrote to the public-script-coord list about the current WebIDL
> grammar having errors:
> 
> http://lists.w3.org/Archives/Public/public-script-coord/2012OctDec/0398.html

That's difficult for me to use to fix the grammar in the spec.  If specific errors in the grammar could be pointed out, that would be helpful.

> In addition, some developers on the IE team have raised similar complaints
> about errors in the grammar:
> 
> "Other includes "other" which eats the ] <-- So you can't break out of an
> attribute.

The intention was to always match quoted terminal symbols rather than the named terminal symbols, not just for identifier.  I've mentioned that now:

http://dev.w3.org/cvsweb/2006/webapi/WebIDL/Overview.xml.diff?r1=1.675;r2=1.676;f=h
http://dev.w3.org/cvsweb/2006/webapi/WebIDL/v1.xml.diff?r1=1.117;r2=1.118;f=h

> "we [should] remove "other" as an option and fully specify those values we
> expect to see in Other. For instance, we already added things like "-" and
> ".", so just continue adding as many as are necessary. Also, to simplify
> Other we should remove redundancy.  There are 6 basic tokens:

The "other" terminal symbol is meant to match any text that isn't one of the quoted terminals or float/integer/etc.  So I think it needs to match any character except for any character that could possibly match the other named terminals.

> "integer, float, identifier, string, whitespace, and other. If we remove
> other and whitespace is already eaten between any tokens implicitly, then
> the correct production is:
> 
> "Other = float / integer / identifier / string / ... { list of other tokens
> we want to allow }

I don't think we want to leave out "other" from Other.

Please reopen if I've misunderstood.
Comment 2 Travis Leithead [MSFT] 2013-08-08 19:09:08 UTC
Works for me. Thanks!