Issue 17940: extra code in argparse.py (original) (raw)

Created on 2013-05-08 22:32 by aho, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
issue17940.patch Yogesh.Chaudhari,2013-05-11 10:20 review
issue17940-27.patch Yogesh.Chaudhari,2013-05-11 10:26 Patch for 2.7 branch review
Messages (8)
msg188742 - (view) Author: Alex (aho) Date: 2013-05-08 22:32
In class HelpFormatter, class _Section, format_help(self) (lines 199 & 200), the two lines for func, args in self.items: func(*args) aren't necessary; the results of func are ignored. "func" is applied (again) on the next line when the output is joined into item_help. Cheers .a
msg188902 - (view) Author: Yogesh Chaudhari (Yogesh.Chaudhari) * Date: 2013-05-11 10:20
Removed the duplicated code from argparse.py
msg188903 - (view) Author: Yogesh Chaudhari (Yogesh.Chaudhari) * Date: 2013-05-11 10:26
Remove extra code in argparse.py for 2.7 branch
msg188904 - (view) Author: Yogesh Chaudhari (Yogesh.Chaudhari) * Date: 2013-05-11 10:27
I have added a patch for default branch as well, because IMO the same issue exists in all branches
msg188985 - (view) Author: Yogesh Chaudhari (Yogesh.Chaudhari) * Date: 2013-05-12 06:59
I guess that was simple enough. I am new to this issue tracker for Cpython. How will the patch review/commit proceed?
msg189084 - (view) Author: paul j3 (paul.j3) * (Python triager) Date: 2013-05-13 00:24
I was wondering about that block of code earlier. It would be a good idea to look at what func() does, just to make sure there aren't any side effects - e.g. set maximum line length, required indention, etc.
msg261213 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2016-03-05 12:06
New changeset be3874cf87f3 by Berker Peksag in branch 'default': Issue #17940: Remove redundant code from _Section.format_help() https://hg.python.org/cpython/rev/be3874cf87f3
msg261214 - (view) Author: Berker Peksag (berker.peksag) * (Python committer) Date: 2016-03-05 12:06
Thanks!
History
Date User Action Args
2022-04-11 14:57:45 admin set github: 62140
2016-03-05 12:06:40 berker.peksag set status: open -> closedversions: + Python 3.6, - Python 3.4nosy: + berker.peksagmessages: + resolution: fixedstage: patch review -> resolved
2016-03-05 12:06:07 python-dev set nosy: + python-devmessages: +
2013-06-29 00:01:15 berker.peksag set stage: needs patch -> patch reviewversions: + Python 3.4, - Python 2.7
2013-05-13 00:24:51 paul.j3 set nosy: + paul.j3messages: +
2013-05-12 21:35:24 Yogesh.Chaudhari set hgrepos: - hgrepo190
2013-05-12 06:59:36 Yogesh.Chaudhari set messages: +
2013-05-11 10:27:54 Yogesh.Chaudhari set messages: +
2013-05-11 10:26:23 Yogesh.Chaudhari set files: + issue17940-27.patchhgrepos: + hgrepo190messages: +
2013-05-11 10:20:37 Yogesh.Chaudhari set files: + issue17940.patchnosy: + Yogesh.Chaudharimessages: + keywords: + patch
2013-05-10 18:19:03 tshepang set nosy: + tshepang
2013-05-10 17:59:47 ezio.melotti set nosy: + bethardstage: needs patch
2013-05-08 22:32:30 aho create