<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">Neil, you have no idea. Please back off.<br></div><div class="gmail_quote"><br>On Fri, Mar 20, 2015 at 8:02 PM, Neil Girdhar <span dir="ltr"><<a href="mailto:mistersheik@gmail.com" target="_blank">mistersheik@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">The code reviews I got asked me to revert PEP 7 changes.  I can understand that, but then logically someone should go ahead and clean up the code.  It's not "high risk" if you just check for whitespace equivalence of the source code and binary equivalence of the compiled code.  The value is for people who are new to the codebase.<div><br></div><div>Best,</div><div><br></div><div>Neil</div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 20, 2015 at 10:35 PM, Brian Curtin <span dir="ltr"><<a href="mailto:brian@python.org" target="_blank">brian@python.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>On Fri, Mar 20, 2015 at 7:54 PM, Neil Girdhar <<a href="mailto:mistersheik@gmail.com" target="_blank">mistersheik@gmail.com</a>> wrote:<br>
> If ever someone wants to clean up the repository to conform to PEP 7, I<br>
> wrote a program that catches a couple hundred PEP 7 violations in ./Python<br>
> alone (1400 in the whole codebase):<br>
><br>
> import os<br>
> import re<br>
><br>
> def grep(path, regex):<br>
>     reg_obj = re.compile(regex, re.M)<br>
>     res = []<br>
>     for root, dirs, fnames in os.walk(path):<br>
>         for fname in fnames:<br>
>             if fname.endswith('.c'):<br>
>                 path = os.path.join(root, fname)<br>
>                 with open(path) as f:<br>
>                     data = f.read()<br>
>                     for m in reg_obj.finditer(data):<br>
>                         line_number = sum(c == '\n'<br>
>                                           for c in data[:m.start()]) + 1<br>
>                         res.append("{}: {}".format(path, line_number))<br>
>     return res<br>
><br>
> for pattern in [<br>
>         r'^\s*\|\|',<br>
>         r'^\s*\&\&',<br>
>         r'} else {',<br>
>         r'\<return\s*\(',<br>
> ]:<br>
>     print("Searching for", pattern)<br>
>     print("\n".join(grep('.', pattern)))<br>
><br>
> In my experience, it was hard to write PEP 7 conforming code when the<br>
> surrounding code is inconsistent.<br>
<br>
</div></div>You can usually change surrounding code within reason if you want to<br>
add conforming code of your own, but there's little value and high<br>
risk in any mass change just to apply the style guidelines.<br>
</blockquote></div><br></div>
</div></div><br>_______________________________________________<br>
Python-Dev mailing list<br>
<a href="mailto:Python-Dev@python.org">Python-Dev@python.org</a><br>
<a href="https://mail.python.org/mailman/listinfo/python-dev" target="_blank">https://mail.python.org/mailman/listinfo/python-dev</a><br>
Unsubscribe: <a href="https://mail.python.org/mailman/options/python-dev/guido%40python.org" target="_blank">https://mail.python.org/mailman/options/python-dev/guido%40python.org</a><br>
<br></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature">--Guido van Rossum (<a href="http://python.org/~guido">python.org/~guido</a>)</div>
</div></div>