On 7/31/07, Simon McVittie <[EMAIL PROTECTED]> 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 _______________________________________________ Sugar mailing list [email protected] http://lists.laptop.org/listinfo/sugar
