] > pylint0.3.2 devel & common-0.4.2 devel: Some remarks (Logilab.org)

Email pylint0.3.2 devel & common-0.4.2 devel: Some remarks

from
to
Sylvain Thenault <syt at logilab dot fr>
subject
pylint0.3.2 devel & common-0.4.2 devel: Some remarks
date
2004/01/23 16:17
I found my company email address through Google on your pages:
    http://lists.logilab.org/pipermail/python-projects/2003-October/000016.html
    http://lists.logilab.org/pipermail/python-projects/2003-October.txt

During my Christmas Holidays Spammers sent more than 1200 messages to me. From
almost no Spam in the beginning of last year to more than 30 Spams per day
now. The only thing I can do is to keep my addresses in the web to a minimum
so I kindly ask you to erase or scramble my email address if you publish
our correspondence.

With regards

Wolfgang Grafen


> W:  6:Messenger: Class has no __init__ method
Isn't this too paranoid?

> W: 13:Messenger.__call__: Use * or ** magic
On purpose. That is what this construct is for.

> W:  0:Requester.register: Comma not followed by a space
>         self.request_dict.setdefault(address,{req_name:handler})
>                                            ^^
What is the message?? which line??

> E: 36:Cntr_class.get_cntr: Class method doesn't have "cls" as first argument
I used this construct for the first time but it seems working for me

> W: 50:A: Invalid name "A" (should match [A-Z][a-zA-Z1-9]+)
As it is working it is obviously not invalid. Change '+' to '*'

> W:  0: Comma not followed by a space
> m.register("class_created",c)
???

> MESSAGE ID  OCCURENCES  
> W0324       30 
You mentioned my name so I found pylint again through Google search.
My imagination was to print out

  W0324-1-22:Requester.__call__: Comma not followed by a space

continuing with W0324-1-23 and so on if continuing lines refer to the same
message. With the meaning <MSG><ID>-<CNT>-<LINENO>.
Then you can filter with grep

pylint instcntr5.py |grep W0324

or suppress
pylint instcntr5.py |grep -v W0324

and so on.



> Global evaluation
> -----------------
>
> Your code has been rated at -0.16/10 (previous run: -0.16/10)
Minus means even worse than worse?


has reply