Éric Araujo mer...@netwok.org added the comment:
Éric, thanks for paying attention to this.
You’re welcome. I hope that my commits get reviewed too.
In this particular case, I checked the code and verified that the
variables were not used anywhere.
Yep, I can’t imagine third-party code
Jesús Cea Avión j...@jcea.es added the comment:
Éric, thanks for paying attention to this. In this particular case, I checked
the code and verified that the variables were not used anywhere. But I will
comply with the policy in the future.
Thanks again for pointing out.
--
Éric Araujo mer...@netwok.org added the comment:
Our policy is to not commit code cleanup patches that are not strict bug fixes;
see thread at
http://mail.python.org/pipermail/python-dev/2011-October/114281.html and
http://mail.python.org/pipermail/python-dev/2011-November/114301.html
Jesús Cea Avión j...@jcea.es added the comment:
2.7 is a long term maintenance branch, and this patch is trivial enough. I
think it is risk free :)
--
assignee: - jcea
nosy: +jcea
___
Python tracker rep...@bugs.python.org
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset 0694ebb5db99 by Jesus Cea in branch '2.7':
Closes #13283: removal of two unused variable in locale.py
http://hg.python.org/cpython/rev/0694ebb5db99
New changeset a479aad0231e by Jesus Cea in branch '3.2':
Closes
New submission from Nicolas Évrard ni...@no-log.org:
While using pyflake on some of my file I noticed in a copied version of _group
two unused variables.
The attached patch fixed that very little annoyance.
--
components: Library (Lib)
files: locale.diff
keywords: patch
messages:
Changes by Petri Lehtinen pe...@digip.org:
--
nosy: +petri.lehtinen
stage: - patch review
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13283
___
Florent Xicluna florent.xicl...@gmail.com added the comment:
This patch looks good.
Is it relevant for minor releases 2.7.3 and 3.2.3? I cannot confirm.
--
nosy: +flox
priority: normal - low
versions: +Python 3.2, Python 3.3
___
Python tracker