On Mon, Sep 21, 2015 at 02:24:02PM -0400, Derek Schrock wrote:
> On Mon, Sep 21, 2015 at 06:45:48PM +0800, Kevin J. McCarthy wrote:
> > On Sun, Sep 20, 2015 at 11:16:20PM -0400, Derek Schrock wrote:
> > > I was going to make a small change to the expandos and make them
> > > optional.
> > 
> > Perhaps I'm misunderstanding, but I don't see it necessary to make the
> > expandos optional.  They don't change any behavior unless you actually
> > use them.  But feel free to resubmit your patches as you see fit.
> 
> I felt the expandos should be optional if the message doesn't have a To:
> or Cc: header then if you're using them in your attribution the messages
> looks cleaner without the two blank lines.
> 
> attribution="...%?r?To: %r\n?..."
> 
> In my case I thought about using some autocmd in vim to auto remove
> blank To:/Cc: lines but figured it was better to do it in mutt.
> 
> If something doesn't seem right about that let me know.

No, I did misunderstand.  :-)  That's just fine!

> Ok It's a single patch

I'll just give a day or two for any other comments and then I'll push it
up.  Thanks again for the patch!

-- 
Kevin J. McCarthy
GPG Fingerprint: 8975 A9B3 3AA3 7910 385C  5308 ADEF 7684 8031 6BDA
http://www.8t8.us/configs/gpg-key-transition-statement.txt

Attachment: signature.asc
Description: PGP signature

Reply via email to