Re: Formatting problem send_mail in version 2.10.0

From: Matthieu Moy <matthieu.moy@grenoble-inp.fr>
Date: Oct 13, 2016, 05:37 UTC
Message-ID: <vpqmvi8n71g.fsf@anie.imag.fr>
In-Reply-To: <20161012231321.ar2csdmvhdya2q7r@sigill.intra.peff.net>

Jeff King <peff@peff.net> writes:

>   2. Drop everything after the trailing ">". This gives a valid rfc2822
>      cc, and people can pick the "# 4.8" from the cc line in the body.

That would work for me, but it's inconsistent with Mail::Address and I'd really like to avoid having a behavior depending on libraries installed.

Plus, consistency with Mail::Address gives us consistency with any other program using Mail::Address.

Show 10 quoted lines

>   3. Rewrite
>
>        A <B@C> D
>
>      into
>
>        A D <B@C>
>
>      regardless of what is in "D". This retains the information in the
>      rfc2822 cc.

There's another one I considered:

   4. Consider '#.*' as a comment when parsing trailers (but not in
      other places where we parse addresses)

This is much harder to define properly because of

Cc: "Foo # Bar" <email@example.com>

=> we'd need to handle the ".*" syntax when stripping comments. And again, that wouldn't be consistent with Mail::Address.

So, I ended up implementing 3., which actually isn't hard, and gives code IMHO cleaner than it used to be.

-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/