Skip to content

Duplicate rule? #2

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.

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

Closed
Eroc33 opened this issue Oct 4, 2017 · 2 comments
Closed

Duplicate rule? #2

Eroc33 opened this issue Oct 4, 2017 · 2 comments

Comments

@Eroc33
Copy link

Eroc33 commented Oct 4, 2017

At https://github.com/ronsavage/SQL/blob/master/sql-2003-2.bnf#L1344-L1346 <path-resolved user-defined type name> seems to be duplicated

Edit:
Also <character set specification list> at L4491 and L5307
and <item number> at L5422 and L5356

@jleffler
Copy link
Collaborator

jleffler commented Oct 4, 2017

I've fixed the headline problem with commit 84842e6. I've only just read the issue in full and hence didn't previously see the other two repetitions (which are subtler — the entries are further apart in the source) so I still need to fix those. This issue is still active, therefore.

Thanks for the bug report, Euan.

@jleffler
Copy link
Collaborator

Fix the other repeated rule problems, and upgrade the bnf2html.pl script so it spots them automatically. Also strip trailing blanks.

Thank you, Euan, for the bug report, and I'm sorry it took so long to complete the changes.

ronsavage pushed a commit that referenced this issue Mar 1, 2020

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
[cosmetic change]add missing --/p to sql-2003-1.bnf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants