[Patches] Review (was: Please review before applying)

Greg Stein gstein@lyra.org
Sat, 22 Apr 2000 12:16:02 -0700 (PDT)


On Sat, 22 Apr 2000, Christian Tismer wrote:
>...
> ---/+++ cgi.py, line 704
> you are changing
>     plist = map(string.strip, string.splitfields(line, ';'))
> into
>     plist = map(lambda s: s.strip(), line.split(';'))
> 
> I don't the extra lambda-ness and the extra frames generated here
> are an advantage.

No... use the lambda. string.strip is a function that is equal to that
lambda. Therefore, it is better to avoid the string module altogether and
to use the lambda.


Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/