GNU bug report logs - #12375
Broken matching of regexps in fancy splitting

Previous Next

Packages: emacs, gnus;

Reported by: jathd <jathdr <at> gmail.com>

Date: Fri, 7 Sep 2012 05:31:02 UTC

Severity: normal

Tags: fixed

Done: Lars Ingebrigtsen <larsi <at> gnus.org>

Bug is archived. No further changes may be made.

Full log


Message #20 received at 12375 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: jathd <jathdr <at> gmail.com>
Cc: 12375 <at> debbugs.gnu.org
Subject: Re: bug#12375: Broken matching of regexps in fancy splitting
Date: Thu, 30 Jan 2014 15:37:49 -0800
jathd <jathdr <at> gmail.com> writes:

> The `value' is the regexp I specified in `nnmail-split-fancy'. Before
> that, to see whether a header matches, the function does
>
>     (re-search-backward (cdr cached-pair) nil t)
>
> where (cdr cached-pair) is a regexp it constructed the first time
> around. Since it prepended a "\\<" to *that* regexp, the header is
> correctly identified; but it didn't prepend anything to `value', so the
> extraction goes wrong.

Aha.

I've now rewritten that function to save the match data explicitly after
doing the first match so that we don't have to redo the search.  It
should be faster, too.

Does this fix the problem for you?  It's in git Gnus, but should be in
bzr Emacs soon...

-- 
(domestic pets only, the antidote for overdose, milk.)
  bloggy blog http://lars.ingebrigtsen.no/




This bug report was last modified 8 years and 176 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.