Re: Small grammar problem

Robin Berjon wrote:
> I found a small error in the grammar. The terminals are supposed to be
> given in Perl regex syntax, but the whitespace terminal has unescaped
> slashes:
>
> [\t\n\r ]+|[\t\n\r ]*((//.*|/\*(.|\n)*?\*/)[\t\n\r ]*)+
>
> whereas I believe you want
>
> [\t\n\r ]+|[\t\n\r ]*((\/\/.*|\/\*(.|\n)*?\*\/)[\t\n\r ]*)+

That assumes I'm implying /.../ around them rather than qr{...}. :) 
Better to make it explicit, though.  (White space and comments have been 
split out since you sent in this comment btw.)

http://dev.w3.org/cvsweb/2006/webapi/WebIDL/Overview.xml.diff?r1=1.644;r2=1.645;f=h

Received on Tuesday, 23 July 2013 06:08:55 UTC