camel-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "James Strachan" <james.strac...@gmail.com>
Subject Re: Question about QuartzEndpoint.java
Date Tue, 10 Jun 2008 13:28:41 GMT
2008/6/10 MeBigFatGuy <dbrosius@apache.org>:
>
> <code>
>    @Override
>    public QuartzExchange createExchange(ExchangePattern pattern) {
>        return new QuartzExchange(getCamelContext(), pattern, null);
>    }
> </code>
> <code>
>    public QuartzExchange(CamelContext context, ExchangePattern pattern,
> JobExecutionContext jobExecutionContext) {
>        super(context, pattern);
>        setIn(new QuartzMessage(this, jobExecutionContext));
>    }
> </code>
> <code>
>    public QuartzMessage(QuartzExchange exchange, JobExecutionContext
> jobExecutionContext) {
>        this.jobExecutionContext = jobExecutionContext;
>        setExchange(exchange);
>        setBody(jobExecutionContext.getJobDetail());
>    }
> </code>
>
> looks to me that null in the first code block, gets passed down in the
> QuartzMessage ctor, and is dereferenced there. Seems like it always creates
> NPEs, to me.

Great catch! Patch applied with thanks
-- 
James
-------
http://macstrac.blogspot.com/

Open Source Integration
http://open.iona.com

Mime
View raw message