Email RE: [Python-projects] Re: PyLint problems with qt

Thanks... I will be waiting for it... (I will do a pydev release next week
to).

[]s

Fabio Zadrozny
------------------------------------------------------
Software Developer
ESSS - Engineering Simulation and Scientific Software
www.esss.com.br


-----Original Message-----
From: Sylvain.Thenault@logilab.fr [mailto:Sylvain.Thenault@logilab.fr] 
Sent: quinta-feira, 27 de janeiro de 2005 13:26
To: Sylvain Thénault
Cc: Fabio Zadrozny; python-projects@logilab.org
Subject: Re: [Python-projects] Re: PyLint problems with qt

On Thursday 27 January à 16:47, Sylvain Thénault wrote:
> On Thursday 27 January à 12:41, Fabio Zadrozny wrote:
> > Hi Sylvain,
> 
> Hey Fabio,
> 
> long time you didn't recorded a pylint bug :)
>  
> > The code below is giving the error:
> > ID:W0221 ToElapse.timerEvent: Arguments number differs from 
> > overriden method
> > 
> > But it shouldn't, as the timerEvent signature is correct ( virtual 
> > void timerEvent ( QTimerEvent * ) ) And I couldn't make it work with 
> > less or more arguments.
> > 
> > import qt
> > 
> > class ToElapse(qt.QObject):
> > 
> >     def __init__(self):
> >         qt.QObject.__init__(self)
> >         
> >     def timerEvent(self, event):
> >         pass
> 
> ok, the problem was that with builtin methods, the astng builder is 
> unable to get arguments information, but the released code set 
> argnames (the attribute listing function/method arguments) to an empty 
> list instead of None (which informs that we have no arguments 
> information, not no arguments). That's why pylint was complaining...
> Attached a fixed builder correcting this problem, still replacing the 
> logilab/common/astng/builder.py file.

oops, it seems that this bug fix create another bug in some checkers.
If you want to use it, you'll have to use the attached checkers (replace the
logilab/pylint/checkers/*.py file with the attached ones).

A bug fixes release of common and pylint will probably run out next week.

--
Sylvain, happy to have built a nice test suite, even if I should have ran it
before sending this patch...