Re: [Gramps-devel] Rev 14135: lots of small edits

[Brian M. <br...@gr...>]

Gerald,

> I just committed rec 14135 with lots
> of small edits aimed at
> simplifying the code by taking advantage of built-in
> functions where
> possible.  For example:
> 
> old code:
> 
>     for x in y:
>          f(x)
> 
> now becomes:
> 
>     map(f, y)
> 

Interesting. I just complained at Doug the other day that he was using map() instead of list comprehensions.

Using map() reduces the pylint score. See the discussion here:
http://bytes.com/topic/python/answers/552954-pylint-whats-wrong-builtin-map

The complete pylint message is:
W0141:	Used builtin function %r Used when a black listed builtin function is used (see the bad-function option). Usual black listed functions are the ones like map, or filter , where Python offers now some cleaner alternative like list comprehension.

Should we switch to using list comprehensions?

~BM