[shex] ShExC grammar for comment has extraneous `| '\\/')` (#114)

• From: Eric Prud'hommeaux via GitHub <sysbot+gh@w3.org>
• Date: Fri, 19 Feb 2021 11:18:43 +0000
• To: public-shex-dev@w3.org
• Message-ID: <issues.opened-811937875-1613733522-sysbot+gh@w3.org>

ericprud has just created a new issue for https://github.com/shexSpec/shex:

== ShExC grammar for comment has extraneous `| '\\/')` ==
http://shex.io/shex-semantics/#term-PASSED_TOKENS says:

<table>
<tr>
<td>[98]</td><td>PASSED TOKENS</td><td>::=</td><td><pre>[ \t\r\n]+
| "#" [^\r\n]*
| "/*" ([^*] | '*' ([^/] | '\\/'))* "*/"</pre></td>
</tr>
On the last line, this clause is not needed: `| '\\/')`.

Demo'd using PCRE as implemented in node javascript:
```javascript
> console.log("ab/*c*\\/d*/ef")
ab/*c*\/d*/ef
undefined
> "ab/*c*\\/d*/ef".match(/\/\*((?:[^*]|\*[^/])*)\*\//)
[
  '/*c*\\/d*/',
  'c*\\/d',
  index: 2,
  input: 'ab/*c*\\/d*/ef',
  groups: undefined
]
```

Please view or discuss this issue at https://github.com/shexSpec/shex/issues/114 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Friday, 19 February 2021 11:18:44 UTC