You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by "Andrew S. Tomlinson" <an...@excelsis.ca> on 2010/12/22 20:25:19 UTC

CamelProcessor broken

Constructor does not assign processor field.
Correct code should be:

public CamelProcessor(Mailet mailet, Log logger, CamelMailetProcessor
container) {
        this.mailet = mailet;
        this.logger = logger;
        this.processor = container;
    }


Andrew S. Tomlinson 




---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org


Re: CamelProcessor broken

Posted by Norman Maurer <no...@apache.org>.
Hi there,

you are right.. Thx for the review. I just committed a fix with some
refactoring in the mailetcontainer-* code. I think I'm done now

Bye,
Norman


2010/12/22 Andrew S. Tomlinson <an...@excelsis.ca>:
> Constructor does not assign processor field.
> Correct code should be:
>
> public CamelProcessor(Mailet mailet, Log logger, CamelMailetProcessor
> container) {
>        this.mailet = mailet;
>        this.logger = logger;
>        this.processor = container;
>    }
>
>
> Andrew S. Tomlinson
>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
> For additional commands, e-mail: server-dev-help@james.apache.org
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org