<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 15 April 2016 at 18:03, 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-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;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></span></blockquote><div><br></div><div>Personally, I think what you propose looks ugly.  The first version looks so much better.</div><div><br></div><div><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">It helps to visually see that the multiline test and the raise<br>instruction are in two different blocks.</blockquote></div><div><br></div><div>The only thing I would add would be an empty line to help distinguish the if expression block from the "then" code block:</div><div><br></div><div><div><font face="monospace, monospace">        if (width == 0</font></div><div><font face="monospace, monospace">            and height == 0</font></div><div><font face="monospace, monospace">            and color == 'red'</font></div><div><font face="monospace, monospace">            and emphasis == 'strong'</font></div><div><font face="monospace, monospace">            or highlight > 100):</font></div><div><font face="monospace, monospace"><br></font></div><div><font face="monospace, monospace">            raise ValueError("sorry, you lose")</font></div></div><div><br></div></div><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Gustavo J. A. M. Carneiro<div>Gambit Research<br>"The universe is always one step beyond logic." -- Frank Herbert</div></div></div>
</div></div>