Skip to content

Remove duplicated <path-resolved user-defined type name> rule #3

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

Merged
merged 2 commits into from
Oct 4, 2017

Conversation

jleffler
Copy link
Collaborator

@jleffler jleffler commented Oct 4, 2017

As Euan Rochester observed, sql-2003-2.bnf contained two copies of the rule for <path-resolved user-define type name> where only one is necessary.

Primary fix removes the duplicate. There was also a stray U+0096 (SPA or 'start of guard area') control character which was replaced by a colon. The RCS version information was updated too, for better or worse. File sql-2003-2.bnf.sql was regenerated from the updated sql-2003-2.bnf.

Remove repeated entry for `<path-resolved user-defined type name>` in sql-2003-2.bnf, as indicated by Euan Rochester.
Update RCS version information.
The generated sql-2003-2.bnf.html file still has to be regenerated (so this isn't ready for commit to master).
Replace U+0096 SPA (start of guarded area) with a colon in sql-2003-2.bnf. Regenerate sql-2003-2.bnf.html from updated sql-2003-2.bnf.
@jleffler jleffler merged commit 84842e6 into master Oct 4, 2017
@jleffler jleffler deleted the jleffler-patch-1 branch October 4, 2017 15:27
ronsavage pushed a commit that referenced this pull request Mar 3, 2020

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
…tical_bar

[cosmetic change]add missing space before vertical bar 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

Successfully merging this pull request may close these issues.

None yet

1 participant