diff options
author | yaroslav-o <29219583+yaroslav-o@users.noreply.github.com> | 2019-09-25 05:44:54 -0700 |
---|---|---|
committer | Eli Bendersky <eliben@users.noreply.github.com> | 2019-09-25 05:44:54 -0700 |
commit | a4a7127dadf79ebf0deacf49f70ed9e588c40596 (patch) | |
tree | 5f2c3d980c653b644f3f357b63a896a91d597847 /tests/test_c_lexer.py | |
parent | 62ee4ba5fbe58f469c72e7b5b02e88584577a147 (diff) | |
download | pycparser-a4a7127dadf79ebf0deacf49f70ed9e588c40596.tar.gz |
Recognize integer multicharacter constants like 'ABCD' (#350)
Recognize integer multicharacter constants like 'ABCD'
The feature I am adding is defined here - 5th case.
https://en.cppreference.com/w/c/language/character_constant
Also here: 6.4.4.4.10 of C99.
Put simply, pycparser thought a statement like this is an error:
int a = 'ABCD';
However it is not.
It is likely possible to just modify char_const regular expression in c_lexer.py:240 to allow longer characters, but the way it is done in this PR - multicharacter constants are clearly separated. I am also limiting the length of multicharacter const integers to 4 characters - this matches VS compiler behavior (gcc allows any length with a warning) and lets pycparser NOT consider lengthy single-quoted strings as integers - these would be nonsensical anyway.
Diffstat (limited to 'tests/test_c_lexer.py')
-rw-r--r-- | tests/test_c_lexer.py | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/tests/test_c_lexer.py b/tests/test_c_lexer.py index 3a70c18..d63d6fd 100644 --- a/tests/test_c_lexer.py +++ b/tests/test_c_lexer.py @@ -77,6 +77,10 @@ class TestCLexerNoErrors(unittest.TestCase): self.assertTokensTypes('0xf7', ['INT_CONST_HEX']) self.assertTokensTypes('0b110', ['INT_CONST_BIN']) self.assertTokensTypes('0x01202AAbbf7Ul', ['INT_CONST_HEX']) + self.assertTokensTypes("'12'", ['INT_CONST_CHAR']) + self.assertTokensTypes("'123'", ['INT_CONST_CHAR']) + self.assertTokensTypes("'1AB4'", ['INT_CONST_CHAR']) + self.assertTokensTypes(r"'1A\n4'", ['INT_CONST_CHAR']) # no 0 before x, so ID catches it self.assertTokensTypes('xf7', ['ID']) @@ -448,11 +452,11 @@ class TestCLexerErrors(unittest.TestCase): self.assertLexerError("'", ERR_UNMATCHED_QUOTE) self.assertLexerError("'b\n", ERR_UNMATCHED_QUOTE) self.assertLexerError("'\\xaa\n'", ERR_UNMATCHED_QUOTE) - - self.assertLexerError(r"'\12a'", ERR_INVALID_CCONST) - self.assertLexerError(r"'\xabg'", ERR_INVALID_CCONST) + + self.assertLexerError(r"'123\12a'", ERR_INVALID_CCONST) + self.assertLexerError(r"'123\xabg'", ERR_INVALID_CCONST) self.assertLexerError("''", ERR_INVALID_CCONST) - self.assertLexerError("'jx'", ERR_INVALID_CCONST) + self.assertLexerError("'abcjx'", ERR_INVALID_CCONST) self.assertLexerError(r"'\*'", ERR_INVALID_CCONST) def test_string_literals(self): |