diff options
| author | Amaury Forgeot d'Arc <amauryfa@gmail.com> | 2011-11-04 21:17:45 (GMT) |
|---|---|---|
| committer | Amaury Forgeot d'Arc <amauryfa@gmail.com> | 2011-11-04 21:17:45 (GMT) |
| commit | 97c1bef6a4f3c522826386480c62ec27fe46301d (patch) | |
| tree | ad74de7776f27145f7341dfe24ca961e5267603d | |
| parent | 3c85fe07f40a18d9f5733e85b213a6992c5b2ed4 (diff) | |
| download | cpython-97c1bef6a4f3c522826386480c62ec27fe46301d.zip cpython-97c1bef6a4f3c522826386480c62ec27fe46301d.tar.gz cpython-97c1bef6a4f3c522826386480c62ec27fe46301d.tar.bz2 | |
Issue #13343: Fix a SystemError when a lambda expression uses a global
variable in the default value of a keyword-only argument:
(lambda *, arg=GLOBAL_NAME: None)
| -rw-r--r-- | Lib/test/test_keywordonlyarg.py | 8 | ||||
| -rw-r--r-- | Misc/NEWS | 4 | ||||
| -rw-r--r-- | Python/symtable.c | 3 |
3 files changed, 15 insertions, 0 deletions
diff --git a/Lib/test/test_keywordonlyarg.py b/Lib/test/test_keywordonlyarg.py index d7f7541..3aebd68 100644 --- a/Lib/test/test_keywordonlyarg.py +++ b/Lib/test/test_keywordonlyarg.py @@ -162,6 +162,14 @@ class KeywordOnlyArgTestCase(unittest.TestCase): self.assertEqual(Example.f(Example(), k1=1, k2=2), (1, 2)) self.assertRaises(TypeError, Example.f, k1=1, k2=2) + def test_issue13343(self): + # The Python compiler must scan all symbols of a function to + # determine their scope: global, local, cell... + # This was not done for the default values of keyword + # arguments in a lambda definition, and the following line + # used to fail with a SystemError. + lambda *, k1=unittest: None + def test_main(): run_unittest(KeywordOnlyArgTestCase) @@ -10,6 +10,10 @@ What's New in Python 3.2.3? Core and Builtins ----------------- +- Issue #13343: Fix a SystemError when a lambda expression uses a global + variable in the default value of a keyword-only argument: + (lambda *, arg=GLOBAL_NAME: None) + - Issue #10519: Avoid unnecessary recursive function calls in setobject.c. diff --git a/Python/symtable.c b/Python/symtable.c index 5eff364..1ec51f7 100644 --- a/Python/symtable.c +++ b/Python/symtable.c @@ -1334,6 +1334,9 @@ symtable_visit_expr(struct symtable *st, expr_ty e) return 0; if (e->v.Lambda.args->defaults) VISIT_SEQ(st, expr, e->v.Lambda.args->defaults); + if (e->v.Lambda.args->kw_defaults) + VISIT_KWONLYDEFAULTS(st, + e->v.Lambda.args->kw_defaults); if (!symtable_enter_block(st, lambda, FunctionBlock, (void *)e, e->lineno, e->col_offset)) |