Absolutely go for it.

     Richard

Sent from my expensive toy

On 17/02/2009, at 9:27 AM, Stefan Seefeld <issues@roundup-tracker.org>  
wrote:

>
> New submission from Stefan Seefeld <stefan@codesourcery.com>:
>
> The roundup "filter" method (see hyperdb.py) has a "search_matches"
> parameter that can be used to constrain the search.  In particular,
> only elements which otherwise match the search *and* are in
> search_matches will be considered.
>
> Conceptually, 'search_matches' is a set. However, as some backend code
> calls 'has_key()' on it, it needs to be a dictionary.
>
> The attached patch simplifies this, so any sequence type may be used.
>
> OK to check in ?
>
> ----------
> assignee: stefan
> files: search_matches.patch
> keywords: patch
> messages: 3550
> nosy: richard, stefan
> severity: normal
> status: new
> title: Simplify 'search_matches' argument to filter()
> type: rfe
> versions: devel
>
> ________________________________________________
> Roundup tracker <issues@roundup-tracker.org>
> <http://issues.roundup-tracker.org/issue2550505>
> ________________________________________________