Email Re: [Python-projects] pylint 0.9.2 problem with config file parsing

On Wednesday 23 February à 19:09, Kring Gerhard wrote:
> Dear Pylint developers,
> I just did a first test with pylint 0.9.2 and had the following problem:
> 
> In the lint.py module, line 273, method set_option
> the following statements try to parse a comma separated list of Messages, e.g. C0301,W0706,W0103 
> and fails because it takes 'C' instead of C0301 as the first item:
>                 for _id in value: 
>                     meth(_id)
> 
> I replaced this with
>                 values = value.split(',')     ##
>                 for _id in values:             ##
>                     meth(_id)
> and everything worked fine.
> 
> Is value a string or what should be the magic behind this?
 
hum, ok. I've introduced this bug with the latest logilab's common
release which introduce slight modifications in the options handling
module. Your fix sounds good enough for now, a minor amelioration would
be to use the get_csv function so that comma separated list with spaces
included will work to :

for _id in get_csv(value):
    meth(_id)

thanks for reporting this
-- 
Sylvain Thénault                               LOGILAB, Paris (France).

http://www.logilab.com   http://www.logilab.fr  http://www.logilab.org

_______________________________________________
Python-Projects mailing list
Python-Projects@lists.logilab.org
http://lists.logilab.org/mailman/listinfo/python-projects