[Python-Dev] cpython: Remove some extraneous parentheses and swap the comparison order to
solipsis at pitrou.net
Tue Jun 7 10:25:01 CEST 2011
On Tue, 07 Jun 2011 08:57:10 +0200
Georg Brandl <g.brandl at gmx.net> wrote:
> On 06/07/11 05:20, brett.cannon wrote:
> > http://hg.python.org/cpython/rev/fc282e375703
> > changeset: 70695:fc282e375703
> > user: Brett Cannon <brett at python.org>
> > date: Mon Jun 06 20:20:36 2011 -0700
> > summary:
> > Remove some extraneous parentheses and swap the comparison order to
> > prevent accidental assignment.
> > Silences a warning from LLVM/clang 2.9.
> Swapping the comparison order here seems a bit inconsistent to me. There are
> lots of others around (e.g. "len == 0" in the patch context below). Why is
> this one so special?
Agreed. Either we do it wholesale (I find these "reversed" comparisons
a bit ugly myself) or there's no point doing it on a single occurrence.
More information about the Python-Dev