[prev in list] [next in list] [prev in thread] [next in thread] 

List:       mutt-dev
Subject:    Re: Forwarding with references
From:       Aaron Schrab <aaron+mutt () schrab ! com>
Date:       2001-03-26 10:49:25
[Download RAW message or body]

At 12:10 +0200 26 Mar 2001, Martin Trautmann <traut@gmx.de> wrote:
> My wish is, that forwarded message shall include references. I feel this is
> very useful, since a forwarded message still belongs to the current thread
> and should be kept within this thread.

> The mechanism is already in there, used for replies.
> Is there any special reason against the usage of references for forward,
> other than the argument that a forward is no reply? (I feel, a forward is a
> special kind of reply).

The main reason I can think of is that it wasn't thought to be useful.
Unlike a reply, a forward generally goes to people who didn't receive
the original message(s) so the recpient(s) wouldn't be able to see the
thread structure anyway (also for this reason I'd disagree about forward
being a special kind of reply).

The attached patch modifies mutt so that it will include a References:
header when forwarding messages.  An In-Reply-To: header is not
included, since it's not a reply.

-- 
Aaron Schrab     aaron@schrab.com      http://www.execpc.com/~aarons/
 As I look across the web, what I find is this vast wasteland that
 makes television almost attractive.  -- Clifford Stoll

["ats.forw_ref.diff" (text/plain)]

--- send.c.dist	Mon Mar 26 04:22:42 2001
+++ send.c	Mon Mar 26 04:33:44 2001
@@ -667,7 +667,10 @@
 
     mutt_fix_reply_recipients (env);
     mutt_make_misc_reply_headers (env, ctx, cur, curenv);
-    
+  }
+
+  if (flags & (SENDREPLY | SENDFORWARD))
+  {
     if(tag)
     {
       HEADER *h;
@@ -686,9 +689,9 @@
     }
     else
       env->references = mutt_make_references(curenv);
-
   }
-  else if (flags & SENDFORWARD)
+
+  if (flags & SENDFORWARD)
     mutt_make_forward_subject (env, ctx, cur);
 
   return (0);


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic