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).

        Simon
_______________________________________________
Sugar mailing list
[email protected]
http://lists.laptop.org/listinfo/sugar

Reply via email to