Re: Problem with gnus-registry-grep-in-list

[Ted Zlatanov <tzz@lifelogs.com>]

On Mon, 05 May 2008 19:38:56 +0100 Joao Cachopo <joao@cachopo.org> wrote: 

JC> The string-match function is called with the regexp in the second
JC> argument, rather than the first.

Fixed, please test.  I used catch/throw now instead of mapcar, it should
be more efficient.

JC> Actually, according to the CVS logs, this was a deliberate change made
JC> in revision 7.37:

JC>    date: 2007/09/12 09:16:25;  author: tzz;  state: Exp;  lines: +3 -3
JC>    (gnus-registry-unfollowed-groups): Add INBOX to the
JC>    list of groups not followed by default.  Fix type to be regexp.
JC>    (gnus-registry-grep-in-list): Fix inverted parameters to string-match.

My mistake, sorry.

JC> As the gnus-registry-grep-in-list function is used in one more place
JC> (the function gnus-registry-find-keywords), I'm not sure that swapping
JC> the arguments of the string-match again will not break this other
JC> functionality.

I doubt anyone uses gnus-registry-find-keywords, but I fixed it anyhow
to just use the member function instead of the regular expression
matching.

Ted