[issue13283] removal of two unused variable in locale.py

2011-11-07 Thread Éric Araujo
Éric Araujo 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 being broken by thi

[issue13283] removal of two unused variable in locale.py

2011-11-06 Thread Jesús Cea Avión
Jesús Cea Avión 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. -- __

[issue13283] removal of two unused variable in locale.py

2011-11-04 Thread Éric Araujo
Éric Araujo 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 -- nosy: +eric.a

[issue13283] removal of two unused variable in locale.py

2011-10-31 Thread Roundup Robot
Roundup Robot 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 #13283: removal of two unused variab

[issue13283] removal of two unused variable in locale.py

2011-10-31 Thread Jesús Cea Avión
Jesús Cea Avión 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

[issue13283] removal of two unused variable in locale.py

2011-10-28 Thread Florent Xicluna
Florent Xicluna 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

[issue13283] removal of two unused variable in locale.py

2011-10-28 Thread Petri Lehtinen
Changes by Petri Lehtinen : -- nosy: +petri.lehtinen stage: -> patch review ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue13283] removal of two unused variable in locale.py

2011-10-28 Thread Nicolas Évrard
New submission from Nicolas Évrard : 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: 146549 nosy: nicoe pr