Author: davsclaus
Date: Mon May 12 05:28:42 2008
New Revision: 655488
URL: http://svn.apache.org/viewvc?rev=655488&view=rev
Log:
CAMEL-433: using the new getAndRemoveParameter programming best practice
Modified:
activemq/camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsComponent.java
Modified:
activemq/camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsComponent.java
URL:
http://svn.apache.org/viewvc/activemq/camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsComponent.java?rev=655488&r1=655487&r2=655488&view=diff
==============================================================================
---
activemq/camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsComponent.java
(original)
+++
activemq/camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsComponent.java
Mon May 12 05:28:42 2008
@@ -330,9 +330,8 @@
try {
queueBrowseStrategy =
tryCreateDefaultQueueBrowseStrategy();
} catch (Throwable e) {
- LOG.warn(
- "Could not instantiate the QueueBrowseStrategy
are you using Spring 2.0.x by any chance? Error: "
- + e, e);
+ LOG.warn("Could not instantiate the QueueBrowseStrategy
are you using Spring 2.0.x"
+ + " by any chance? Error: " + e, e);
}
}
}
@@ -397,7 +396,7 @@
}
}
- String selector = (String)parameters.remove("selector");
+ String selector = getAndRemoveParameter(parameters, "selector",
String.class);
if (selector != null) {
endpoint.setSelector(selector);
}
@@ -427,7 +426,7 @@
* Attempts to instantiate the default [EMAIL PROTECTED]
QueueBrowseStrategy} which
* should work fine if Spring 2.5.x or later is on the classpath but this
* will fail if 2.0.x are on the classpath. We can continue to operate on
- * this version we just cannot support the browsable queues supported by
+ * this version we just cannot support the browseable queues supported by
* [EMAIL PROTECTED] JmsQueueEndpoint}
*
* @return the queue browse strategy or null if it cannot be supported
