Roundup Tracker - Issues

Message3652

Author stefan
Recipients loewis, richard, stefan
Date 2009-03-17.06:34:44
Message-id <49BE6376.40709@codesourcery.com>
In-reply-to <1237168332.89.0.295542555485.issue2550505@psf.upfronthosting.co.za>
Martin v. Löwis @psf.upfronthosting.co.za wrote:

> The patch is incorrect. It leaves the line
> 
>   args = args + v
> 
> even though the variable v has no value anymore.

I'd propose this fix:

-            args = args + v
+            args = args + [x for x in search_matches]

as the intention of the original patch was to use 'in' so any container 
type would be possible. Richard, how does this look ?

Thanks,
		Stefan
History
Date User Action Args
2009-03-17 06:34:44stefansetrecipients: + stefan, richard, loewis
2009-03-17 06:34:44stefanlinkissue2550505 messages
2009-03-17 06:34:44stefancreate