➜

This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author methane
Recipients IanLee1521, Rosuav, berker.peksag, gvanrossum, methane
Date 2016-04-15.15:50:34
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <CAEfz+TyKnh4jiai-MUKzuXfP=hQr6y2rUmaiXGYA+n4v5R0+hQ@mail.gmail.com>
In-reply-to <1460730710.57.0.504506870228.issue26763@psf.upfronthosting.co.za>
Content 
On Fri, Apr 15, 2016 at 11:31 PM, Guido van Rossum <report@bugs.python.org>
wrote:

>
> Guido van Rossum added the comment:
>
> I don't like the way "):" is indented in that example.

Me too.
I haven't know about this PEP8 update.
https://github.com/python/peps/commit/843b7d2cdb954dc0aa5ea70f140db2bd42943e1f

> I propose:
>
>             if ((width == 0
>                  and height == 0
>                  and color == 'red'
>                  and emphasis == 'strong')
>                 or highlight > 100):
>
>
Looks better to me.  But how about this?

if (width == 0
        and height == 0
        and color == 'red'
        and emphasis == 'strong'
    or highlight > 100):

Changing indentation level based on operator precedence is acceptable?
History
Date User Action Args
2016-04-15 15:50:34methanesetrecipients: + methane, gvanrossum, Rosuav, berker.peksag, IanLee1521
2016-04-15 15:50:34methanelinkissue26763 messages
2016-04-15 15:50:34methanecreate