[issue232] PATCH gtk/process.py cleanup

David Farning at Labix Tracker tracker at labix.org
Tue Oct 24 10:54:04 PDT 2006


David Farning <dfarning at gmail.com> added the comment:

Sorry if anyone has gotten a previous copy of this message.  I tried replying
via the mailing list and got a bunch of message not accepted errors.  I think
the Re: [issue232] [PATCH] subject link was causing some problems.

I am interested in working on the various interfaces [Text|gt|qt] for
smartpm.  Seems like a good way to get my feet wet without getting into
the dependency solver and backend stuff.

In order to start adding features and fixing bugs, I would like to
enhance the readability of the existing codebase.

A very simple readability fix would be to replace self.progress with
self.progressbar in /gtk/progress.  According to HACKING, private
variables should be _progress. Calling the gtk.ProgressBar self.progress
rather than progressbar or _progress is confusing.

Dave

----------
status: resolved -> chatting
title: [PATCH] gtk/process.py cleanup -> PATCH gtk/process.py cleanup

_______________________________________
Labix issue tracker <tracker at labix.org>
<http://tracker.labix.org/issue232>
_______________________________________



More information about the Smart mailing list