Raymond Hettinger added the comment:

I really don't like how this patch is done or even the spirit of it.  To my 
eyes, some of the code looks worse (especially the addition of trailing 
backslashes and some odd line-wraps that aren't sensitive to readability or 
context).

I'm going to close this.   If Ethan wants to make minor tweaks to his 
whitespace or line-wrapping, I believe he can do a better job with his own code.

Jean-Sebastien, thank you for trying to contribute, but PEP-8ing other people's 
code generally isn't helpful.  I recommend that you find a tracker item that 
needs a patch for a substantive change and work on that.  I think you will find 
it more satisfying and will have a much greater chance of acceptance.

----------
nosy: +rhettinger
resolution:  -> rejected
status: open -> closed

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue29103>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to