[sugar] Initial Security Patches

Marco Pesenti Gritti mpgritti
Tue Jul 31 11:59:41 EDT 2007


On 7/31/07, Simon McVittie <simon.mcvittie at collabora.co.uk> wrote:
> On Tue, 31 Jul 2007 at 11:00:01 +0200, Marco Pesenti Gritti wrote:
> > The os.path.exists('/etc/olpc-security') is a little ugly, let's at
> > least avoid doing it when it's not necessary. I think we can just make
> > self._rainbow None when '/etc/olpc-security' doesn't exist above and
> > just "if self._rainbow" here.
>
> "if self._rainbow is not None" is more idiomatic (and efficient).

Sounds good, would be nice if you could update the style guide...

http://wiki.laptop.org/go/Python_Style_Guide

Marco



More information about the Sugar-devel mailing list