Now that we have started to see conditional expressions ... I would like to see them parenthesized. The parens aren't as important as they are with generator expressions, but ... they matter.<br><br>From a recent checkin --
<br><br> level = 0 if "absolute_import" in self.futures else -1<br><br>Mentally, I can't help parsing that as "level = 0" plus comments that turn out to be code that triggers backracking.<br><br>When the expressions (particularly the true case) are longer, it gets even worse.
<br><br>I think that adding parentheses would help, by at least signalling that the logic is longer than just the next (single) expression.<br><br> level = (0 if "absolute_import" in self.futures else -1)<br>
<br>-jJ<br><br><br><br>