Gnus development mailing list
 help / color / mirror / Atom feed
From: Lars Magne Ingebrigtsen <larsi@gnus.org>
To: Matt Ford <matt@dancingfrog.co.uk>
Cc: ding@gnus.org
Subject: Re: Bug with gnus agent when processing message queue and having archive alist
Date: Wed, 11 Apr 2012 00:38:47 +0200	[thread overview]
Message-ID: <m3ehrvqkjs.fsf@stories.gnus.org> (raw)
In-Reply-To: <87fwcbqodu.fsf@dancingfrog.co.uk> (Matt Ford's message of "Tue, 10 Apr 2012 22:15:57 +0100")

Matt Ford <matt@dancingfrog.co.uk> writes:

> Here it is: the string "Manchester:" is the first regex of my alist.
> There's no check before the string-match to see if group has actually
> been defined which is why forcing it "" works...

I've applied the patch below to No Gnus.  Does it fix the problem?

> It's easy to invoke the bug, in title mode use the arrow keys to move
> the cursor to a title heading and `C-u m'.  One sees it also on return
> from unplugged mode.

Title heading?

diff --git a/lisp/gnus-msg.el b/lisp/gnus-msg.el
index 500ace9..507f040 100644
--- a/lisp/gnus-msg.el
+++ b/lisp/gnus-msg.el
@@ -1670,11 +1670,13 @@ this is a reply."
 			(not
 			 (setq result
 			       (cond
-				((stringp (caar var))
+				((and group
+				      (stringp (caar var)))
 				 ;; Regexp.
 				 (when (string-match (caar var) group)
 				   (cdar var)))
-				((functionp (car var))
+				((and group
+				      (functionp (car var)))
 				 ;; Function.
 				 (funcall (car var) group))
 				(t


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



  reply	other threads:[~2012-04-10 22:38 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-06 16:04 Matt Ford
2012-04-06 18:04 ` Matt Ford
2012-04-06 19:39   ` Matt Ford
2012-04-10 18:49     ` Lars Magne Ingebrigtsen
2012-04-10 21:15       ` Matt Ford
2012-04-10 22:38         ` Lars Magne Ingebrigtsen [this message]
2012-04-10 22:48           ` Matt Ford
2012-04-10 22:52             ` Lars Magne Ingebrigtsen
2012-04-12 10:25               ` Matt Ford
2012-04-12 18:35                 ` Lars Magne Ingebrigtsen
2012-04-10 18:48 ` Lars Magne Ingebrigtsen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=m3ehrvqkjs.fsf@stories.gnus.org \
    --to=larsi@gnus.org \
    --cc=ding@gnus.org \
    --cc=matt@dancingfrog.co.uk \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).