-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 flex warning about default rule #3459
Conversation
What's it? It jumps to the current page |
👍 fixed |
nebula/src/common/time/parser/datetime_scanner.lex Lines 60 to 64 in cc9e6bf
I don't know why datetime_scanner.lex hasn't added this rule before line 62:
Both scannner.lex and wkt_scanner.lex have it because the last rule |
This parser can't ignore these characters directly. |
So change the rule to |
So could i close this PR ? |
Yes, please review this PR #3461 |
Replace by #3461 |
What type of PR is this?
What does this PR do?
Fixed the warning introduced by #3179
reference: https://stackoverflow.com/questions/1622493/bison-build-warning-s-option-given-but-default-rule-can-be-matched