I think this gross bit of code ProcessorDefinition defn = (ProcessorDefinition) this; if (defn instanceof TryDefinition) {
was needed to compile on AIX. I can't recall why this was needed so it may not be an issue anymore but I guess you'll see in the next AIX build :) On Thu, Jun 3, 2010 at 4:38 PM, <gno...@apache.org> wrote: > Author: gnodet > Date: Thu Jun 3 19:08:36 2010 > New Revision: 951115 > > URL: http://svn.apache.org/viewvc?rev=951115&view=rev > Log: > Remove a few ide warnings > > Modified: > > > camel/trunk/camel-core/src/main/java/org/apache/camel/model/ProcessorDefinition.java > > Modified: > camel/trunk/camel-core/src/main/java/org/apache/camel/model/ProcessorDefinition.java > URL: > http://svn.apache.org/viewvc/camel/trunk/camel-core/src/main/java/org/apache/camel/model/ProcessorDefinition.java?rev=951115&r1=951114&r2=951115&view=diff > > ============================================================================== > --- > camel/trunk/camel-core/src/main/java/org/apache/camel/model/ProcessorDefinition.java > (original) > +++ > camel/trunk/camel-core/src/main/java/org/apache/camel/model/ProcessorDefinition.java > Thu Jun 3 19:08:36 2010 > @@ -997,8 +997,7 @@ public abstract class ProcessorDefinitio > // when using doTry .. doCatch .. doFinally we should always > // end the try definition to avoid having to use 2 x end() in the > route > // this is counter intuitive for end users > - ProcessorDefinition defn = (ProcessorDefinition) this; > - if (defn instanceof TryDefinition) { > + if (this instanceof TryDefinition) { > popBlock(); > } > > @@ -1370,7 +1369,7 @@ public abstract class ProcessorDefinitio > * Creates a routing slip allowing you to route a message consecutively > through a series of processing > * steps where the sequence of steps is not known at design time and > can vary for each message. > * > - * @param expresion to decide the destinations > + * @param expression to decide the destinations > * @param uriDelimiter is the delimiter that will be used to split up > * the list of URIs in the routing slip. > * > @@ -1389,7 +1388,7 @@ public abstract class ProcessorDefinitio > * <p> > * The list of URIs will be split based on the default delimiter {...@link > RoutingSlipDefinition#DEFAULT_DELIMITER} > * > - * @param expresion to decide the destinations > + * @param expression to decide the destinations > * > * @return the builder > */ > @@ -2186,7 +2185,7 @@ public abstract class ProcessorDefinitio > */ > public ExpressionClause<ProcessorDefinition<Type>> transform() { > ExpressionClause<ProcessorDefinition<Type>> clause = > - new > ExpressionClause<ProcessorDefinition<Type>>((ProcessorDefinition<Type>) > this); > + new ExpressionClause<ProcessorDefinition<Type>>(this); > TransformDefinition answer = new TransformDefinition(clause); > addOutput(answer); > return clause; > > > -- Cheers, Jon Camel in Action: http://manning.com/ibsen Blog: http://janstey.blogspot.com