Gnus development mailing list
 help / color / mirror / Atom feed
From: Simon Josefsson <jas@extundo.com>
Subject: Re: Fancy splitting getting headers from included text
Date: Sun, 30 Mar 2003 14:08:22 +0200	[thread overview]
Message-ID: <ilur88pcb0p.fsf@latte.josefsson.org> (raw)
In-Reply-To: <843cl5i4kg.fsf@lucy.is.informatik.uni-duisburg.de> (kai.grossjohann@gmx.net's message of "Sun, 30 Mar 2003 11:31:11 +0200")

kai.grossjohann@gmx.net (Kai Großjohann) writes:

> Lars Magne Ingebrigtsen <larsi@gnus.org> writes:
>
>> It shouldn't -- the buffer should be limited to the headers only.
>> (Unless somebody changed that while I wasn't looking.  If so, they
>> should change it back again.  :-)
>
> Some people needed the body for spam processing.  There is a variable
> for this.  Hmm...  Ah.  nnimap-split-download-body, perhaps?

But it is off by default, and it is easy to see if the body is
incorrectly downloaded anyway in *imap-log*.

I think it was reported to happen for nnmail too.  Hm.  Those who are
seeing this, are you using spam.el and downloading bodies too?  Could
you edebug nnmail-article-group to see if nnmail-split-fancy is
somehow invoked without narrowed headers?  I'm not sure where the spam
code is invoked, but if it does a (widen) it should probably do it
inside a save-restriction.  Does this patch help?

--- spam.el.~6.87.~	Thu Mar 20 00:10:30 2003
+++ spam.el	Sun Mar 30 14:08:05 2003
@@ -595,7 +595,7 @@
 
 See the Info node `(gnus)Fancy Mail Splitting' for more details."
   (interactive)
-
+  (save-excursion
   (dolist (check spam-list-of-statistical-checks)
     (when (symbol-value check)
       (widen)
@@ -612,7 +612,7 @@
 	  (setq decision (funcall (cdr pair))))))
     (if (eq decision t)
 	nil
-      decision)))
+	decision))))
 
 (defun spam-setup-widening ()
   (dolist (check spam-list-of-statistical-checks)




  reply	other threads:[~2003-03-30 12:08 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-03-25 19:02 David Z Maze
2003-03-26  8:21 ` Niklas Morberg
2003-03-30  2:47 ` Lars Magne Ingebrigtsen
2003-03-30  9:31   ` Kai Großjohann
2003-03-30 12:08     ` Simon Josefsson [this message]
2003-04-05 13:29       ` Ted Zlatanov
2003-04-14 15:53       ` David Z Maze
2003-04-14 16:33         ` Simon Josefsson
2003-04-14 20:25           ` David Z Maze
2003-04-15 17:44             ` Ted Zlatanov
2003-04-15 18:12               ` David Z Maze
2003-04-15 19:47                 ` Ted Zlatanov
2003-05-07 14:27                   ` Niklas Morberg
2003-06-11  9:23                     ` Niklas Morberg
2003-06-13  8:33                       ` Niklas Morberg
2003-06-13  9:35                         ` Niklas Morberg
2003-06-13  9:50                           ` [PATCH] " Niklas Morberg
2003-06-13 21:07                             ` Kai Großjohann

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=ilur88pcb0p.fsf@latte.josefsson.org \
    --to=jas@extundo.com \
    /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).