summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAnthony Sottile <asottile@umich.edu>2019-05-18 11:27:17 -0700
committerMiss Islington (bot) <31488909+miss-islington@users.noreply.github.com>2019-05-18 11:27:16 -0700
commitabea73bf4a320ff658c9a98fef3d948a142e61a9 (patch)
tree22acd64b07fa3dbfcbd30479900e9b4ca57f8f1e
parente917f2ed9af044fe808fc9b4ddc6c5eb99003500 (diff)
downloadcpython-git-abea73bf4a320ff658c9a98fef3d948a142e61a9.tar.gz
bpo-2180: Treat line continuation at EOF as a `SyntaxError` (GH-13401)
This makes the parser consistent with the tokenize module (already the case in `pypy`). sample ------ ```python x = 5\ ``` before ------ ```console $ python3 t.py $ python3 -mtokenize t.py t.py:2:0: error: EOF in multi-line statement ``` after ----- ```console $ ./python t.py File "t.py", line 3 x = 5\ ^ SyntaxError: unexpected EOF while parsing $ ./python -m tokenize t.py t.py:2:0: error: EOF in multi-line statement ``` https://bugs.python.org/issue2180
-rw-r--r--Lib/test/test_eof.py26
-rw-r--r--Misc/NEWS.d/next/Core and Builtins/2019-05-17-18-34-30.bpo-2180.aBqHeW.rst1
-rw-r--r--Parser/tokenizer.c11
3 files changed, 36 insertions, 2 deletions
diff --git a/Lib/test/test_eof.py b/Lib/test/test_eof.py
index 7baa7ae57c..a091ceaa25 100644
--- a/Lib/test/test_eof.py
+++ b/Lib/test/test_eof.py
@@ -1,7 +1,9 @@
"""test script for a few new invalid token catches"""
-import unittest
+import sys
from test import support
+from test.support import script_helper
+import unittest
class EOFTestCase(unittest.TestCase):
def test_EOFC(self):
@@ -24,5 +26,27 @@ class EOFTestCase(unittest.TestCase):
else:
raise support.TestFailed
+ def test_line_continuation_EOF(self):
+ """A contination at the end of input must be an error; bpo2180."""
+ expect = 'unexpected EOF while parsing (<string>, line 1)'
+ with self.assertRaises(SyntaxError) as excinfo:
+ exec('x = 5\\')
+ self.assertEqual(str(excinfo.exception), expect)
+ with self.assertRaises(SyntaxError) as excinfo:
+ exec('\\')
+ self.assertEqual(str(excinfo.exception), expect)
+
+ @unittest.skipIf(not sys.executable, "sys.executable required")
+ def test_line_continuation_EOF_from_file_bpo2180(self):
+ """Ensure tok_nextc() does not add too many ending newlines."""
+ with support.temp_dir() as temp_dir:
+ file_name = script_helper.make_script(temp_dir, 'foo', '\\')
+ rc, out, err = script_helper.assert_python_failure(file_name)
+ self.assertIn(b'unexpected EOF while parsing', err)
+
+ file_name = script_helper.make_script(temp_dir, 'foo', 'y = 6\\')
+ rc, out, err = script_helper.assert_python_failure(file_name)
+ self.assertIn(b'unexpected EOF while parsing', err)
+
if __name__ == "__main__":
unittest.main()
diff --git a/Misc/NEWS.d/next/Core and Builtins/2019-05-17-18-34-30.bpo-2180.aBqHeW.rst b/Misc/NEWS.d/next/Core and Builtins/2019-05-17-18-34-30.bpo-2180.aBqHeW.rst
new file mode 100644
index 0000000000..a2207c17ae
--- /dev/null
+++ b/Misc/NEWS.d/next/Core and Builtins/2019-05-17-18-34-30.bpo-2180.aBqHeW.rst
@@ -0,0 +1 @@
+Treat line continuation at EOF as a ``SyntaxError`` by Anthony Sottile.
diff --git a/Parser/tokenizer.c b/Parser/tokenizer.c
index 5dc2ae65c4..e52d498d55 100644
--- a/Parser/tokenizer.c
+++ b/Parser/tokenizer.c
@@ -983,7 +983,8 @@ tok_nextc(struct tok_state *tok)
return EOF;
/* Last line does not end in \n,
fake one */
- strcpy(tok->inp, "\n");
+ if (tok->inp[-1] != '\n')
+ strcpy(tok->inp, "\n");
}
tok->inp = strchr(tok->inp, '\0');
done = tok->inp[-1] == '\n';
@@ -1674,6 +1675,14 @@ tok_get(struct tok_state *tok, char **p_start, char **p_end)
tok->cur = tok->inp;
return ERRORTOKEN;
}
+ c = tok_nextc(tok);
+ if (c == EOF) {
+ tok->done = E_EOF;
+ tok->cur = tok->inp;
+ return ERRORTOKEN;
+ } else {
+ tok_backup(tok, c);
+ }
tok->cont_line = 1;
goto again; /* Read next line */
}