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. Because a) operator.add is more clear to me and b) adding the spaces would have required breaking the line, making it harder to read. Sascha -- http://sascha.silbe.org/ http://www.infra-silbe.de/
signature.asc
Description: PGP signature
_______________________________________________ Sugar-devel mailing list [email protected] http://lists.sugarlabs.org/listinfo/sugar-devel
