[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