summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBenjamin Peterson <benjamin@python.org>2014-06-07 19:36:39 (GMT)
committerBenjamin Peterson <benjamin@python.org>2014-06-07 19:36:39 (GMT)
commitc41616230211822e838634b84b445f8a124add08 (patch)
treee6b7fdd58675a3e0e5957dea0ad159168c5371db
parent024b2f52bf76358eea62f0206ea32c1ca518fde3 (diff)
downloadcpython-c41616230211822e838634b84b445f8a124add08.zip
cpython-c41616230211822e838634b84b445f8a124add08.tar.gz
cpython-c41616230211822e838634b84b445f8a124add08.tar.bz2
allow the keyword else immediately after (no space) an integer (closes #21642)
-rw-r--r--Lib/test/test_grammar.py6
-rw-r--r--Misc/NEWS4
-rw-r--r--Parser/tokenizer.c19
3 files changed, 24 insertions, 5 deletions
diff --git a/Lib/test/test_grammar.py b/Lib/test/test_grammar.py
index bba8820..f012de1 100644
--- a/Lib/test/test_grammar.py
+++ b/Lib/test/test_grammar.py
@@ -80,6 +80,12 @@ class TokenTests(unittest.TestCase):
x = .3e14
x = 3.1e4
+ def test_float_exponent_tokenization(self):
+ # See issue 21642.
+ self.assertEqual(1 if 1else 0, 1)
+ self.assertEqual(1 if 0else 0, 0)
+ self.assertRaises(SyntaxError, eval, "0 if 1Else 0")
+
def test_string_literals(self):
x = ''; y = ""; self.assertTrue(len(x) == 0 and x == y)
x = '\''; y = "'"; self.assertTrue(len(x) == 1 and x == y and ord(x) == 39)
diff --git a/Misc/NEWS b/Misc/NEWS
index acc76b4..8f05a13 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -10,6 +10,10 @@ Release date: XXXX-XX-XX
Core and Builtins
-----------------
+- Issue #21642: If the conditional if-else expression, allow an integer written
+ with no space between itself and the ``else`` keyword (e.g. ``True if 42else
+ False``) to be valid syntax.
+
- Issue #21523: Fix over-pessimistic computation of the stack effect of
some opcodes in the compiler. This also fixes a quadratic compilation
time issue noticeable when compiling code with a large number of "and"
diff --git a/Parser/tokenizer.c b/Parser/tokenizer.c
index 7283058..22accd1 100644
--- a/Parser/tokenizer.c
+++ b/Parser/tokenizer.c
@@ -1597,15 +1597,24 @@ tok_get(struct tok_state *tok, char **p_start, char **p_end)
} while (isdigit(c));
}
if (c == 'e' || c == 'E') {
- exponent:
+ int e;
+ exponent:
+ e = c;
/* Exponent part */
c = tok_nextc(tok);
- if (c == '+' || c == '-')
+ if (c == '+' || c == '-') {
c = tok_nextc(tok);
- if (!isdigit(c)) {
- tok->done = E_TOKEN;
+ if (!isdigit(c)) {
+ tok->done = E_TOKEN;
+ tok_backup(tok, c);
+ return ERRORTOKEN;
+ }
+ } else if (!isdigit(c)) {
tok_backup(tok, c);
- return ERRORTOKEN;
+ tok_backup(tok, e);
+ *p_start = tok->start;
+ *p_end = tok->cur;
+ return NUMBER;
}
do {
c = tok_nextc(tok);