Repository navigation
Warn about tuple parameters #47310
Description
| BPO | 3060 |
|---|---|
| Nosy | @birkenfeld, @benjaminp |
| Files |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
assignee = 'https://github.com/birkenfeld'
closed_at = <Date 2008-06-08.23:11:06.886>
created_at = <Date 2008-06-08.01:45:53.270>
labels = ['interpreter-core', 'type-bug']
title = 'Warn about tuple parameters'
updated_at = <Date 2008-06-08.23:11:06.885>
user = 'https://github.com/benjaminp'bugs.python.org fields:
activity = <Date 2008-06-08.23:11:06.885>
actor = 'benjamin.peterson'
assignee = 'georg.brandl'
closed = True
closed_date = <Date 2008-06-08.23:11:06.886>
closer = 'benjamin.peterson'
components = ['Interpreter Core']
creation = <Date 2008-06-08.01:45:53.270>
creator = 'benjamin.peterson'
dependencies = []
files = ['10549', '10553']
hgrepos = []
issue_num = 3060
keywords = ['patch']
message_count = 6.0
messages = ['67820', '67823', '67825', '67842', '67845', '67850']
nosy_count = 2.0
nosy_names = ['georg.brandl', 'benjamin.peterson']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = None
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue3060'
versions = ['Python 2.6']
Activity
benjaminp commentedon Jun 8, 2008
This patch adds Py3k warnings to nested tuple parameters.
birkenfeld commentedon Jun 8, 2008
Why not warn in the AST, like for the other syntax-related changes?
The relevant location is ast_for_arguments, line 680 in current SVN.
birkenfeld commentedon Jun 8, 2008
Also a test is missing.
benjaminp commentedon Jun 8, 2008
because I was reading through symtable.c at the time. :)
Anyway, here's a better patch.
birkenfeld commentedon Jun 8, 2008
Replace "upacking" with "unpacking" and you can commit it :)
benjaminp commentedon Jun 8, 2008
Done with r64045.