Noel J. Bergman wrote:
+++ james/server/trunk/src/java/org/apache/james/transport/LinearProcessor.java // and store it in the next spot
                 Mail notMail = new MailImpl(mail,newName(mail));
                 notMail.setRecipients(notRecipients);
+                notMail.setState(mail.getState());

Should that go into the MailImpl(Mail, String) constructor, instead?

        --- Noel

I evaluated the 2 options and decided to put this in the linear processor.

In fact we call the "duplicator" constructor in many places but only from linear processor we have really the "state" concept. Other times we simply clone that mail to redirect/forward/bounce it and we don't want to bring the status but start from the root processor.

Stefano


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

Reply via email to