Skip to content
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

Fix rule for token position example #211

Merged
merged 1 commit into from Nov 15, 2017
Merged

Conversation

mluszczyk
Copy link
Contributor

The sample rule for CIdent has a "|" in it by mistake. The error makes CIdent accept empty strings which causes problems and raises the following warning:

Warning :
The following tokens accept the empty string:
CIdent
This is error-prone and will not be supported in the future.

The sample rule for CIdent has a "|" in it by mistake. The error makes CIdent accept empty strings which causes problems and raises the following warning:

Warning : 
  The following tokens accept the empty string: 
    CIdent
  This is error-prone and will not be supported in the future.
@andreasabel andreasabel merged commit 529a465 into BNFC:master Nov 15, 2017
@mluszczyk
Copy link
Contributor Author

Please also update the "BNF Converter Tutorial" page (http://bnfc.digitalgrammars.com/tutorial.html). The following line appears there:

position token CIdent (letter | (letter | digit | '_')*) ;

@andreasabel
Copy link
Member

andreasabel commented Nov 16, 2017 via email

@andreasabel andreasabel added this to the 2.8.2 milestone Nov 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants