You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by ni...@apache.org on 2008/11/11 14:17:52 UTC

svn commit: r713030 - in /activemq/camel/trunk: camel-core/ camel-core/src/main/java/org/apache/camel/model/ camel-core/src/main/java/org/apache/camel/util/ camel-core/src/test/java/org/apache/camel/converter/ components/camel-http/src/main/java/org/ap...

Author: ningjiang
Date: Tue Nov 11 05:17:51 2008
New Revision: 713030

URL: http://svn.apache.org/viewvc?rev=713030&view=rev
Log:
Fixed the CS errors also reenabled the DataSetSedaTest

Modified:
    activemq/camel/trunk/camel-core/pom.xml
    activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/model/AggregatorType.java
    activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/util/URISupport.java
    activemq/camel/trunk/camel-core/src/test/java/org/apache/camel/converter/ObjectHelperTest.java
    activemq/camel/trunk/components/camel-http/src/main/java/org/apache/camel/component/http/HttpProducer.java
    activemq/camel/trunk/components/camel-mail/src/main/java/org/apache/camel/component/mail/MailBinding.java

Modified: activemq/camel/trunk/camel-core/pom.xml
URL: http://svn.apache.org/viewvc/activemq/camel/trunk/camel-core/pom.xml?rev=713030&r1=713029&r2=713030&view=diff
==============================================================================
--- activemq/camel/trunk/camel-core/pom.xml (original)
+++ activemq/camel/trunk/camel-core/pom.xml Tue Nov 11 05:17:51 2008
@@ -136,8 +136,7 @@
           <forkMode>pertest</forkMode>
           <excludes>
             <!-- TODO FIXME ASAP -->
-            <exclude>**/XXXTest.*</exclude>
-            <exclude>**/DataSetSedaTest.*</exclude>
+            <exclude>**/XXXTest.*</exclude>            
           </excludes>
         </configuration>
       </plugin>

Modified: activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/model/AggregatorType.java
URL: http://svn.apache.org/viewvc/activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/model/AggregatorType.java?rev=713030&r1=713029&r2=713030&view=diff
==============================================================================
--- activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/model/AggregatorType.java (original)
+++ activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/model/AggregatorType.java Tue Nov 11 05:17:51 2008
@@ -170,8 +170,8 @@
             AggregationStrategy strategy = createAggregationStrategy(routeContext);
 
             if (getExpression() == null) {
-                throw new IllegalArgumentException("You need to specify an expression or " + 
-                                                   "aggregation collection for this aggregator: " + this);
+                throw new IllegalArgumentException("You need to specify an expression or "
+                                                   + "aggregation collection for this aggregator: " + this);
             }
             
             Expression aggregateExpression = getExpression().createExpression(routeContext);           

Modified: activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/util/URISupport.java
URL: http://svn.apache.org/viewvc/activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/util/URISupport.java?rev=713030&r1=713029&r2=713030&view=diff
==============================================================================
--- activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/util/URISupport.java (original)
+++ activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/util/URISupport.java Tue Nov 11 05:17:51 2008
@@ -32,7 +32,11 @@
  *
  * @version $Revision$
  */
-public class URISupport {
+public final class URISupport {
+    
+    private URISupport() {
+        // Helper class
+    }
 
     /**
      * Holder to get parts of the URI.

Modified: activemq/camel/trunk/camel-core/src/test/java/org/apache/camel/converter/ObjectHelperTest.java
URL: http://svn.apache.org/viewvc/activemq/camel/trunk/camel-core/src/test/java/org/apache/camel/converter/ObjectHelperTest.java?rev=713030&r1=713029&r2=713030&view=diff
==============================================================================
--- activemq/camel/trunk/camel-core/src/test/java/org/apache/camel/converter/ObjectHelperTest.java (original)
+++ activemq/camel/trunk/camel-core/src/test/java/org/apache/camel/converter/ObjectHelperTest.java Tue Nov 11 05:17:51 2008
@@ -18,9 +18,8 @@
 
 import java.util.Iterator;
 
-import org.apache.camel.util.ObjectHelper;
-
 import junit.framework.TestCase;
+import org.apache.camel.util.ObjectHelper;
 
 /**
  * @version $Revision: 640438 $

Modified: activemq/camel/trunk/components/camel-http/src/main/java/org/apache/camel/component/http/HttpProducer.java
URL: http://svn.apache.org/viewvc/activemq/camel/trunk/components/camel-http/src/main/java/org/apache/camel/component/http/HttpProducer.java?rev=713030&r1=713029&r2=713030&view=diff
==============================================================================
--- activemq/camel/trunk/components/camel-http/src/main/java/org/apache/camel/component/http/HttpProducer.java (original)
+++ activemq/camel/trunk/components/camel-http/src/main/java/org/apache/camel/component/http/HttpProducer.java Tue Nov 11 05:17:51 2008
@@ -16,8 +16,6 @@
  */
 package org.apache.camel.component.http;
 
-import static org.apache.camel.component.http.HttpMethods.HTTP_METHOD;
-
 import java.io.IOException;
 import java.io.InputStream;
 import java.io.UnsupportedEncodingException;
@@ -42,6 +40,8 @@
 import org.apache.commons.io.IOUtils;
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
+
+import static org.apache.camel.component.http.HttpMethods.HTTP_METHOD;
 /**
  * @version $Revision$
  */

Modified: activemq/camel/trunk/components/camel-mail/src/main/java/org/apache/camel/component/mail/MailBinding.java
URL: http://svn.apache.org/viewvc/activemq/camel/trunk/components/camel-mail/src/main/java/org/apache/camel/component/mail/MailBinding.java?rev=713030&r1=713029&r2=713030&view=diff
==============================================================================
--- activemq/camel/trunk/components/camel-mail/src/main/java/org/apache/camel/component/mail/MailBinding.java (original)
+++ activemq/camel/trunk/components/camel-mail/src/main/java/org/apache/camel/component/mail/MailBinding.java Tue Nov 11 05:17:51 2008
@@ -21,6 +21,7 @@
 import java.util.HashMap;
 import java.util.Iterator;
 import java.util.Map;
+
 import javax.activation.DataHandler;
 import javax.activation.DataSource;
 import javax.mail.Address;
@@ -37,11 +38,11 @@
 
 import org.apache.camel.Exchange;
 import org.apache.camel.RuntimeCamelException;
-import org.apache.camel.util.ObjectHelper;
 import org.apache.camel.converter.ObjectConverter;
 import org.apache.camel.impl.DefaultHeaderFilterStrategy;
 import org.apache.camel.spi.HeaderFilterStrategy;
 import org.apache.camel.util.CollectionHelper;
+import org.apache.camel.util.ObjectHelper;
 
 /**
  * A Strategy used to convert between a Camel {@link Exchange} and {@link Message} to and