Add support for x'HEX' and X'HEX' Numeric Constants in SQLite by RubieV · Pull Request #582 · JSQLParser/JSqlParser
Navigation Menu
{{ message }}
JSQLParser / JSqlParser Public
- Notifications You must be signed in to change notification settings
- Fork 1.4k
Closed
Add support for x'HEX' and X'HEX' Numeric Constants in SQLite#582
RubieV wants to merge 1 commit intoJSQLParser:masterfrom
Add support for x'HEX' and X'HEX' Numeric Constants in SQLite#582
RubieV wants to merge 1 commit intoJSQLParser:masterfrom
Conversation
Copy link Copy Markdown
RubieV
commented
Feb 8, 2018
RubieV
commented
Currently, only x'HEX' numeric constants are supported. SQLite supports X'HEX' as well, as "string literals containing hexadecimal data"
Currently, only x'HEX' numeric constants are supported. SQLite supports X'HEX' as well, as "string literals containing hexadecimal data"
Copy link Copy Markdown
coveralls
commented
Feb 8, 2018
coveralls commented
Feb 8, 2018
1 similar comment
Copy link Copy Markdown
coveralls
commented
Feb 8, 2018
coveralls commented
Feb 8, 2018wumpz reviewed Feb 9, 2018
| | < S_LONG: ( <DIGIT> )+ > | ||
| | < #DIGIT: ["0" - "9"] > | ||
| | < S_HEX: ("x'" ( <HEX_VALUE> )+ "'" | "0x" ( <HEX_VALUE> )+ ) > | ||
| | < S_HEX: (["x","X"] "'" ( <HEX_VALUE> )+ "'" | "0x" ( <HEX_VALUE> )+ ) > |
Copy link Copy Markdown
Member
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since javacc builds case insensitive parser, this change is not necessary. I also included a test for this and it succeeded.
Copy link Copy Markdown
Author
RubieV
commented
Feb 9, 2018
RubieV commented
Feb 9, 2018Sorry for the inconvenience, this is indeed the case. I must have been running an older version of JSqlParser.
RubieV
closed this
Copy link Copy Markdown
Member
wumpz
commented
Feb 9, 2018
wumpz commented
Feb 9, 2018No problem. Thx for using JSqlParser.
Cheers Tobias
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment