[sugar] [PATCH] sugar pylint (vol2)

Marco Pesenti Gritti mpgritti
Tue Apr 22 09:03:47 EDT 2008


On Tue, Apr 22, 2008 at 2:47 PM, Simon Schampijer <simon at schampijer.de> wrote:
> Hi,
>
>  two more patches for pylint. One thing that I have not solved yet in the
> lint_intro patch is the problem that pylint is not aware of members of the
> window class. I think that is the case because the gtk.gdk.window is only
> available when the window is realized.
>
>  - window xid: shell/intro/glive.py
>   - Class 'window' has no 'xid' member
>   - other examples: set_type_hint, set_accept_focus...

I worked these around with a patch to our pylint. If you do a clean
jhbuild you should have these solved.

>  Another issue I have quite often is the one of unused variables due to
> returned tuples and lists where not all the variables are used in the code.
>
>  examaple from src/view/frame/clipboardpanelwindow.py:
>
>  scheme, netloc, path, parameters, query, fragment = urlparse.urlparse(uri)
>
>  on_disk = (scheme == 'file')
>
>  What about doing here, or any other options?
>  scheme = urlparse.urlparse(uri)[0]

That's what I've been doing. Better ideas welcome.

Marco



More information about the Sugar-devel mailing list