On 3/6/06, Jim Jewett <firstname.lastname@example.org> wrote:
From a recent checkin --
level = 0 if "absolute_import" in self.futures else -1
Mentally, I can't help parsing that as "level = 0" plus comments that turn out to be code that triggers backracking.
When the expressions (particularly the true case) are longer, it gets even worse.
I think that adding parentheses would help, by at least signalling that the logic is longer than just the next (single) expression.
level = (0 if "absolute_import" in self.futures else -1)
I'm not sure, are you suggesting the grammar/parser enforces it that way, or that it should be in PEP-8?