[Sugar-devel] [PATCH 05/21 sugar-toolkit] PEP8 cleanup: fix whitespace around operator
sascha-ml-reply-to-2010-3 at silbe.org
Fri Nov 19 11:24:50 EST 2010
Excerpts from Simon Schampijer's message of Fri Oct 29 12:46:54 +0200 2010:
> > I've left the powers as-is because IMO they should directly follow the base,
> > unlike other operators.
> Hmmm, I see your argument, however I think it is not worth to make an
> exception here.
I want to get this patch landed, so I just did it the way you want it
(i.e. added the spaces) for v2. ;)
> > - self.bundle_name = reduce(lambda x, y: x+y, self.activity_name.split())
> > + self.bundle_name = reduce(operator.add, self.activity_name.split())
> Why do you prefer operator.add here instead of doing the white spaces
> around the '+'? To me at least it does not get easier to read.
a) operator.add is more clear to me and
b) adding the spaces would have required breaking the line, making it
harder to read.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 490 bytes
Desc: not available
More information about the Sugar-devel