<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Apr 15, 2016 at 10:03 AM, Victor Stinner <span dir="ltr"><<a href="mailto:victor.stinner@gmail.com" target="_blank">victor.stinner@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hum.<br>
<span class=""><br>
        if (width == 0<br>
            and height == 0<br>
            and color == 'red'<br>
            and emphasis == 'strong'<br>
            or highlight > 100):<br>
            raise ValueError("sorry, you lose")<br>
<br>
</span>Please remove one space to vertically align "and" operators with the<br>
opening parenthesis:<br>
<span class=""><br>
        if (width == 0<br>
           and height == 0<br>
           and color == 'red'<br>
           and emphasis == 'strong'<br>
           or highlight > 100):<br>
            raise ValueError("sorry, you lose")<br>
<br>
</span>(I'm not sure that the difference is obvious in a mail client, you<br>
need a fixed width font which is not the case in my Gmail editor.)<br></blockquote><div><br></div><div>I can see it perfectly fin and I disagree.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
It helps to visually see that the multiline test and the raise<br>
instruction are in two different blocks.<br>
<br>
(Moreover, the pep8 checks of OpenStack simply reject such syntax, but<br>
I cannot use this syntax anymore :-))</blockquote><div><br></div><div>That's why that tool shouldn't be named after the PEP. See <a href="https://github.com/PyCQA/pycodestyle/issues/466">https://github.com/PyCQA/pycodestyle/issues/466</a> <br></div></div><br>-- <br><div class="gmail_signature">--Guido van Rossum (<a href="http://python.org/~guido" target="_blank">python.org/~guido</a>)</div>
</div></div>