[Bugs] #1317 UNSP: check for Rainbow binaries before trying to use them
Sugar Labs Bugs
bugtracker-noreply at sugarlabs.org
Sat Oct 3 07:42:10 EDT 2009
#1317: check for Rainbow binaries before trying to use them
------------------------------------------+---------------------------------
Reporter: sascha_silbe | Owner: sascha_silbe
Type: defect | Status: accepted
Priority: Unspecified by Maintainer | Milestone: 0.86
Component: sugar-toolkit | Version: Git as of bugdate
Severity: Unspecified | Keywords: r?
Distribution: Unspecified | Status_field: Assigned
------------------------------------------+---------------------------------
Changes (by sascha_silbe):
* keywords: r! => r?
Comment:
Replying to [comment:8 tomeu]:
> {{{
> 247 dev_null = file('/dev/null', 'w')
> }}}
> Won't be leaking descriptors like this?
Nop, Python garbage collection should take care of it.
> I think that it's better to try to have one statement per line, rather
than cramming several stuff in just one statement. Intermediate variables
are a chance to communicate what your code is doing without having to use
comments or letting the reader guess it.
New patch attached.
--
Ticket URL: <http://bugs.sugarlabs.org/ticket/1317#comment:9>
Sugar Labs <http://sugarlabs.org/>
Sugar Labs bug tracking system
More information about the Bugs
mailing list