Simon Kitching <[EMAIL PROTECTED]> writes:

>And as Troy Poppe mentions, if 35881 is being considered for 1.0 then
>perhaps 34056 should be as well.

For what it is worth, I'd -1 the patch by Troy until he fixed up the

[...]
        try {
            if (StringUtils.isNotEmpty(charset))
            {
//                BROKEN!
//                getPrimaryBodyPart().setText(msg, charset);
                getPrimaryBodyPart().setText(msg);
            }
            else
            {
                getPrimaryBodyPart().setText(msg);
            }
[...]

conditional in MultiPartEmail that his patch introduces. Either get
rid of the conditional or fix up setText(msg, charset). But that hurts
my eyes.

        Regards
                Henning

-- 
Dipl.-Inf. (Univ.) Henning P. Schmiedehausen          INTERMETA GmbH
[EMAIL PROTECTED]        +49 9131 50 654 0   http://www.intermeta.de/

RedHat Certified Engineer -- Jakarta Turbine Development  -- hero for hire
   Linux, Java, perl, Solaris -- Consulting, Training, Development

                      4 - 8 - 15 - 16 - 23 - 42

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to