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

[Bug] %ignorecase ignored by chars class definition [sf#57] #59

Closed
lsf37 opened this issue Feb 15, 2015 · 3 comments
Closed

[Bug] %ignorecase ignored by chars class definition [sf#57] #59

lsf37 opened this issue Feb 15, 2015 · 3 comments

Comments

@lsf37
Copy link
Member

@lsf37 lsf37 commented Feb 15, 2015

*Reported by anonymous on 2003-06-10 06:50 UTC
JLex generated lexer parses string "aEiU" with following
lexer definition

%%

%ignorecase

VOWEL=[aeiouy]

%%

{VOWEL}+ { return 2; }

but JFlex generated parser fails, because it doesn't
properly handles character class definition with respect
to case sensitivity.

Osvaldas

@lsf37 lsf37 changed the title %ignorecase ignored by chars class definition [Bug] %ignorecase ignored by chars class definition [sf#57] Feb 15, 2015
@lsf37 lsf37 added this to the jlex incompatibility milestone Feb 15, 2015
@lsf37 lsf37 closed this Feb 15, 2015
@lsf37
Copy link
Member Author

@lsf37 lsf37 commented Feb 15, 2015

Commented by lsf37 on 2003-06-10 10:26 UTC
Logged In: YES
user_id=93534

I'm tempted to say that this is not a bug, but a feature.
As specified in the manual, %ignorecase should not work on
character classes.

The problem is that %ignorecase in JFlex is older than the
same option in JLex and JLex behaves differently. If I now
change %ignorecase to the JLex behaviour existing JFLex
specifications might no longer work.

I will propably add a command line switch --jlex or
something similar to force the JLex behaviour. This way
you can choose what you like better, and it might also be
useful for other JLex issues.

Gerwin

@lsf37
Copy link
Member Author

@lsf37 lsf37 commented Feb 15, 2015

Updated by lsf37 on 2003-06-10 10:26 UTC

  • milestone: 104412 --> jlex incompatibility
@lsf37
Copy link
Member Author

@lsf37 lsf37 commented Feb 15, 2015

Updated by lsf37 on 2004-04-12 12:26 UTC

  • status: open --> closed
@lsf37 lsf37 removed this from the jlex incompatibility milestone Feb 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.