Commit 05ab2e69 authored by Jeremy Hylton's avatar Jeremy Hylton

Fix SF bug [ 561825 ] Confusing error for "del f()"

In the error message, say del for del and assign for everything else.
parent 5352d8c0
import re
import unittest
import test_support
class SyntaxTestCase(unittest.TestCase):
def _check_error(self, code, errtext,
filename="<testcase>", mode="exec"):
"""Check that compiling code raises SyntaxError with errtext.
errtest is a regular expression that must be present in the
test of the exception raised.
"""
try:
compile(code, filename, mode)
except SyntaxError, err:
mo = re.search(errtext, str(err))
if mo is None:
self.fail("SyntaxError did not contain '%s'" % `errtext`)
else:
self.fail("compile() did not raise SyntaxError")
def test_assign_call(self):
self._check_error("f() = 1", "assign")
def test_assign_del(self):
self._check_error("del f()", "delete")
def test_main():
test_support.run_unittest(SyntaxTestCase)
if __name__ == "__main__":
test_main()
......@@ -2417,6 +2417,10 @@ com_assign_trailer(struct compiling *c, node *n, int assigning, node *augn)
REQ(n, trailer);
switch (TYPE(CHILD(n, 0))) {
case LPAR: /* '(' [exprlist] ')' */
if (assigning == OP_DELETE)
com_error(c, PyExc_SyntaxError,
"can't delete function call");
else
com_error(c, PyExc_SyntaxError,
"can't assign to function call");
break;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment