[1/4] cxf git commit: Don't use deprecated method of configuration

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[1/4] cxf git commit: Don't use deprecated method of configuration

Daniel  Kulp
Administrator
Repository: cxf
Updated Branches:
  refs/heads/master ff2a2793e -> b69bd4273


Don't use deprecated method of configuration


Project: http://git-wip-us.apache.org/repos/asf/cxf/repo
Commit: http://git-wip-us.apache.org/repos/asf/cxf/commit/baf0980c
Tree: http://git-wip-us.apache.org/repos/asf/cxf/tree/baf0980c
Diff: http://git-wip-us.apache.org/repos/asf/cxf/diff/baf0980c

Branch: refs/heads/master
Commit: baf0980cfaed32f5b69d87c0c727cde7fadd4159
Parents: ff2a279
Author: Daniel Kulp <[hidden email]>
Authored: Thu Aug 10 13:38:44 2017 -0400
Committer: Daniel Kulp <[hidden email]>
Committed: Thu Aug 10 13:38:44 2017 -0400

----------------------------------------------------------------------
 .../apache/cxf/maven_plugin/java2swagger/Java2SwaggerMojo.java    | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cxf/blob/baf0980c/maven-plugins/java2swagger-plugin/src/main/java/org/apache/cxf/maven_plugin/java2swagger/Java2SwaggerMojo.java
----------------------------------------------------------------------
diff --git a/maven-plugins/java2swagger-plugin/src/main/java/org/apache/cxf/maven_plugin/java2swagger/Java2SwaggerMojo.java b/maven-plugins/java2swagger-plugin/src/main/java/org/apache/cxf/maven_plugin/java2swagger/Java2SwaggerMojo.java
index a585366..6972907 100644
--- a/maven-plugins/java2swagger-plugin/src/main/java/org/apache/cxf/maven_plugin/java2swagger/Java2SwaggerMojo.java
+++ b/maven-plugins/java2swagger-plugin/src/main/java/org/apache/cxf/maven_plugin/java2swagger/Java2SwaggerMojo.java
@@ -34,7 +34,6 @@ import com.fasterxml.jackson.annotation.JsonInclude;
 import com.fasterxml.jackson.core.util.DefaultPrettyPrinter;
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.fasterxml.jackson.databind.ObjectWriter;
-import com.fasterxml.jackson.databind.SerializationFeature;
 
 import org.apache.cxf.helpers.FileUtils;
 import org.apache.maven.plugin.AbstractMojo;
@@ -286,8 +285,8 @@ public class Java2SwaggerMojo extends AbstractMojo {
 
     private void configureSwagger() {
         swagger = new Swagger();
-        mapper.configure(SerializationFeature.WRITE_EMPTY_JSON_ARRAYS, false);
         mapper.setSerializationInclusion(JsonInclude.Include.NON_NULL);
+        mapper.setSerializationInclusion(JsonInclude.Include.NON_EMPTY);
         Info info = new Info();
         Contact swaggerContact = new Contact();
         License swaggerLicense = new License();

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[2/4] cxf git commit: Remove a bunch of pointless "TODO Auto-generated method" comments

Daniel  Kulp
Administrator
http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/GreeterSessionImpl.java
----------------------------------------------------------------------
diff --git a/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/GreeterSessionImpl.java b/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/GreeterSessionImpl.java
index 44f177b..0c262f9 100644
--- a/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/GreeterSessionImpl.java
+++ b/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/GreeterSessionImpl.java
@@ -96,13 +96,11 @@ public class GreeterSessionImpl implements Greeter {
 
 
     public Future<?> greetMeAsync(String requestType, AsyncHandler<GreetMeResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<GreetMeResponse> greetMeAsync(String requestType) {
-        // TODO Auto-generated method stub
         return null;
     }
 
@@ -129,25 +127,21 @@ public class GreeterSessionImpl implements Greeter {
 
 
     public Future<?> pingMeAsync(AsyncHandler<PingMeResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<PingMeResponse> pingMeAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Future<?> sayHiAsync(AsyncHandler<SayHiResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<SayHiResponse> sayHiAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/MapIdentityManager.java
----------------------------------------------------------------------
diff --git a/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/MapIdentityManager.java b/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/MapIdentityManager.java
index d4b74da..7352df1 100644
--- a/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/MapIdentityManager.java
+++ b/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/MapIdentityManager.java
@@ -63,7 +63,6 @@ class MapIdentityManager implements IdentityManager {
 
     @Override
     public Account verify(Credential credential) {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/PerRequestAnnotationGreeterImpl.java
----------------------------------------------------------------------
diff --git a/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/PerRequestAnnotationGreeterImpl.java b/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/PerRequestAnnotationGreeterImpl.java
index e9d5d7a..fc6e3fc 100644
--- a/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/PerRequestAnnotationGreeterImpl.java
+++ b/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/PerRequestAnnotationGreeterImpl.java
@@ -55,43 +55,35 @@ public class PerRequestAnnotationGreeterImpl implements Greeter {
 
 
     public Future<?> greetMeAsync(String requestType, AsyncHandler<GreetMeResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<GreetMeResponse> greetMeAsync(String requestType) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public void greetMeOneWay(String requestType) {
-        // TODO Auto-generated method stub
-
     }
 
 
     public Future<?> pingMeAsync(AsyncHandler<PingMeResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<PingMeResponse> pingMeAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Future<?> sayHiAsync(AsyncHandler<SayHiResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<SayHiResponse> sayHiAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/SessionAnnotationGreeterImpl.java
----------------------------------------------------------------------
diff --git a/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/SessionAnnotationGreeterImpl.java b/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/SessionAnnotationGreeterImpl.java
index 02ed1c5..6cbc252 100644
--- a/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/SessionAnnotationGreeterImpl.java
+++ b/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/SessionAnnotationGreeterImpl.java
@@ -58,43 +58,35 @@ public class SessionAnnotationGreeterImpl implements Greeter {
 
 
     public Future<?> greetMeAsync(String requestType, AsyncHandler<GreetMeResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<GreetMeResponse> greetMeAsync(String requestType) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public void greetMeOneWay(String requestType) {
-        // TODO Auto-generated method stub
-
     }
 
 
     public Future<?> pingMeAsync(AsyncHandler<PingMeResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<PingMeResponse> pingMeAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Future<?> sayHiAsync(AsyncHandler<SayHiResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<SayHiResponse> sayHiAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/SpringAnnotationGreeterImpl.java
----------------------------------------------------------------------
diff --git a/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/SpringAnnotationGreeterImpl.java b/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/SpringAnnotationGreeterImpl.java
index a30badc..8b718cd 100644
--- a/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/SpringAnnotationGreeterImpl.java
+++ b/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/SpringAnnotationGreeterImpl.java
@@ -56,43 +56,35 @@ public class SpringAnnotationGreeterImpl implements Greeter {
 
 
     public Future<?> greetMeAsync(String requestType, AsyncHandler<GreetMeResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<GreetMeResponse> greetMeAsync(String requestType) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public void greetMeOneWay(String requestType) {
-        // TODO Auto-generated method stub
-
     }
 
 
     public Future<?> pingMeAsync(AsyncHandler<PingMeResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<PingMeResponse> pingMeAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Future<?> sayHiAsync(AsyncHandler<SayHiResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<SayHiResponse> sayHiAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/StatefulGreeterImpl.java
----------------------------------------------------------------------
diff --git a/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/StatefulGreeterImpl.java b/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/StatefulGreeterImpl.java
index 3c3937d..cc0c48b 100644
--- a/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/StatefulGreeterImpl.java
+++ b/systests/transport-undertow/src/test/java/org/apache/cxf/systest/http_undertow/StatefulGreeterImpl.java
@@ -53,43 +53,36 @@ public class StatefulGreeterImpl implements Greeter {
 
 
     public Future<?> greetMeAsync(String requestType, AsyncHandler<GreetMeResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<GreetMeResponse> greetMeAsync(String requestType) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public void greetMeOneWay(String requestType) {
-        // TODO Auto-generated method stub
 
     }
 
 
     public Future<?> pingMeAsync(AsyncHandler<PingMeResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<PingMeResponse> pingMeAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Future<?> sayHiAsync(AsyncHandler<SayHiResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<SayHiResponse> sayHiAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/transports/src/test/java/org/apache/cxf/systest/http/GreeterSessionImpl.java
----------------------------------------------------------------------
diff --git a/systests/transports/src/test/java/org/apache/cxf/systest/http/GreeterSessionImpl.java b/systests/transports/src/test/java/org/apache/cxf/systest/http/GreeterSessionImpl.java
index a18a9f5..d41e22a 100644
--- a/systests/transports/src/test/java/org/apache/cxf/systest/http/GreeterSessionImpl.java
+++ b/systests/transports/src/test/java/org/apache/cxf/systest/http/GreeterSessionImpl.java
@@ -96,13 +96,11 @@ public class GreeterSessionImpl implements Greeter {
 
 
     public Future<?> greetMeAsync(String requestType, AsyncHandler<GreetMeResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<GreetMeResponse> greetMeAsync(String requestType) {
-        // TODO Auto-generated method stub
         return null;
     }
 
@@ -129,25 +127,21 @@ public class GreeterSessionImpl implements Greeter {
 
 
     public Future<?> pingMeAsync(AsyncHandler<PingMeResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<PingMeResponse> pingMeAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Future<?> sayHiAsync(AsyncHandler<SayHiResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<SayHiResponse> sayHiAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/transports/src/test/java/org/apache/cxf/systest/http/PerRequestAnnotationGreeterImpl.java
----------------------------------------------------------------------
diff --git a/systests/transports/src/test/java/org/apache/cxf/systest/http/PerRequestAnnotationGreeterImpl.java b/systests/transports/src/test/java/org/apache/cxf/systest/http/PerRequestAnnotationGreeterImpl.java
index e9afc2a..1787860 100644
--- a/systests/transports/src/test/java/org/apache/cxf/systest/http/PerRequestAnnotationGreeterImpl.java
+++ b/systests/transports/src/test/java/org/apache/cxf/systest/http/PerRequestAnnotationGreeterImpl.java
@@ -55,43 +55,35 @@ public class PerRequestAnnotationGreeterImpl implements Greeter {
 
 
     public Future<?> greetMeAsync(String requestType, AsyncHandler<GreetMeResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<GreetMeResponse> greetMeAsync(String requestType) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public void greetMeOneWay(String requestType) {
-        // TODO Auto-generated method stub
-
     }
 
 
     public Future<?> pingMeAsync(AsyncHandler<PingMeResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<PingMeResponse> pingMeAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Future<?> sayHiAsync(AsyncHandler<SayHiResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<SayHiResponse> sayHiAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/transports/src/test/java/org/apache/cxf/systest/http/SessionAnnotationGreeterImpl.java
----------------------------------------------------------------------
diff --git a/systests/transports/src/test/java/org/apache/cxf/systest/http/SessionAnnotationGreeterImpl.java b/systests/transports/src/test/java/org/apache/cxf/systest/http/SessionAnnotationGreeterImpl.java
index 10327cd..a6bfd8e 100644
--- a/systests/transports/src/test/java/org/apache/cxf/systest/http/SessionAnnotationGreeterImpl.java
+++ b/systests/transports/src/test/java/org/apache/cxf/systest/http/SessionAnnotationGreeterImpl.java
@@ -58,43 +58,36 @@ public class SessionAnnotationGreeterImpl implements Greeter {
 
 
     public Future<?> greetMeAsync(String requestType, AsyncHandler<GreetMeResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<GreetMeResponse> greetMeAsync(String requestType) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public void greetMeOneWay(String requestType) {
-        // TODO Auto-generated method stub
 
     }
 
 
     public Future<?> pingMeAsync(AsyncHandler<PingMeResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<PingMeResponse> pingMeAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Future<?> sayHiAsync(AsyncHandler<SayHiResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<SayHiResponse> sayHiAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/transports/src/test/java/org/apache/cxf/systest/http/SpringAnnotationGreeterImpl.java
----------------------------------------------------------------------
diff --git a/systests/transports/src/test/java/org/apache/cxf/systest/http/SpringAnnotationGreeterImpl.java b/systests/transports/src/test/java/org/apache/cxf/systest/http/SpringAnnotationGreeterImpl.java
index 22531f2..bde19d4 100644
--- a/systests/transports/src/test/java/org/apache/cxf/systest/http/SpringAnnotationGreeterImpl.java
+++ b/systests/transports/src/test/java/org/apache/cxf/systest/http/SpringAnnotationGreeterImpl.java
@@ -56,43 +56,35 @@ public class SpringAnnotationGreeterImpl implements Greeter {
 
 
     public Future<?> greetMeAsync(String requestType, AsyncHandler<GreetMeResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<GreetMeResponse> greetMeAsync(String requestType) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public void greetMeOneWay(String requestType) {
-        // TODO Auto-generated method stub
-
     }
 
 
     public Future<?> pingMeAsync(AsyncHandler<PingMeResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<PingMeResponse> pingMeAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Future<?> sayHiAsync(AsyncHandler<SayHiResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<SayHiResponse> sayHiAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/transports/src/test/java/org/apache/cxf/systest/http/StatefulGreeterImpl.java
----------------------------------------------------------------------
diff --git a/systests/transports/src/test/java/org/apache/cxf/systest/http/StatefulGreeterImpl.java b/systests/transports/src/test/java/org/apache/cxf/systest/http/StatefulGreeterImpl.java
index df7e8da..9777bbc 100644
--- a/systests/transports/src/test/java/org/apache/cxf/systest/http/StatefulGreeterImpl.java
+++ b/systests/transports/src/test/java/org/apache/cxf/systest/http/StatefulGreeterImpl.java
@@ -53,43 +53,36 @@ public class StatefulGreeterImpl implements Greeter {
 
 
     public Future<?> greetMeAsync(String requestType, AsyncHandler<GreetMeResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<GreetMeResponse> greetMeAsync(String requestType) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public void greetMeOneWay(String requestType) {
-        // TODO Auto-generated method stub
 
     }
 
 
     public Future<?> pingMeAsync(AsyncHandler<PingMeResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<PingMeResponse> pingMeAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Future<?> sayHiAsync(AsyncHandler<SayHiResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public Response<SayHiResponse> sayHiAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/transports/src/test/java/org/apache/cxf/systest/http/auth/GreeterImpl.java
----------------------------------------------------------------------
diff --git a/systests/transports/src/test/java/org/apache/cxf/systest/http/auth/GreeterImpl.java b/systests/transports/src/test/java/org/apache/cxf/systest/http/auth/GreeterImpl.java
index b5d4c84..deec3f5 100644
--- a/systests/transports/src/test/java/org/apache/cxf/systest/http/auth/GreeterImpl.java
+++ b/systests/transports/src/test/java/org/apache/cxf/systest/http/auth/GreeterImpl.java
@@ -28,12 +28,9 @@ public class GreeterImpl implements Greeter {
     }
 
     public void pingMe() throws PingMeFault {
-        // TODO Auto-generated method stub
-
     }
 
     public String greetMe(String requestType) {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/uncategorized/src/test/java/org/apache/cxf/systest/beanincreationexception/DummyPlatformTransactionManager.java
----------------------------------------------------------------------
diff --git a/systests/uncategorized/src/test/java/org/apache/cxf/systest/beanincreationexception/DummyPlatformTransactionManager.java b/systests/uncategorized/src/test/java/org/apache/cxf/systest/beanincreationexception/DummyPlatformTransactionManager.java
index f60fed8..b972071 100644
--- a/systests/uncategorized/src/test/java/org/apache/cxf/systest/beanincreationexception/DummyPlatformTransactionManager.java
+++ b/systests/uncategorized/src/test/java/org/apache/cxf/systest/beanincreationexception/DummyPlatformTransactionManager.java
@@ -26,17 +26,14 @@ import org.springframework.transaction.TransactionStatus;
 public class DummyPlatformTransactionManager implements PlatformTransactionManager {
 
     public void commit(TransactionStatus arg0) throws TransactionException {
-        // TODO Auto-generated method stub
 
     }
 
     public TransactionStatus getTransaction(TransactionDefinition arg0) throws TransactionException {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public void rollback(TransactionStatus arg0) throws TransactionException {
-        // TODO Auto-generated method stub
 
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/uncategorized/src/test/java/org/apache/cxf/systest/clustering/ControlImpl.java
----------------------------------------------------------------------
diff --git a/systests/uncategorized/src/test/java/org/apache/cxf/systest/clustering/ControlImpl.java b/systests/uncategorized/src/test/java/org/apache/cxf/systest/clustering/ControlImpl.java
index df693ae..a193d1d 100644
--- a/systests/uncategorized/src/test/java/org/apache/cxf/systest/clustering/ControlImpl.java
+++ b/systests/uncategorized/src/test/java/org/apache/cxf/systest/clustering/ControlImpl.java
@@ -106,27 +106,22 @@ public class ControlImpl implements Control {
     }
 
     public Response<?> setFaultLocationAsync(FaultLocation in) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public Future<?> setFaultLocationAsync(FaultLocation in, AsyncHandler<?> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public Response<?> setRobustInOnlyModeAsync(boolean in) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public Future<?> setRobustInOnlyModeAsync(boolean in, AsyncHandler<?> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public void setRobustInOnlyMode(boolean in) {
-        // TODO Auto-generated method stub
 
     }
 }

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/uncategorized/src/test/java/org/apache/cxf/systest/mtom/TestMtomJMSImpl.java
----------------------------------------------------------------------
diff --git a/systests/uncategorized/src/test/java/org/apache/cxf/systest/mtom/TestMtomJMSImpl.java b/systests/uncategorized/src/test/java/org/apache/cxf/systest/mtom/TestMtomJMSImpl.java
index 86ef151..5e6862e 100644
--- a/systests/uncategorized/src/test/java/org/apache/cxf/systest/mtom/TestMtomJMSImpl.java
+++ b/systests/uncategorized/src/test/java/org/apache/cxf/systest/mtom/TestMtomJMSImpl.java
@@ -36,7 +36,6 @@ import org.apache.cxf.mime.types.XopStringType;
 public class TestMtomJMSImpl implements TestMtom {
 
     public void testXop(Holder<String> name, Holder<DataHandler> attachinfo) {
-        // TODO Auto-generated method stub
         name.value = "return detail + " + name.value;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/uncategorized/src/test/java/org/apache/cxf/systest/schema_validation/SchemaValidationImpl.java
----------------------------------------------------------------------
diff --git a/systests/uncategorized/src/test/java/org/apache/cxf/systest/schema_validation/SchemaValidationImpl.java b/systests/uncategorized/src/test/java/org/apache/cxf/systest/schema_validation/SchemaValidationImpl.java
index d80f930..5c6f8eb 100644
--- a/systests/uncategorized/src/test/java/org/apache/cxf/systest/schema_validation/SchemaValidationImpl.java
+++ b/systests/uncategorized/src/test/java/org/apache/cxf/systest/schema_validation/SchemaValidationImpl.java
@@ -94,7 +94,6 @@ public class SchemaValidationImpl implements SchemaValidation {
 
     public SomeResponseWithHeader doSomethingWithHeader(SomeRequestWithHeader in,
                                                         SomeHeader inHeader) {
-        // TODO Auto-generated method stub
         return null;
     }
 }

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/ws-rm/src/test/java/org/apache/cxf/systest/ws/rm/MessageLossSimulator.java
----------------------------------------------------------------------
diff --git a/systests/ws-rm/src/test/java/org/apache/cxf/systest/ws/rm/MessageLossSimulator.java b/systests/ws-rm/src/test/java/org/apache/cxf/systest/ws/rm/MessageLossSimulator.java
index c044d11..16c920a 100644
--- a/systests/ws-rm/src/test/java/org/apache/cxf/systest/ws/rm/MessageLossSimulator.java
+++ b/systests/ws-rm/src/test/java/org/apache/cxf/systest/ws/rm/MessageLossSimulator.java
@@ -165,7 +165,6 @@ public class MessageLossSimulator extends AbstractPhaseInterceptor<Message> {
 
         @Override
         public void write(int b) throws IOException {
-            // TODO Auto-generated method stub
 
         }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/testutils/src/main/java/org/apache/cxf/greeter_control/AbstractGreeterImpl.java
----------------------------------------------------------------------
diff --git a/testutils/src/main/java/org/apache/cxf/greeter_control/AbstractGreeterImpl.java b/testutils/src/main/java/org/apache/cxf/greeter_control/AbstractGreeterImpl.java
index e311449..8396846 100644
--- a/testutils/src/main/java/org/apache/cxf/greeter_control/AbstractGreeterImpl.java
+++ b/testutils/src/main/java/org/apache/cxf/greeter_control/AbstractGreeterImpl.java
@@ -86,12 +86,10 @@ public class AbstractGreeterImpl implements Greeter {
     }
 
     public Future<?> greetMeAsync(String arg0, AsyncHandler<GreetMeResponse> arg1) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public Response<GreetMeResponse> greetMeAsync(String arg0) {
-        // TODO Auto-generated method stub
         return null;
     }
 
@@ -115,27 +113,22 @@ public class AbstractGreeterImpl implements Greeter {
     }
 
     public Response<PingMeResponse> pingMeAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public Future<?> pingMeAsync(AsyncHandler<PingMeResponse> arg0) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public String sayHi() {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public Response<SayHiResponse> sayHiAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public Future<?> sayHiAsync(AsyncHandler<SayHiResponse> arg0) {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/testutils/src/main/java/org/apache/cxf/greeter_control/ControlImpl.java
----------------------------------------------------------------------
diff --git a/testutils/src/main/java/org/apache/cxf/greeter_control/ControlImpl.java b/testutils/src/main/java/org/apache/cxf/greeter_control/ControlImpl.java
index 91ac8f8..95ec3b4 100644
--- a/testutils/src/main/java/org/apache/cxf/greeter_control/ControlImpl.java
+++ b/testutils/src/main/java/org/apache/cxf/greeter_control/ControlImpl.java
@@ -167,22 +167,18 @@ public class ControlImpl implements Control {
     }
 
     public Response<?> setFaultLocationAsync(FaultLocation in) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public Future<?> setFaultLocationAsync(FaultLocation in, AsyncHandler<?> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public Response<?> setRobustInOnlyModeAsync(boolean in) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public Future<?> setRobustInOnlyModeAsync(boolean in, AsyncHandler<?> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/testutils/src/main/java/org/apache/cxf/ordered_param_holder/OrderedParamHolderImpl.java
----------------------------------------------------------------------
diff --git a/testutils/src/main/java/org/apache/cxf/ordered_param_holder/OrderedParamHolderImpl.java b/testutils/src/main/java/org/apache/cxf/ordered_param_holder/OrderedParamHolderImpl.java
index 87ebfbf..ab27bbc 100644
--- a/testutils/src/main/java/org/apache/cxf/ordered_param_holder/OrderedParamHolderImpl.java
+++ b/testutils/src/main/java/org/apache/cxf/ordered_param_holder/OrderedParamHolderImpl.java
@@ -31,7 +31,6 @@ import javax.xml.ws.Holder;
 public class OrderedParamHolderImpl implements OrderedParamHolder {
 
     public void orderedParamHolder(Holder<ComplexStruct> part3, Holder<Integer> part2, Holder<String> part1) {
-        // TODO Auto-generated method stub
         part2.value = Integer.valueOf(part2.value.intValue() + 1);
         part1.value = "return " + part1.value;
         part3.value.elem1 = "return " + part3.value.elem1;

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/testutils/src/main/java/org/apache/headers/HeaderTesterImpl.java
----------------------------------------------------------------------
diff --git a/testutils/src/main/java/org/apache/headers/HeaderTesterImpl.java b/testutils/src/main/java/org/apache/headers/HeaderTesterImpl.java
index 5201817..08046db 100644
--- a/testutils/src/main/java/org/apache/headers/HeaderTesterImpl.java
+++ b/testutils/src/main/java/org/apache/headers/HeaderTesterImpl.java
@@ -39,7 +39,6 @@ import org.apache.headers.types.SOAPHeaderData;
 public class HeaderTesterImpl implements HeaderTester {
 
     public InHeaderResponse inHeader(InHeader me, SOAPHeaderData headerInfo) {
-        // TODO Auto-generated method stub
         InHeaderResponse resp = new InHeaderResponse();
         resp.setResponseType("requestType=" + me.getRequestType() + "\nheaderData.message="
                 + headerInfo.getMessage() + "\nheaderData.getOriginator=" + headerInfo.getOriginator());
@@ -47,7 +46,6 @@ public class HeaderTesterImpl implements HeaderTester {
     }
 
     public InoutHeaderResponse inoutHeader(InoutHeader me, Holder<SOAPHeaderData> headerInfo) {
-        // TODO Auto-generated method stub
         InoutHeaderResponse resp = new InoutHeaderResponse();
         resp.setResponseType("requestType=" + me.getRequestType());
         if (headerInfo.value != null) {

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/testutils/src/main/java/org/apache/hello_world_soap_http/AnotherDerivedGreeterImpl.java
----------------------------------------------------------------------
diff --git a/testutils/src/main/java/org/apache/hello_world_soap_http/AnotherDerivedGreeterImpl.java b/testutils/src/main/java/org/apache/hello_world_soap_http/AnotherDerivedGreeterImpl.java
index 0c97383..c6692bc 100644
--- a/testutils/src/main/java/org/apache/hello_world_soap_http/AnotherDerivedGreeterImpl.java
+++ b/testutils/src/main/java/org/apache/hello_world_soap_http/AnotherDerivedGreeterImpl.java
@@ -186,7 +186,6 @@ public class AnotherDerivedGreeterImpl implements GreeterEndpointInterface {
     }
 
     public String testNillable(String nillElem, int intElem) {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/testutils/src/main/java/org/apache/hello_world_soap_http/DerivedGreeterImpl.java
----------------------------------------------------------------------
diff --git a/testutils/src/main/java/org/apache/hello_world_soap_http/DerivedGreeterImpl.java b/testutils/src/main/java/org/apache/hello_world_soap_http/DerivedGreeterImpl.java
index b5ff92c..e4ee5f9 100644
--- a/testutils/src/main/java/org/apache/hello_world_soap_http/DerivedGreeterImpl.java
+++ b/testutils/src/main/java/org/apache/hello_world_soap_http/DerivedGreeterImpl.java
@@ -186,7 +186,6 @@ public class DerivedGreeterImpl implements Greeter {
     }
 
     public String testNillable(String nillElem, int intElem) {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/testutils/src/main/java/org/apache/hello_world_soap_http/NotAnnotatedGreeterImpl.java
----------------------------------------------------------------------
diff --git a/testutils/src/main/java/org/apache/hello_world_soap_http/NotAnnotatedGreeterImpl.java b/testutils/src/main/java/org/apache/hello_world_soap_http/NotAnnotatedGreeterImpl.java
index a27dd06..7b16739 100644
--- a/testutils/src/main/java/org/apache/hello_world_soap_http/NotAnnotatedGreeterImpl.java
+++ b/testutils/src/main/java/org/apache/hello_world_soap_http/NotAnnotatedGreeterImpl.java
@@ -156,7 +156,6 @@ public class NotAnnotatedGreeterImpl implements Greeter {
     }
 
     public String testNillable(String nillElem, int intElem) {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/testutils/src/main/java/org/apache/hello_world_xml_http/bare/GreeterImpl.java
----------------------------------------------------------------------
diff --git a/testutils/src/main/java/org/apache/hello_world_xml_http/bare/GreeterImpl.java b/testutils/src/main/java/org/apache/hello_world_xml_http/bare/GreeterImpl.java
index 5823946..5ecff5d 100644
--- a/testutils/src/main/java/org/apache/hello_world_xml_http/bare/GreeterImpl.java
+++ b/testutils/src/main/java/org/apache/hello_world_xml_http/bare/GreeterImpl.java
@@ -31,33 +31,27 @@ import org.apache.hello_world_xml_http.bare.types.MyComplexStructType;
 public class GreeterImpl implements Greeter {
 
     public String greetMe(String me) {
-        // TODO Auto-generated method stub
         return "Hello " + me;
     }
 
     public String sayHi() {
-        // TODO Auto-generated method stub
         return "Bonjour";
     }
 
     public MyComplexStructType sendReceiveData(MyComplexStructType in) {
-        // TODO Auto-generated method stub
         return in;
     }
 
     public String testMultiParamPart(MyComplexStructType in2, String in1) {
-        // TODO Auto-generated method stub
         in2.setElem1(in1);
         return "Bonjour";
     }
 
     public String testTriPart(MyComplexStructType in1, String in3, String in2) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public String testTriPartNoOrder(String in3, MyComplexStructType in1, String in2) {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/tools/common/src/test/java/org/apache/cxf/tools/util/FileWriterUtilTest.java
----------------------------------------------------------------------
diff --git a/tools/common/src/test/java/org/apache/cxf/tools/util/FileWriterUtilTest.java b/tools/common/src/test/java/org/apache/cxf/tools/util/FileWriterUtilTest.java
index 4283f49..cb7ec49 100644
--- a/tools/common/src/test/java/org/apache/cxf/tools/util/FileWriterUtilTest.java
+++ b/tools/common/src/test/java/org/apache/cxf/tools/util/FileWriterUtilTest.java
@@ -54,7 +54,6 @@ public class FileWriterUtilTest extends Assert {
             String path = targetDir.getAbsolutePath() + packPath;
             assertNotNull(new File(path).getName());
         } catch (IOException e) {
-            // TODO Auto-generated catch block
             e.printStackTrace();
         } finally {
             cleanDir(targetDir);
@@ -73,7 +72,6 @@ public class FileWriterUtilTest extends Assert {
             fileWriter = new FileWriterUtil(targetDir.getAbsolutePath(), null);
             assertNotNull(fileWriter.getWriter("com.iona.test.SAMPLE", "A.java"));
         } catch (IOException e) {
-            // TODO Auto-generated catch block
             e.printStackTrace();
         } finally {
             cleanDir(targetDir);

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/tools/javato/ws/src/test/java/org/apache/cxf/tools/fortest/inherit/A.java
----------------------------------------------------------------------
diff --git a/tools/javato/ws/src/test/java/org/apache/cxf/tools/fortest/inherit/A.java b/tools/javato/ws/src/test/java/org/apache/cxf/tools/fortest/inherit/A.java
index d8bf64f..82e5fe0 100644
--- a/tools/javato/ws/src/test/java/org/apache/cxf/tools/fortest/inherit/A.java
+++ b/tools/javato/ws/src/test/java/org/apache/cxf/tools/fortest/inherit/A.java
@@ -49,7 +49,6 @@ public class A implements B {
 
     @Override
     public String hello() {
-        // TODO Auto-generated method stub
         return null;
     }
 }
\ No newline at end of file

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[3/4] cxf git commit: Remove a bunch of pointless "TODO Auto-generated method" comments

Daniel  Kulp
Administrator
In reply to this post by Daniel Kulp
Remove a bunch of pointless "TODO Auto-generated method" comments


Project: http://git-wip-us.apache.org/repos/asf/cxf/repo
Commit: http://git-wip-us.apache.org/repos/asf/cxf/commit/5f277de6
Tree: http://git-wip-us.apache.org/repos/asf/cxf/tree/5f277de6
Diff: http://git-wip-us.apache.org/repos/asf/cxf/diff/5f277de6

Branch: refs/heads/master
Commit: 5f277de68568c7e3df68c2f5f40a09faf353192f
Parents: baf0980
Author: Daniel Kulp <[hidden email]>
Authored: Thu Aug 10 14:00:07 2017 -0400
Committer: Daniel Kulp <[hidden email]>
Committed: Thu Aug 10 14:00:07 2017 -0400

----------------------------------------------------------------------
 .../cxf/bus/managers/ServerRegistryImpl.java    |  2 -
 .../catalog/CatalogXmlSchemaURIResolver.java    |  2 +-
 .../org/apache/cxf/common/jaxb/JAXBUtils.java   |  2 -
 .../apache/cxf/common/util/Base64Exception.java |  3 --
 .../cxf/databinding/source/NodeDataReader.java  |  6 ---
 .../interceptor/MessageSenderInterceptor.java   |  2 +-
 .../interceptor/OutgoingChainInterceptor.java   | 11 +++--
 .../cxf/staxutils/AbstractDOMStreamReader.java  |  2 -
 .../org/apache/cxf/endpoint/DummyServer.java    |  2 -
 .../org/apache/cxf/jca/cxf/test/DummyBus.java   |  7 ---
 .../corba/runtime/CorbaStreamReader.java        | 24 ----------
 .../cxf/binding/corba/utils/CorbaAnyHelper.java |  1 -
 .../cxf/aegis/util/stax/JDOMStreamWriter.java   |  1 -
 .../java/org/apache/cxf/jaxrs/Customer.java     |  2 -
 .../apache/cxf/jaxrs/JAXBContextProvider.java   |  1 -
 .../apache/cxf/jaxrs/JAXBContextProvider2.java  |  1 -
 .../cxf/jaxrs/JAXRSServerFactoryBeanTest.java   |  2 -
 .../cxf/jaxrs/ext/MessageContextImplTest.java   |  1 -
 .../cxf/jaxrs/impl/ConfigurationImplTest.java   |  1 -
 .../jaxrs/provider/JAXBElementProviderTest.java | 46 ++++++--------------
 .../cxf/jaxrs/provider/ProviderFactoryTest.java | 17 --------
 .../apache/cxf/jaxrs/resources/BookStore.java   |  2 -
 .../apache/cxf/jaxrs/utils/JAXRSUtilsTest.java  |  4 --
 .../interceptors/MessageModeOutInterceptor.java |  1 -
 .../cxf/jaxws/ConfiguredEndpointTest.java       |  1 -
 .../jaxws/handler/HandlerChainInvokerTest.java  |  2 -
 .../cxf/jaxws/service/FooServiceImpl.java       |  1 -
 .../org/apache/cxf/jaxws/service/Hello2.java    |  3 --
 .../org/apache/cxf/jaxws/service/Hello3.java    |  3 --
 .../cxf/service/factory/ServerFactoryTest.java  |  5 ---
 .../apache/cxf/jaxrs/client/WebClientTest.java  |  1 -
 .../apache/cxf/jaxrs/resources/BookStore.java   |  2 -
 .../oauth2/grants/OAuthDataProviderImpl.java    |  7 ---
 .../crypto/CodeGrantEncryptingDataProvider.java |  1 -
 .../utils/crypto/EncryptingDataProvider.java    |  2 -
 .../saml/sso/state/HTTPSPStateManager.java      |  2 -
 .../http/netty/client/NettyHttpConduit.java     |  3 --
 .../jetty/WebSocketVirtualServletRequest.java   |  9 ----
 .../jetty9/Jetty9WebSocketDestination.java      |  1 -
 .../WebSocketUndertowServletRequest.java        |  9 ----
 .../org/apache/cxf/ws/policy/TestAssertion.java |  1 -
 .../cxf/ws/rm/soap/RedeliveryQueueImpl.java     |  1 -
 .../apache/cxf/ws/rm/ManagedRMManagerTest.java  |  5 ---
 .../cxf/ws/rm/RMManagerConfigurationTest.java   | 26 +++--------
 .../apache/cxf/wsdl/JAXBExtensionHelper.java    |  1 -
 .../apache/cxf/systest/source/GreeterImpl.java  |  2 -
 .../systest/source/PutLastTradePriceImpl.java   |  4 --
 .../apache/cxf/systest/jaxrs/BookServer.java    |  2 -
 .../cxf/systest/jaxrs/CustomResponse.java       | 22 ----------
 .../jaxrs/JAXRS20ClientServerBookTest.java      |  2 -
 .../cxf/systest/jaxrs/JAXRSAsyncClientTest.java |  2 -
 .../cxf/systest/jaxws/ClientServerTest.java     |  1 -
 .../jaxws/cxf5064/SOAPHeaderServiceImpl.java    |  1 -
 .../apache/cxf/systest/jaxws/types/BarImpl.java |  1 -
 .../provider/HWDOMSourcePayloadProvider.java    |  1 -
 .../cxf/systest/jms/TwoWayJMSImplBase.java      |  6 ---
 .../systest/jms/security/TwoWayJMSImplBase.java |  6 ---
 .../http_undertow/GreeterSessionImpl.java       |  6 ---
 .../http_undertow/MapIdentityManager.java       |  1 -
 .../PerRequestAnnotationGreeterImpl.java        |  8 ----
 .../SessionAnnotationGreeterImpl.java           |  8 ----
 .../SpringAnnotationGreeterImpl.java            |  8 ----
 .../http_undertow/StatefulGreeterImpl.java      |  7 ---
 .../cxf/systest/http/GreeterSessionImpl.java    |  6 ---
 .../http/PerRequestAnnotationGreeterImpl.java   |  8 ----
 .../http/SessionAnnotationGreeterImpl.java      |  7 ---
 .../http/SpringAnnotationGreeterImpl.java       |  8 ----
 .../cxf/systest/http/StatefulGreeterImpl.java   |  7 ---
 .../cxf/systest/http/auth/GreeterImpl.java      |  3 --
 .../DummyPlatformTransactionManager.java        |  3 --
 .../cxf/systest/clustering/ControlImpl.java     |  5 ---
 .../cxf/systest/mtom/TestMtomJMSImpl.java       |  1 -
 .../schema_validation/SchemaValidationImpl.java |  1 -
 .../cxf/systest/ws/rm/MessageLossSimulator.java |  1 -
 .../greeter_control/AbstractGreeterImpl.java    |  7 ---
 .../apache/cxf/greeter_control/ControlImpl.java |  4 --
 .../OrderedParamHolderImpl.java                 |  1 -
 .../org/apache/headers/HeaderTesterImpl.java    |  2 -
 .../AnotherDerivedGreeterImpl.java              |  1 -
 .../DerivedGreeterImpl.java                     |  1 -
 .../NotAnnotatedGreeterImpl.java                |  1 -
 .../hello_world_xml_http/bare/GreeterImpl.java  |  6 ---
 .../cxf/tools/util/FileWriterUtilTest.java      |  2 -
 .../org/apache/cxf/tools/fortest/inherit/A.java |  1 -
 84 files changed, 29 insertions(+), 364 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/core/src/main/java/org/apache/cxf/bus/managers/ServerRegistryImpl.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/cxf/bus/managers/ServerRegistryImpl.java b/core/src/main/java/org/apache/cxf/bus/managers/ServerRegistryImpl.java
index 7cffcbc..bb535d3 100644
--- a/core/src/main/java/org/apache/cxf/bus/managers/ServerRegistryImpl.java
+++ b/core/src/main/java/org/apache/cxf/bus/managers/ServerRegistryImpl.java
@@ -76,8 +76,6 @@ public class ServerRegistryImpl implements ServerRegistry, BusLifeCycleListener
     }
 
     public void initComplete() {
-        // TODO Auto-generated method stub
-
     }
 
     @PreDestroy

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/core/src/main/java/org/apache/cxf/catalog/CatalogXmlSchemaURIResolver.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/cxf/catalog/CatalogXmlSchemaURIResolver.java b/core/src/main/java/org/apache/cxf/catalog/CatalogXmlSchemaURIResolver.java
index f0241ae..e04007d 100644
--- a/core/src/main/java/org/apache/cxf/catalog/CatalogXmlSchemaURIResolver.java
+++ b/core/src/main/java/org/apache/cxf/catalog/CatalogXmlSchemaURIResolver.java
@@ -69,7 +69,7 @@ public class CatalogXmlSchemaURIResolver implements URIResolver {
             in = this.resolver.resolve(resolvedSchemaLocation, baseUri);
         }
 
-        // XXX: If we return null, a NPE is raised in SchemaBuilder.
+        // If we return null, a NPE is raised in SchemaBuilder.
         // If we return new InputSource(), a XmlSchemaException is raised
         // but without any nice error message. So let's just throw a nice error here.
         if (in == null) {

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/core/src/main/java/org/apache/cxf/common/jaxb/JAXBUtils.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/cxf/common/jaxb/JAXBUtils.java b/core/src/main/java/org/apache/cxf/common/jaxb/JAXBUtils.java
index a8ab707..e33a121 100644
--- a/core/src/main/java/org/apache/cxf/common/jaxb/JAXBUtils.java
+++ b/core/src/main/java/org/apache/cxf/common/jaxb/JAXBUtils.java
@@ -655,7 +655,6 @@ public final class JAXBUtils {
                 cls = Class.forName("com.sun.xml.bind.api.JAXBRIContext");
                 refClass = Class.forName(pkg + "api.TypeReference");
             } catch (ClassNotFoundException e) {
-                // TODO Auto-generated catch block
                 cls = Class.forName("com.sun.xml.internal.bind.api.JAXBRIContext", true, getXJCClassLoader());
                 pkg = "com.sun.xml.internal.bind.";
                 refClass = Class.forName(pkg + "api.TypeReference", true, getXJCClassLoader());
@@ -786,7 +785,6 @@ public final class JAXBUtils {
             try {
                 cls = Class.forName("com.sun.codemodel.writer.FileCodeWriter");
             } catch (ClassNotFoundException e) {
-                // TODO Auto-generated catch block
                 cls = Class.forName("com.sun.codemodel.internal.writer.FileCodeWriter",
                                     true, getXJCClassLoader());
             }

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/core/src/main/java/org/apache/cxf/common/util/Base64Exception.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/cxf/common/util/Base64Exception.java b/core/src/main/java/org/apache/cxf/common/util/Base64Exception.java
index dd51d83..ee17c38 100644
--- a/core/src/main/java/org/apache/cxf/common/util/Base64Exception.java
+++ b/core/src/main/java/org/apache/cxf/common/util/Base64Exception.java
@@ -28,17 +28,14 @@ public class Base64Exception extends Exception {
 
     public Base64Exception(Message msg) {
         super(msg);
-        // TODO Auto-generated constructor stub
     }
 
     public Base64Exception(Message msg, Throwable t) {
         super(msg, t);
-        // TODO Auto-generated constructor stub
     }
 
     public Base64Exception(Throwable cause) {
         super(cause);
-        // TODO Auto-generated constructor stub
     }
 
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/core/src/main/java/org/apache/cxf/databinding/source/NodeDataReader.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/cxf/databinding/source/NodeDataReader.java b/core/src/main/java/org/apache/cxf/databinding/source/NodeDataReader.java
index b98ec22..c55f79e 100644
--- a/core/src/main/java/org/apache/cxf/databinding/source/NodeDataReader.java
+++ b/core/src/main/java/org/apache/cxf/databinding/source/NodeDataReader.java
@@ -75,18 +75,12 @@ public class NodeDataReader implements DataReader<Node> {
     }
 
     public void setSchema(Schema s) {
-        // TODO Auto-generated method stub
-
     }
 
     public void setAttachments(Collection<Attachment> attachments) {
-        // TODO Auto-generated method stub
-
     }
 
     public void setProperty(String prop, Object value) {
-        // TODO Auto-generated method stub
-
     }
 
 }

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/core/src/main/java/org/apache/cxf/interceptor/MessageSenderInterceptor.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/cxf/interceptor/MessageSenderInterceptor.java b/core/src/main/java/org/apache/cxf/interceptor/MessageSenderInterceptor.java
index d2558cd..8288a70 100644
--- a/core/src/main/java/org/apache/cxf/interceptor/MessageSenderInterceptor.java
+++ b/core/src/main/java/org/apache/cxf/interceptor/MessageSenderInterceptor.java
@@ -66,7 +66,7 @@ public class MessageSenderInterceptor extends AbstractPhaseInterceptor<Message>
         }
     }
 
-    public static Conduit getConduit(Message message) {
+    public static Conduit getConduit(Message message) throws IOException {
         Exchange exchange = message.getExchange();
         Conduit conduit = exchange.getConduit(message);
         if (conduit == null

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/core/src/main/java/org/apache/cxf/interceptor/OutgoingChainInterceptor.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/cxf/interceptor/OutgoingChainInterceptor.java b/core/src/main/java/org/apache/cxf/interceptor/OutgoingChainInterceptor.java
index 715291e..34b49e1 100644
--- a/core/src/main/java/org/apache/cxf/interceptor/OutgoingChainInterceptor.java
+++ b/core/src/main/java/org/apache/cxf/interceptor/OutgoingChainInterceptor.java
@@ -69,7 +69,11 @@ public class OutgoingChainInterceptor extends AbstractPhaseInterceptor<Message>
         }
         Message out = ex.getOutMessage();
         if (out != null) {
-            getBackChannelConduit(message);
+            try {
+                getBackChannelConduit(message);
+            } catch (IOException ioe) {
+                throw new Fault(ioe);
+            }
             if (binding != null) {
                 out.put(MessageInfo.class, binding.getOperationInfo().getOutput());
                 out.put(BindingMessageInfo.class, binding.getOutput());
@@ -100,7 +104,7 @@ public class OutgoingChainInterceptor extends AbstractPhaseInterceptor<Message>
         }
     }
 
-    protected static Conduit getBackChannelConduit(Message message) {
+    protected static Conduit getBackChannelConduit(Message message) throws IOException {
         Conduit conduit = null;
         Exchange ex = message.getExchange();
         if (ex.getConduit(message) == null
@@ -110,8 +114,7 @@ public class OutgoingChainInterceptor extends AbstractPhaseInterceptor<Message>
                 ex.put(ConduitSelector.class,
                        new PreexistingConduitSelector(conduit, ex.getEndpoint()));
             } catch (IOException e) {
-                // TODO Auto-generated catch block
-                e.printStackTrace();
+                new Fault(e);
             }
         }
         return conduit;

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/core/src/main/java/org/apache/cxf/staxutils/AbstractDOMStreamReader.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/cxf/staxutils/AbstractDOMStreamReader.java b/core/src/main/java/org/apache/cxf/staxutils/AbstractDOMStreamReader.java
index a841eb9..1659bc9 100644
--- a/core/src/main/java/org/apache/cxf/staxutils/AbstractDOMStreamReader.java
+++ b/core/src/main/java/org/apache/cxf/staxutils/AbstractDOMStreamReader.java
@@ -352,12 +352,10 @@ public abstract class AbstractDOMStreamReader<T, I> implements XMLStreamReader {
     }
 
     public boolean standaloneSet() {
-        // TODO Auto-generated method stub
         return false;
     }
 
     public String getCharacterEncodingScheme() {
-        // TODO Auto-generated method stub
         return null;
     }
 }

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/core/src/test/java/org/apache/cxf/endpoint/DummyServer.java
----------------------------------------------------------------------
diff --git a/core/src/test/java/org/apache/cxf/endpoint/DummyServer.java b/core/src/test/java/org/apache/cxf/endpoint/DummyServer.java
index 1c44ab8..b4df864 100644
--- a/core/src/test/java/org/apache/cxf/endpoint/DummyServer.java
+++ b/core/src/test/java/org/apache/cxf/endpoint/DummyServer.java
@@ -30,12 +30,10 @@ public class DummyServer implements Server {
     }
 
     public Destination getDestination() {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public Endpoint getEndpoint() {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/integration/jca/src/test/java/org/apache/cxf/jca/cxf/test/DummyBus.java
----------------------------------------------------------------------
diff --git a/integration/jca/src/test/java/org/apache/cxf/jca/cxf/test/DummyBus.java b/integration/jca/src/test/java/org/apache/cxf/jca/cxf/test/DummyBus.java
index b7e1f7c..0f7430d 100644
--- a/integration/jca/src/test/java/org/apache/cxf/jca/cxf/test/DummyBus.java
+++ b/integration/jca/src/test/java/org/apache/cxf/jca/cxf/test/DummyBus.java
@@ -90,20 +90,17 @@ public class DummyBus extends AbstractBasicInterceptorProvider implements Bus {
 
     //    @Override
     public String getId() {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     public <T> T getConfiguration(AbstractPropertiesHolder props, T defaultValue, Class<T> type) {
-        // TODO Auto-generated method stub
         return null;
     }
 
 
     //    @Override
     public void run() {
-        // TODO Auto-generated method stub
 
     }
 
@@ -142,7 +139,6 @@ public class DummyBus extends AbstractBasicInterceptorProvider implements Bus {
 
 
     public Collection<Feature> getFeatures() {
-        // TODO Auto-generated method stub
         return null;
     }
 
@@ -151,17 +147,14 @@ public class DummyBus extends AbstractBasicInterceptorProvider implements Bus {
     }
 
     public void setId(String i) {
-        // TODO Auto-generated method stub
 
     }
 
     public void setProperties(Map<String, Object> properties) {
-        // TODO Auto-generated method stub
 
     }
 
     public void setFeatures(Collection<? extends Feature> features) {
-        // TODO Auto-generated method stub
 
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/runtime/CorbaStreamReader.java
----------------------------------------------------------------------
diff --git a/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/runtime/CorbaStreamReader.java b/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/runtime/CorbaStreamReader.java
index 7a9c6aa..1ceca95 100644
--- a/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/runtime/CorbaStreamReader.java
+++ b/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/runtime/CorbaStreamReader.java
@@ -103,7 +103,6 @@ public class CorbaStreamReader implements XMLStreamReader {
     }
 
     public void close() throws XMLStreamException {
-        // TODO Auto-generated method stub
     }
 
     public int getAttributeCount() {
@@ -176,7 +175,6 @@ public class CorbaStreamReader implements XMLStreamReader {
     }
 
     public String getCharacterEncodingScheme() {
-        // TODO Auto-generated method stub
         return null;
     }
 
@@ -185,36 +183,29 @@ public class CorbaStreamReader implements XMLStreamReader {
     }
 
     public String getEncoding() {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public Location getLocation() {
-        // TODO Auto-generated method stub
         return new Location() {
 
             public int getCharacterOffset() {
-                // TODO Auto-generated method stub
                 return -1;
             }
 
             public int getColumnNumber() {
-                // TODO Auto-generated method stub
                 return -1;
             }
 
             public int getLineNumber() {
-                // TODO Auto-generated method stub
                 return -1;
             }
 
             public String getPublicId() {
-                // TODO Auto-generated method stub
                 return null;
             }
 
             public String getSystemId() {
-                // TODO Auto-generated method stub
                 return null;
             }
 
@@ -222,7 +213,6 @@ public class CorbaStreamReader implements XMLStreamReader {
     }
 
     public NamespaceContext getNamespaceContext() {
-        // TODO Auto-generated method stub
         return null;
     }
 
@@ -248,7 +238,6 @@ public class CorbaStreamReader implements XMLStreamReader {
     }
 
     public String getNamespaceURI(String arg0) {
-        // TODO Auto-generated method stub
         return null;
     }
 
@@ -266,22 +255,18 @@ public class CorbaStreamReader implements XMLStreamReader {
     }
 
     public String getPIData() {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public String getPITarget() {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public String getPrefix() {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public Object getProperty(String arg0) throws IllegalArgumentException {
-        // TODO Auto-generated method stub
         return null;
     }
 
@@ -290,17 +275,14 @@ public class CorbaStreamReader implements XMLStreamReader {
     }
 
     public int getTextCharacters(int arg0, char[] arg1, int arg2, int arg3) throws XMLStreamException {
-        // TODO Auto-generated method stub
         throw new RuntimeException("Not implemented");
     }
 
     public int getTextStart() {
-        // TODO Auto-generated method stub
         return 0;
     }
 
     public String getVersion() {
-        // TODO Auto-generated method stub
         return null;
     }
 
@@ -309,22 +291,18 @@ public class CorbaStreamReader implements XMLStreamReader {
     }
 
     public boolean hasText() {
-        // TODO Auto-generated method stub
         throw new RuntimeException("Not implemented");
     }
 
     public boolean isAttributeSpecified(int arg0) {
-        // TODO Auto-generated method stub
         return false;
     }
 
     public boolean isStandalone() {
-        // TODO Auto-generated method stub
         return false;
     }
 
     public boolean isWhiteSpace() {
-        // TODO Auto-generated method stub
         return false;
     }
 
@@ -333,11 +311,9 @@ public class CorbaStreamReader implements XMLStreamReader {
     }
 
     public void require(int arg0, String arg1, String arg2) throws XMLStreamException {
-        // TODO Auto-generated method stub
     }
 
     public boolean standaloneSet() {
-        // TODO Auto-generated method stub
         return false;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/utils/CorbaAnyHelper.java
----------------------------------------------------------------------
diff --git a/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/utils/CorbaAnyHelper.java b/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/utils/CorbaAnyHelper.java
index 0f7d135..fe50c1b 100644
--- a/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/utils/CorbaAnyHelper.java
+++ b/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/utils/CorbaAnyHelper.java
@@ -404,7 +404,6 @@ public final class CorbaAnyHelper {
     }
 
     private static void addExtractOverride(ASMHelper helper, ClassWriter cw) {
-        // TODO Auto-generated method stub
         MethodVisitor mv = cw.visitMethod(Opcodes.ACC_PUBLIC, "extract_Streamable",
                             "()Lorg/omg/CORBA/portable/Streamable;", null, null);
         mv.visitCode();

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/util/stax/JDOMStreamWriter.java
----------------------------------------------------------------------
diff --git a/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/util/stax/JDOMStreamWriter.java b/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/util/stax/JDOMStreamWriter.java
index e061e4d..9d48f1c 100644
--- a/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/util/stax/JDOMStreamWriter.java
+++ b/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/util/stax/JDOMStreamWriter.java
@@ -178,7 +178,6 @@ public class JDOMStreamWriter implements XMLStreamWriter {
     }
 
     public void writeCharacters(char[] text, int start, int len) throws XMLStreamException {
-        // TODO Auto-generated method stub
         currentNode.addContent(new String(text, start, len));
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/Customer.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/Customer.java b/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/Customer.java
index e570a87..54ed359 100644
--- a/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/Customer.java
+++ b/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/Customer.java
@@ -514,7 +514,6 @@ public class Customer extends AbstractCustomer implements CustomerInfo {
 
         @Override
         public String marshal(CustomerBean v) throws Exception {
-            // TODO Auto-generated method stub
             return null;
         }
 
@@ -546,7 +545,6 @@ public class Customer extends AbstractCustomer implements CustomerInfo {
 
         @Override
         public ABean marshal(CustomerBean v) throws Exception {
-            // TODO Auto-generated method stub
             return null;
         }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/JAXBContextProvider.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/JAXBContextProvider.java b/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/JAXBContextProvider.java
index fb7f45d..65b3d07 100644
--- a/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/JAXBContextProvider.java
+++ b/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/JAXBContextProvider.java
@@ -27,7 +27,6 @@ import javax.xml.bind.JAXBContext;
 public class JAXBContextProvider implements ContextResolver<JAXBContext> {
 
     public JAXBContext getContext(Class<?> type) {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/JAXBContextProvider2.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/JAXBContextProvider2.java b/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/JAXBContextProvider2.java
index 1bcb3bb..1d9fe12 100644
--- a/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/JAXBContextProvider2.java
+++ b/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/JAXBContextProvider2.java
@@ -27,7 +27,6 @@ import javax.xml.bind.JAXBContext;
 public class JAXBContextProvider2 implements ContextResolver<JAXBContext> {
 
     public JAXBContext getContext(Class<?> type) {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/JAXRSServerFactoryBeanTest.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/JAXRSServerFactoryBeanTest.java b/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/JAXRSServerFactoryBeanTest.java
index 119cd47..73e51c3 100644
--- a/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/JAXRSServerFactoryBeanTest.java
+++ b/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/JAXRSServerFactoryBeanTest.java
@@ -68,7 +68,6 @@ public class JAXRSServerFactoryBeanTest extends Assert {
     private static class CustomExceptionMapper implements ExceptionMapper<Exception> {
 
         public Response toResponse(Exception exception) {
-            // TODO Auto-generated method stub
             return null;
         }
 
@@ -77,7 +76,6 @@ public class JAXRSServerFactoryBeanTest extends Assert {
     private static class RuntimeExceptionMapper implements ExceptionMapper<RuntimeException> {
 
         public Response toResponse(RuntimeException exception) {
-            // TODO Auto-generated method stub
             return null;
         }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/ext/MessageContextImplTest.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/ext/MessageContextImplTest.java b/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/ext/MessageContextImplTest.java
index 8b4122d..3201a5a 100644
--- a/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/ext/MessageContextImplTest.java
+++ b/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/ext/MessageContextImplTest.java
@@ -221,7 +221,6 @@ public class MessageContextImplTest extends Assert {
     public static class CustomContextResolver implements ContextResolver<JAXBContext> {
 
         public JAXBContext getContext(Class<?> type) {
-            // TODO Auto-generated method stub
             return null;
         }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/impl/ConfigurationImplTest.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/impl/ConfigurationImplTest.java b/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/impl/ConfigurationImplTest.java
index de79115..c50bb42 100644
--- a/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/impl/ConfigurationImplTest.java
+++ b/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/impl/ConfigurationImplTest.java
@@ -52,7 +52,6 @@ public class ConfigurationImplTest extends Assert {
         @Override
         public void filter(ContainerRequestContext requestContext, ContainerResponseContext responseContext)
             throws IOException {
-            // TODO Auto-generated method stub
 
         }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/provider/JAXBElementProviderTest.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/provider/JAXBElementProviderTest.java b/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/provider/JAXBElementProviderTest.java
index 6f2b257..a08f9d0 100644
--- a/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/provider/JAXBElementProviderTest.java
+++ b/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/provider/JAXBElementProviderTest.java
@@ -1453,103 +1453,84 @@ public class JAXBElementProviderTest extends Assert {
 
         @SuppressWarnings("rawtypes")
         public <A extends XmlAdapter> A getAdapter(Class<A> type) {
-            // TODO Auto-generated method stub
             return null;
         }
 
         public AttachmentMarshaller getAttachmentMarshaller() {
-            // TODO Auto-generated method stub
             return null;
         }
 
         public ValidationEventHandler getEventHandler() throws JAXBException {
-            // TODO Auto-generated method stub
             return null;
         }
 
         public Listener getListener() {
-            // TODO Auto-generated method stub
             return null;
         }
 
         public Node getNode(Object contentTree) throws JAXBException {
-            // TODO Auto-generated method stub
             return null;
         }
 
         public Object getProperty(String name) throws PropertyException {
-            // TODO Auto-generated method stub
             return null;
         }
 
         public Schema getSchema() {
-            // TODO Auto-generated method stub
             return null;
         }
 
         public void marshal(Object jaxbElement, Result result) throws JAXBException {
-            // TODO Auto-generated method stub
-
+    
         }
 
         public void marshal(Object jaxbElement, OutputStream os) throws JAXBException {
         }
 
         public void marshal(Object jaxbElement, File output) throws JAXBException {
-            // TODO Auto-generated method stub
-
+    
         }
 
         public void marshal(Object jaxbElement, Writer writer) throws JAXBException {
-            // TODO Auto-generated method stub
-
+    
         }
 
         public void marshal(Object jaxbElement, ContentHandler handler) throws JAXBException {
-            // TODO Auto-generated method stub
-
+    
         }
 
         public void marshal(Object jaxbElement, Node node) throws JAXBException {
-            // TODO Auto-generated method stub
-
+    
         }
 
         public void marshal(Object jaxbElement, XMLStreamWriter writer) throws JAXBException {
-            // TODO Auto-generated method stub
-
+    
         }
 
         public void marshal(Object jaxbElement, XMLEventWriter writer) throws JAXBException {
-            // TODO Auto-generated method stub
-
+    
         }
 
         @SuppressWarnings("rawtypes")
         public void setAdapter(XmlAdapter adapter) {
-            // TODO Auto-generated method stub
-
+    
         }
 
         @SuppressWarnings("rawtypes")
         public <A extends XmlAdapter> void setAdapter(Class<A> type, A adapter) {
-            // TODO Auto-generated method stub
-
+    
         }
 
         public void setAttachmentMarshaller(AttachmentMarshaller am) {
-            // TODO Auto-generated method stub
-
+    
         }
 
         public void setEventHandler(ValidationEventHandler handler) throws JAXBException {
-            // TODO Auto-generated method stub
-
+    
         }
 
         public void setListener(Listener listener) {
-            // TODO Auto-generated method stub
-
+    
         }
 
         public void setProperty(String name, Object value) throws PropertyException {
@@ -1558,8 +1539,7 @@ public class JAXBElementProviderTest extends Assert {
         }
 
         public void setSchema(Schema schema) {
-            // TODO Auto-generated method stub
-
+    
         }
 
     }

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/provider/ProviderFactoryTest.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/provider/ProviderFactoryTest.java b/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/provider/ProviderFactoryTest.java
index c4a5d0a..b4d6b91 100644
--- a/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/provider/ProviderFactoryTest.java
+++ b/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/provider/ProviderFactoryTest.java
@@ -823,7 +823,6 @@ public class ProviderFactoryTest extends Assert {
     private static class TestRuntimeExceptionMapper implements ExceptionMapper<RuntimeException> {
 
         public Response toResponse(RuntimeException exception) {
-            // TODO Auto-generated method stub
             return null;
         }
 
@@ -842,13 +841,11 @@ public class ProviderFactoryTest extends Assert {
         public Book readFrom(Class<Book> arg0, Type arg1, Annotation[] arg2,
                              MediaType arg3, MultivaluedMap<String, String> arg4, InputStream arg5)
             throws IOException, WebApplicationException {
-            // TODO Auto-generated method stub
             return null;
         }
 
         public long getSize(Book t, Class<?> type, Type genericType, Annotation[] annotations,
                             MediaType mediaType) {
-            // TODO Auto-generated method stub
             return 0;
         }
 
@@ -860,7 +857,6 @@ public class ProviderFactoryTest extends Assert {
         public void writeTo(Book arg0, Class<?> arg1, Type arg2, Annotation[] arg3,
                             MediaType arg4, MultivaluedMap<String, Object> arg5, OutputStream arg6)
             throws IOException, WebApplicationException {
-            // TODO Auto-generated method stub
 
         }
     }
@@ -878,13 +874,11 @@ public class ProviderFactoryTest extends Assert {
         public SuperBook readFrom(Class<SuperBook> arg0, Type arg1, Annotation[] arg2, MediaType arg3,
                                   MultivaluedMap<String, String> arg4, InputStream arg5)
             throws IOException, WebApplicationException {
-            // TODO Auto-generated method stub
             return null;
         }
 
         public long getSize(SuperBook t, Class<?> type, Type genericType,
                             Annotation[] annotations, MediaType mediaType) {
-            // TODO Auto-generated method stub
             return 0;
         }
 
@@ -896,7 +890,6 @@ public class ProviderFactoryTest extends Assert {
         public void writeTo(SuperBook arg0, Class<?> arg1, Type arg2,
                             Annotation[] arg3, MediaType arg4, MultivaluedMap<String, Object> arg5,
                             OutputStream arg6) throws IOException, WebApplicationException {
-            // TODO Auto-generated method stub
 
         }
 
@@ -915,13 +908,11 @@ public class ProviderFactoryTest extends Assert {
         public T readFrom(Class<T> arg0, Type arg1, Annotation[] arg2, MediaType arg3,
                           MultivaluedMap<String, String> arg4, InputStream arg5)
             throws IOException, WebApplicationException {
-            // TODO Auto-generated method stub
             return null;
         }
 
         public long getSize(T t, Class<?> type, Type genericType,
                             Annotation[] annotations, MediaType mediaType) {
-            // TODO Auto-generated method stub
             return 0;
         }
 
@@ -935,7 +926,6 @@ public class ProviderFactoryTest extends Assert {
                             MediaType mediaType, MultivaluedMap<String, Object> httpHeaders,
                             OutputStream entityStream)
             throws IOException, WebApplicationException {
-            // TODO Auto-generated method stub
         }
 
     }
@@ -952,7 +942,6 @@ public class ProviderFactoryTest extends Assert {
         public Object readFrom(Class<Object> arg0, Type arg1, Annotation[] arg2, MediaType arg3,
                                   MultivaluedMap<String, String> arg4, InputStream arg5)
             throws IOException, WebApplicationException {
-            // TODO Auto-generated method stub
             return null;
         }
 
@@ -969,7 +958,6 @@ public class ProviderFactoryTest extends Assert {
         public Object readFrom(Class<Object> arg0, Type arg1, Annotation[] arg2, MediaType arg3,
                                   MultivaluedMap<String, String> arg4, InputStream arg5)
             throws IOException, WebApplicationException {
-            // TODO Auto-generated method stub
             return null;
         }
 
@@ -989,7 +977,6 @@ public class ProviderFactoryTest extends Assert {
         implements ExceptionMapper<T> {
 
         public Response toResponse(T arg0) {
-            // TODO Auto-generated method stub
             return null;
         }
 
@@ -1008,19 +995,16 @@ public class ProviderFactoryTest extends Assert {
         public Object readFrom(Class<Object> arg0, Type arg1, Annotation[] arg2, MediaType arg3,
                                MultivaluedMap<String, String> arg4, InputStream arg5) throws IOException,
             WebApplicationException {
-            // TODO Auto-generated method stub
             return null;
         }
 
         @Override
         public long getSize(Object arg0, Class<?> arg1, Type arg2, Annotation[] arg3, MediaType arg4) {
-            // TODO Auto-generated method stub
             return 0;
         }
 
         @Override
         public boolean isWriteable(Class<?> arg0, Type arg1, Annotation[] arg2, MediaType arg3) {
-            // TODO Auto-generated method stub
             return false;
         }
 
@@ -1028,7 +1012,6 @@ public class ProviderFactoryTest extends Assert {
         public void writeTo(Object arg0, Class<?> arg1, Type arg2, Annotation[] arg3, MediaType arg4,
                             MultivaluedMap<String, Object> arg5, OutputStream arg6) throws IOException,
             WebApplicationException {
-            // TODO Auto-generated method stub
 
         }
     }

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/resources/BookStore.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/resources/BookStore.java b/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/resources/BookStore.java
index 188049a..6f441f2 100644
--- a/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/resources/BookStore.java
+++ b/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/resources/BookStore.java
@@ -64,12 +64,10 @@ public class BookStore extends BookSuperClass implements BookInterface {
 
     @Override
     public String getDescription() {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public String getAuthor() {
-        // TODO Auto-generated method stub
         return null;
     }
 }

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/utils/JAXRSUtilsTest.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/utils/JAXRSUtilsTest.java b/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/utils/JAXRSUtilsTest.java
index a67b83f..b7181c1 100644
--- a/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/utils/JAXRSUtilsTest.java
+++ b/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/utils/JAXRSUtilsTest.java
@@ -198,7 +198,6 @@ public class JAXRSUtilsTest extends Assert {
         sf.setServiceBeanObjects(customer);
         sf.setProvider(new ContextProvider<CustomerContext>() {
             public CustomerContext createContext(Message message) {
-                // TODO Auto-generated method stub
                 return contextImpl;
             }
         });
@@ -2112,7 +2111,6 @@ public class JAXRSUtilsTest extends Assert {
 
         @Override
         public String toString(MyType<Integer> value) {
-            // TODO Auto-generated method stub
             return null;
         }
 
@@ -2135,7 +2133,6 @@ public class JAXRSUtilsTest extends Assert {
 
         @Override
         public String toString(Locale arg0) throws IllegalArgumentException {
-            // TODO Auto-generated method stub
             return null;
         }
 
@@ -2159,7 +2156,6 @@ public class JAXRSUtilsTest extends Assert {
 
         @Override
         public String toString(Query<String> arg0) throws IllegalArgumentException {
-            // TODO Auto-generated method stub
             return null;
         }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/interceptors/MessageModeOutInterceptor.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/interceptors/MessageModeOutInterceptor.java b/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/interceptors/MessageModeOutInterceptor.java
index 0f49a51..7efb556 100644
--- a/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/interceptors/MessageModeOutInterceptor.java
+++ b/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/interceptors/MessageModeOutInterceptor.java
@@ -131,7 +131,6 @@ public class MessageModeOutInterceptor extends AbstractPhaseInterceptor<Message>
                     }
 
                     public OutputStream getOutputStream() throws IOException {
-                        // TODO Auto-generated method stub
                         return null;
                     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/ConfiguredEndpointTest.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/ConfiguredEndpointTest.java b/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/ConfiguredEndpointTest.java
index 57aeb59..933bc01 100644
--- a/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/ConfiguredEndpointTest.java
+++ b/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/ConfiguredEndpointTest.java
@@ -371,7 +371,6 @@ public class ConfiguredEndpointTest extends Assert {
         }
 
         public void handleMessage(Message message) throws Fault {
-            // TODO Auto-generated method stub
         }
     }
 }

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/handler/HandlerChainInvokerTest.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/handler/HandlerChainInvokerTest.java b/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/handler/HandlerChainInvokerTest.java
index c14b4d9..6a1f997 100644
--- a/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/handler/HandlerChainInvokerTest.java
+++ b/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/handler/HandlerChainInvokerTest.java
@@ -1079,12 +1079,10 @@ public class HandlerChainInvokerTest extends Assert {
 
 
         public void init(Map<String, Object> arg0) {
-            // TODO Auto-generated method stub
         }
 
 
         public void destroy() {
-            // TODO Auto-generated method stub
         }
 
         public int getHandleMessageCount() {

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/service/FooServiceImpl.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/service/FooServiceImpl.java b/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/service/FooServiceImpl.java
index cde4f92..515bb23 100644
--- a/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/service/FooServiceImpl.java
+++ b/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/service/FooServiceImpl.java
@@ -28,7 +28,6 @@ public class FooServiceImpl implements FooService {
 //    }
 
     public Foo echo2(Foo foo, String name) {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/service/Hello2.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/service/Hello2.java b/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/service/Hello2.java
index 369d285a..7697488 100644
--- a/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/service/Hello2.java
+++ b/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/service/Hello2.java
@@ -42,19 +42,16 @@ public class Hello2 implements HelloInterface {
     }
 
     public String[] getStringArray(String[] strs) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public List<String> getStringList(List<String> list) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     @Override
     @WebMethod
     public String addNumbers(int arg0, int arg1) throws AddNumbersException {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/service/Hello3.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/service/Hello3.java b/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/service/Hello3.java
index 639b3f8..db12dba 100644
--- a/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/service/Hello3.java
+++ b/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/service/Hello3.java
@@ -45,19 +45,16 @@ public class Hello3 implements HelloInterface {
     }
 
     public String[] getStringArray(String[] strs) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public List<String> getStringList(List<String> list) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     @Override
     @WebMethod
     public String addNumbers(int arg0, int arg1) throws AddNumbersException {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/frontend/simple/src/test/java/org/apache/cxf/service/factory/ServerFactoryTest.java
----------------------------------------------------------------------
diff --git a/rt/frontend/simple/src/test/java/org/apache/cxf/service/factory/ServerFactoryTest.java b/rt/frontend/simple/src/test/java/org/apache/cxf/service/factory/ServerFactoryTest.java
index 86a54a7..c957fc0 100644
--- a/rt/frontend/simple/src/test/java/org/apache/cxf/service/factory/ServerFactoryTest.java
+++ b/rt/frontend/simple/src/test/java/org/apache/cxf/service/factory/ServerFactoryTest.java
@@ -116,27 +116,22 @@ public class ServerFactoryTest extends AbstractSimpleFrontendTest {
     public static class CustomDestination implements Destination {
 
         public EndpointReferenceType getAddress() {
-            // TODO Auto-generated method stub
             return null;
         }
 
         public Conduit getBackChannel(Message inMessage) throws IOException {
-            // TODO Auto-generated method stub
             return null;
         }
 
         public void shutdown() {
-            // TODO Auto-generated method stub
 
         }
 
         public void setMessageObserver(MessageObserver observer) {
-            // TODO Auto-generated method stub
 
         }
 
         public MessageObserver getMessageObserver() {
-            // TODO Auto-generated method stub
             return null;
         }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/rs/client/src/test/java/org/apache/cxf/jaxrs/client/WebClientTest.java
----------------------------------------------------------------------
diff --git a/rt/rs/client/src/test/java/org/apache/cxf/jaxrs/client/WebClientTest.java b/rt/rs/client/src/test/java/org/apache/cxf/jaxrs/client/WebClientTest.java
index 36e1262..0eaceb4 100644
--- a/rt/rs/client/src/test/java/org/apache/cxf/jaxrs/client/WebClientTest.java
+++ b/rt/rs/client/src/test/java/org/apache/cxf/jaxrs/client/WebClientTest.java
@@ -333,7 +333,6 @@ public class WebClientTest extends Assert {
 
         @Override
         public ComplexObject fromString(String value) throws IllegalArgumentException {
-            // TODO Auto-generated method stub
             return null;
         }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/rs/client/src/test/java/org/apache/cxf/jaxrs/resources/BookStore.java
----------------------------------------------------------------------
diff --git a/rt/rs/client/src/test/java/org/apache/cxf/jaxrs/resources/BookStore.java b/rt/rs/client/src/test/java/org/apache/cxf/jaxrs/resources/BookStore.java
index 188049a..6f441f2 100644
--- a/rt/rs/client/src/test/java/org/apache/cxf/jaxrs/resources/BookStore.java
+++ b/rt/rs/client/src/test/java/org/apache/cxf/jaxrs/resources/BookStore.java
@@ -64,12 +64,10 @@ public class BookStore extends BookSuperClass implements BookInterface {
 
     @Override
     public String getDescription() {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public String getAuthor() {
-        // TODO Auto-generated method stub
         return null;
     }
 }

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/rs/security/oauth-parent/oauth2/src/test/java/org/apache/cxf/rs/security/oauth2/grants/OAuthDataProviderImpl.java
----------------------------------------------------------------------
diff --git a/rt/rs/security/oauth-parent/oauth2/src/test/java/org/apache/cxf/rs/security/oauth2/grants/OAuthDataProviderImpl.java b/rt/rs/security/oauth-parent/oauth2/src/test/java/org/apache/cxf/rs/security/oauth2/grants/OAuthDataProviderImpl.java
index f9e024a..767cb9a 100644
--- a/rt/rs/security/oauth-parent/oauth2/src/test/java/org/apache/cxf/rs/security/oauth2/grants/OAuthDataProviderImpl.java
+++ b/rt/rs/security/oauth-parent/oauth2/src/test/java/org/apache/cxf/rs/security/oauth2/grants/OAuthDataProviderImpl.java
@@ -46,7 +46,6 @@ public class OAuthDataProviderImpl implements OAuthDataProvider {
 
     @Override
     public ServerAccessToken getAccessToken(String accessToken) throws OAuthServiceException {
-        // TODO Auto-generated method stub
         return null;
     }
 
@@ -54,38 +53,32 @@ public class OAuthDataProviderImpl implements OAuthDataProvider {
     public ServerAccessToken getPreauthorizedToken(Client client, List<String> requestedScopes,
                                                    UserSubject subject, String grantType)
         throws OAuthServiceException {
-        // TODO Auto-generated method stub
         return null;
     }
 
     @Override
     public ServerAccessToken refreshAccessToken(Client client, String refreshToken,
                                                 List<String> requestedScopes) throws OAuthServiceException {
-        // TODO Auto-generated method stub
         return null;
     }
 
     @Override
     public List<OAuthPermission> convertScopeToPermissions(Client client, List<String> requestedScope) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     @Override
     public void revokeToken(Client client, String token, String tokenTypeHint) throws OAuthServiceException {
-        // TODO Auto-generated method stub
 
     }
 
     @Override
     public List<ServerAccessToken> getAccessTokens(Client client, UserSubject sub) throws OAuthServiceException {
-        // TODO Auto-generated method stub
         return null;
     }
 
     @Override
     public List<RefreshToken> getRefreshTokens(Client client, UserSubject sub) throws OAuthServiceException {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/rs/security/oauth-parent/oauth2/src/test/java/org/apache/cxf/rs/security/oauth2/utils/crypto/CodeGrantEncryptingDataProvider.java
----------------------------------------------------------------------
diff --git a/rt/rs/security/oauth-parent/oauth2/src/test/java/org/apache/cxf/rs/security/oauth2/utils/crypto/CodeGrantEncryptingDataProvider.java b/rt/rs/security/oauth-parent/oauth2/src/test/java/org/apache/cxf/rs/security/oauth2/utils/crypto/CodeGrantEncryptingDataProvider.java
index dbaaaa3..919b2ab 100644
--- a/rt/rs/security/oauth-parent/oauth2/src/test/java/org/apache/cxf/rs/security/oauth2/utils/crypto/CodeGrantEncryptingDataProvider.java
+++ b/rt/rs/security/oauth-parent/oauth2/src/test/java/org/apache/cxf/rs/security/oauth2/utils/crypto/CodeGrantEncryptingDataProvider.java
@@ -58,7 +58,6 @@ public class CodeGrantEncryptingDataProvider extends EncryptingDataProvider
 
     @Override
     public List<ServerAuthorizationCodeGrant> getCodeGrants(Client c, UserSubject sub) throws OAuthServiceException {
-        // TODO Auto-generated method stub
         return null;
     }
 }

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/rs/security/oauth-parent/oauth2/src/test/java/org/apache/cxf/rs/security/oauth2/utils/crypto/EncryptingDataProvider.java
----------------------------------------------------------------------
diff --git a/rt/rs/security/oauth-parent/oauth2/src/test/java/org/apache/cxf/rs/security/oauth2/utils/crypto/EncryptingDataProvider.java b/rt/rs/security/oauth-parent/oauth2/src/test/java/org/apache/cxf/rs/security/oauth2/utils/crypto/EncryptingDataProvider.java
index a3357d5..4915290 100644
--- a/rt/rs/security/oauth-parent/oauth2/src/test/java/org/apache/cxf/rs/security/oauth2/utils/crypto/EncryptingDataProvider.java
+++ b/rt/rs/security/oauth-parent/oauth2/src/test/java/org/apache/cxf/rs/security/oauth2/utils/crypto/EncryptingDataProvider.java
@@ -145,13 +145,11 @@ public class EncryptingDataProvider implements OAuthDataProvider {
 
     @Override
     public List<ServerAccessToken> getAccessTokens(Client client, UserSubject sub) throws OAuthServiceException {
-        // TODO Auto-generated method stub
         return null;
     }
 
     @Override
     public List<RefreshToken> getRefreshTokens(Client client, UserSubject sub) throws OAuthServiceException {
-        // TODO Auto-generated method stub
         return null;
     }
 }

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/rs/security/sso/saml/src/main/java/org/apache/cxf/rs/security/saml/sso/state/HTTPSPStateManager.java
----------------------------------------------------------------------
diff --git a/rt/rs/security/sso/saml/src/main/java/org/apache/cxf/rs/security/saml/sso/state/HTTPSPStateManager.java b/rt/rs/security/sso/saml/src/main/java/org/apache/cxf/rs/security/saml/sso/state/HTTPSPStateManager.java
index fc41b56..3238cef 100644
--- a/rt/rs/security/sso/saml/src/main/java/org/apache/cxf/rs/security/saml/sso/state/HTTPSPStateManager.java
+++ b/rt/rs/security/sso/saml/src/main/java/org/apache/cxf/rs/security/saml/sso/state/HTTPSPStateManager.java
@@ -78,7 +78,5 @@ public class HTTPSPStateManager implements SPStateManager {
     @POST
     @Path("close")
     public void close() throws IOException {
-        // TODO Auto-generated method stub
-
     }
 }

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/transports/http-netty/netty-client/src/main/java/org/apache/cxf/transport/http/netty/client/NettyHttpConduit.java
----------------------------------------------------------------------
diff --git a/rt/transports/http-netty/netty-client/src/main/java/org/apache/cxf/transport/http/netty/client/NettyHttpConduit.java b/rt/transports/http-netty/netty-client/src/main/java/org/apache/cxf/transport/http/netty/client/NettyHttpConduit.java
index 06ec717..0dcd6e8 100644
--- a/rt/transports/http-netty/netty-client/src/main/java/org/apache/cxf/transport/http/netty/client/NettyHttpConduit.java
+++ b/rt/transports/http-netty/netty-client/src/main/java/org/apache/cxf/transport/http/netty/client/NettyHttpConduit.java
@@ -621,7 +621,6 @@ public class NettyHttpConduit extends URLConnectionHTTPConduit implements BusLif
 
     @Override
     public void initComplete() {
-        // TODO Auto-generated method stub
 
     }
 
@@ -634,8 +633,6 @@ public class NettyHttpConduit extends URLConnectionHTTPConduit implements BusLif
 
     @Override
     public void preShutdown() {
-        // TODO Auto-generated method stub
-
     }
 
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/transports/websocket/src/main/java/org/apache/cxf/transport/websocket/jetty/WebSocketVirtualServletRequest.java
----------------------------------------------------------------------
diff --git a/rt/transports/websocket/src/main/java/org/apache/cxf/transport/websocket/jetty/WebSocketVirtualServletRequest.java b/rt/transports/websocket/src/main/java/org/apache/cxf/transport/websocket/jetty/WebSocketVirtualServletRequest.java
index 44de7bd..a5906d7 100644
--- a/rt/transports/websocket/src/main/java/org/apache/cxf/transport/websocket/jetty/WebSocketVirtualServletRequest.java
+++ b/rt/transports/websocket/src/main/java/org/apache/cxf/transport/websocket/jetty/WebSocketVirtualServletRequest.java
@@ -106,7 +106,6 @@ public class WebSocketVirtualServletRequest implements HttpServletRequest {
 
     @Override
     public String getCharacterEncoding() {
-        // TODO Auto-generated method stub
         LOG.log(Level.FINE, "getCharacterEncoding()");
         return null;
     }
@@ -191,7 +190,6 @@ public class WebSocketVirtualServletRequest implements HttpServletRequest {
 
     @Override
     public String getParameter(String name) {
-        // TODO Auto-generated method stub
         if (LOG.isLoggable(Level.FINE)) {
             LOG.log(Level.FINE, "getParameter({0})", name);
         }
@@ -200,21 +198,18 @@ public class WebSocketVirtualServletRequest implements HttpServletRequest {
 
     @Override
     public Map<String, String[]> getParameterMap() {
-        // TODO Auto-generated method stub
         LOG.log(Level.FINE, "getParameterMap()");
         return null;
     }
 
     @Override
     public Enumeration<String> getParameterNames() {
-        // TODO Auto-generated method stub
         LOG.log(Level.FINE, "getParameterNames()");
         return null;
     }
 
     @Override
     public String[] getParameterValues(String name) {
-        // TODO Auto-generated method stub
         if (LOG.isLoggable(Level.FINE)) {
             LOG.log(Level.FINE, "getParameterValues({0})", name);
         }
@@ -235,7 +230,6 @@ public class WebSocketVirtualServletRequest implements HttpServletRequest {
 
     @Override
     public String getRealPath(String path) {
-        // TODO Auto-generated method stub
         LOG.log(Level.FINE, "getRealPath");
         return null;
     }
@@ -260,7 +254,6 @@ public class WebSocketVirtualServletRequest implements HttpServletRequest {
 
     @Override
     public RequestDispatcher getRequestDispatcher(String path) {
-        // TODO Auto-generated method stub
         LOG.log(Level.FINE, "getRequestDispatcher");
         return null;
     }
@@ -332,14 +325,12 @@ public class WebSocketVirtualServletRequest implements HttpServletRequest {
 
     @Override
     public AsyncContext startAsync(ServletRequest servletRequest, ServletResponse servletResponse) {
-        // TODO Auto-generated method stub
         LOG.log(Level.FINE, "startAsync");
         return null;
     }
 
     @Override
     public boolean authenticate(HttpServletResponse servletResponse) throws IOException, ServletException {
-        // TODO Auto-generated method stub
         LOG.log(Level.FINE, "authenticate");
         return false;
     }

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/transports/websocket/src/main/java/org/apache/cxf/transport/websocket/jetty9/Jetty9WebSocketDestination.java
----------------------------------------------------------------------
diff --git a/rt/transports/websocket/src/main/java/org/apache/cxf/transport/websocket/jetty9/Jetty9WebSocketDestination.java b/rt/transports/websocket/src/main/java/org/apache/cxf/transport/websocket/jetty9/Jetty9WebSocketDestination.java
index 2c332ff..5cad28c 100644
--- a/rt/transports/websocket/src/main/java/org/apache/cxf/transport/websocket/jetty9/Jetty9WebSocketDestination.java
+++ b/rt/transports/websocket/src/main/java/org/apache/cxf/transport/websocket/jetty9/Jetty9WebSocketDestination.java
@@ -146,7 +146,6 @@ public class Jetty9WebSocketDestination extends JettyHTTPDestination implements
                 webSockethandler.setServer(server);
                 webSockethandler.start();
             } catch (Exception e) {
-                // TODO Auto-generated catch block
                 e.printStackTrace();
             }
             webSocketFactory = webSockethandler.getWebSocketFactory();

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/transports/websocket/src/main/java/org/apache/cxf/transport/websocket/undertow/WebSocketUndertowServletRequest.java
----------------------------------------------------------------------
diff --git a/rt/transports/websocket/src/main/java/org/apache/cxf/transport/websocket/undertow/WebSocketUndertowServletRequest.java b/rt/transports/websocket/src/main/java/org/apache/cxf/transport/websocket/undertow/WebSocketUndertowServletRequest.java
index 5a2745c..7aa3e76 100644
--- a/rt/transports/websocket/src/main/java/org/apache/cxf/transport/websocket/undertow/WebSocketUndertowServletRequest.java
+++ b/rt/transports/websocket/src/main/java/org/apache/cxf/transport/websocket/undertow/WebSocketUndertowServletRequest.java
@@ -117,7 +117,6 @@ public class WebSocketUndertowServletRequest implements HttpServletRequest {
 
     @Override
     public String getCharacterEncoding() {
-        // TODO Auto-generated method stub
         LOG.log(Level.FINE, "getCharacterEncoding()");
         return null;
     }
@@ -217,7 +216,6 @@ public class WebSocketUndertowServletRequest implements HttpServletRequest {
 
     @Override
     public String getParameter(String name) {
-        // TODO Auto-generated method stub
         if (LOG.isLoggable(Level.FINE)) {
             LOG.log(Level.FINE, "getParameter({0})", name);
         }
@@ -226,21 +224,18 @@ public class WebSocketUndertowServletRequest implements HttpServletRequest {
 
     @Override
     public Map<String, String[]> getParameterMap() {
-        // TODO Auto-generated method stub
         LOG.log(Level.FINE, "getParameterMap()");
         return null;
     }
 
     @Override
     public Enumeration<String> getParameterNames() {
-        // TODO Auto-generated method stub
         LOG.log(Level.FINE, "getParameterNames()");
         return null;
     }
 
     @Override
     public String[] getParameterValues(String name) {
-        // TODO Auto-generated method stub
         if (LOG.isLoggable(Level.FINE)) {
             LOG.log(Level.FINE, "getParameterValues({0})", name);
         }
@@ -266,7 +261,6 @@ public class WebSocketUndertowServletRequest implements HttpServletRequest {
 
     @Override
     public String getRealPath(String path) {
-        // TODO Auto-generated method stub
         LOG.log(Level.FINE, "getRealPath");
         return null;
     }
@@ -306,7 +300,6 @@ public class WebSocketUndertowServletRequest implements HttpServletRequest {
 
     @Override
     public RequestDispatcher getRequestDispatcher(String path) {
-        // TODO Auto-generated method stub
         LOG.log(Level.FINE, "getRequestDispatcher");
         return null;
     }
@@ -383,14 +376,12 @@ public class WebSocketUndertowServletRequest implements HttpServletRequest {
 
     @Override
     public AsyncContext startAsync(ServletRequest servletRequest, ServletResponse servletResponse) {
-        // TODO Auto-generated method stub
         LOG.log(Level.FINE, "startAsync");
         return null;
     }
 
     @Override
     public boolean authenticate(HttpServletResponse servletResponse) throws IOException, ServletException {
-        // TODO Auto-generated method stub
         LOG.log(Level.FINE, "authenticate");
         return false;
     }

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/ws/policy/src/test/java/org/apache/cxf/ws/policy/TestAssertion.java
----------------------------------------------------------------------
diff --git a/rt/ws/policy/src/test/java/org/apache/cxf/ws/policy/TestAssertion.java b/rt/ws/policy/src/test/java/org/apache/cxf/ws/policy/TestAssertion.java
index 172c75a..4f1443d 100644
--- a/rt/ws/policy/src/test/java/org/apache/cxf/ws/policy/TestAssertion.java
+++ b/rt/ws/policy/src/test/java/org/apache/cxf/ws/policy/TestAssertion.java
@@ -66,7 +66,6 @@ public class TestAssertion implements PolicyAssertion {
     }
 
     public boolean isAsserted(AssertionInfoMap aim) {
-        // TODO Auto-generated method stub
         return false;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RedeliveryQueueImpl.java
----------------------------------------------------------------------
diff --git a/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RedeliveryQueueImpl.java b/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RedeliveryQueueImpl.java
index dc8af3a..2a28b25 100644
--- a/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RedeliveryQueueImpl.java
+++ b/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RedeliveryQueueImpl.java
@@ -224,7 +224,6 @@ public class RedeliveryQueueImpl implements RedeliveryQueue {
 
 
     public void start() {
-        // TODO Auto-generated method stub
 
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/ws/rm/src/test/java/org/apache/cxf/ws/rm/ManagedRMManagerTest.java
----------------------------------------------------------------------
diff --git a/rt/ws/rm/src/test/java/org/apache/cxf/ws/rm/ManagedRMManagerTest.java b/rt/ws/rm/src/test/java/org/apache/cxf/ws/rm/ManagedRMManagerTest.java
index f845311..b05d74e 100755
--- a/rt/ws/rm/src/test/java/org/apache/cxf/ws/rm/ManagedRMManagerTest.java
+++ b/rt/ws/rm/src/test/java/org/apache/cxf/ws/rm/ManagedRMManagerTest.java
@@ -487,11 +487,9 @@ public class ManagedRMManagerTest extends Assert {
         }
 
         public void addUnacknowledged(Message message) {
-            // TODO Auto-generated method stub
         }
 
         public void purgeAcknowledged(SourceSequence seq) {
-            // TODO Auto-generated method stub
         }
 
         public void purgeAll(SourceSequence seq) {
@@ -515,16 +513,13 @@ public class ManagedRMManagerTest extends Assert {
         }
 
         public Map<Long, RetryStatus> getRetransmissionStatuses(SourceSequence seq) {
-            // TODO Auto-generated method stub
             return null;
         }
 
         public void start() {
-            // TODO Auto-generated method stub
         }
 
         public void stop(SourceSequence seq) {
-            // TODO Auto-generated method stub
         }
 
         public void suspend(SourceSequence seq) {

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/ws/rm/src/test/java/org/apache/cxf/ws/rm/RMManagerConfigurationTest.java
----------------------------------------------------------------------
diff --git a/rt/ws/rm/src/test/java/org/apache/cxf/ws/rm/RMManagerConfigurationTest.java b/rt/ws/rm/src/test/java/org/apache/cxf/ws/rm/RMManagerConfigurationTest.java
index aa0ca59..828a88a 100644
--- a/rt/ws/rm/src/test/java/org/apache/cxf/ws/rm/RMManagerConfigurationTest.java
+++ b/rt/ws/rm/src/test/java/org/apache/cxf/ws/rm/RMManagerConfigurationTest.java
@@ -105,62 +105,50 @@ public class RMManagerConfigurationTest extends Assert {
 
 
         public void createDestinationSequence(DestinationSequence seq) {
-            // TODO Auto-generated method stub
-
+    
         }
 
         public void createSourceSequence(SourceSequence seq) {
-            // TODO Auto-generated method stub
-
+    
         }
 
         public Collection<DestinationSequence> getDestinationSequences(String endpointIdentifier) {
-            // TODO Auto-generated method stub
             return null;
         }
 
         public Collection<RMMessage> getMessages(Identifier sid, boolean outbound) {
-            // TODO Auto-generated method stub
             return null;
         }
 
         public Collection<SourceSequence> getSourceSequences(String endpointIdentifier) {
-            // TODO Auto-generated method stub
             return null;
         }
 
         public void persistIncoming(DestinationSequence seq, RMMessage msg) {
-            // TODO Auto-generated method stub
-
+    
         }
 
         public void persistOutgoing(SourceSequence seq, RMMessage msg) {
-            // TODO Auto-generated method stub
-
+    
         }
 
         public void removeDestinationSequence(Identifier seq) {
-            // TODO Auto-generated method stub
-
+    
         }
 
         public void removeMessages(Identifier sid, Collection<Long> messageNrs, boolean outbound) {
-            // TODO Auto-generated method stub
-
+    
         }
 
         public void removeSourceSequence(Identifier seq) {
-            // TODO Auto-generated method stub
-
+    
         }
 
         public SourceSequence getSourceSequence(Identifier seq) {
-            // TODO Auto-generated method stub
             return null;
         }
 
         public DestinationSequence getDestinationSequence(Identifier seq) {
-            // TODO Auto-generated method stub
             return null;
         }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/rt/wsdl/src/main/java/org/apache/cxf/wsdl/JAXBExtensionHelper.java
----------------------------------------------------------------------
diff --git a/rt/wsdl/src/main/java/org/apache/cxf/wsdl/JAXBExtensionHelper.java b/rt/wsdl/src/main/java/org/apache/cxf/wsdl/JAXBExtensionHelper.java
index b02f434..162a6c4 100644
--- a/rt/wsdl/src/main/java/org/apache/cxf/wsdl/JAXBExtensionHelper.java
+++ b/rt/wsdl/src/main/java/org/apache/cxf/wsdl/JAXBExtensionHelper.java
@@ -286,7 +286,6 @@ public class JAXBExtensionHelper implements ExtensionSerializer, ExtensionDeseri
     public void marshall(@SuppressWarnings("rawtypes") Class parent, QName qname,
                          ExtensibilityElement obj, PrintWriter pw,
                          final Definition wsdl, ExtensionRegistry registry) throws WSDLException {
-        // TODO Auto-generated method stub
         try {
             Marshaller u = createMarshaller();
             u.setProperty("jaxb.encoding", StandardCharsets.UTF_8.name());

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/databinding/src/test/java/org/apache/cxf/systest/source/GreeterImpl.java
----------------------------------------------------------------------
diff --git a/systests/databinding/src/test/java/org/apache/cxf/systest/source/GreeterImpl.java b/systests/databinding/src/test/java/org/apache/cxf/systest/source/GreeterImpl.java
index 5b582ce..582b953 100644
--- a/systests/databinding/src/test/java/org/apache/cxf/systest/source/GreeterImpl.java
+++ b/systests/databinding/src/test/java/org/apache/cxf/systest/source/GreeterImpl.java
@@ -36,7 +36,6 @@ import org.apache.hello_world_soap_http_source.source.PingMeFault;
 public class GreeterImpl implements Greeter {
 
     public void greetMeOneWay(DOMSource in) {
-        // TODO Auto-generated method stub
 
     }
 
@@ -87,7 +86,6 @@ public class GreeterImpl implements Greeter {
     }
 
     public void pingMe() throws PingMeFault {
-        // TODO Auto-generated method stub
 
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/databinding/src/test/java/org/apache/cxf/systest/source/PutLastTradePriceImpl.java
----------------------------------------------------------------------
diff --git a/systests/databinding/src/test/java/org/apache/cxf/systest/source/PutLastTradePriceImpl.java b/systests/databinding/src/test/java/org/apache/cxf/systest/source/PutLastTradePriceImpl.java
index 6c55ba8..81ca0b9 100644
--- a/systests/databinding/src/test/java/org/apache/cxf/systest/source/PutLastTradePriceImpl.java
+++ b/systests/databinding/src/test/java/org/apache/cxf/systest/source/PutLastTradePriceImpl.java
@@ -35,22 +35,18 @@ import org.apache.cxf.source.doc_lit_bare.PutLastTradedPricePortType;
 public class PutLastTradePriceImpl implements PutLastTradedPricePortType {
 
     public DOMSource nillableParameter(DOMSource theRequest) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public void sayHi(Holder<DOMSource> body) {
-        // TODO Auto-generated method stub
 
     }
 
     public void putLastTradedPrice(DOMSource body) {
-        // TODO Auto-generated method stub
 
     }
 
     public DOMSource bareNoParam() {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/BookServer.java
----------------------------------------------------------------------
diff --git a/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/BookServer.java b/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/BookServer.java
index b784475..5e9e1fb 100644
--- a/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/BookServer.java
+++ b/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/BookServer.java
@@ -264,8 +264,6 @@ public class BookServer extends AbstractBusTestServerBase {
         @Override
         public void filter(ClientRequestContext requestContext, ClientResponseContext responseContext)
             throws IOException {
-            // TODO Auto-generated method stub
-
         }
 
     }

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/CustomResponse.java
----------------------------------------------------------------------
diff --git a/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/CustomResponse.java b/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/CustomResponse.java
index a78f3fd..84139ff 100644
--- a/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/CustomResponse.java
+++ b/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/CustomResponse.java
@@ -45,22 +45,18 @@ public class CustomResponse extends Response {
     }
     @Override
     public void close() {
-        // TODO Auto-generated method stub
 
     }
     @Override
     public Set<String> getAllowedMethods() {
-        // TODO Auto-generated method stub
         return null;
     }
     @Override
     public Map<String, NewCookie> getCookies() {
-        // TODO Auto-generated method stub
         return null;
     }
     @Override
     public Date getDate() {
-        // TODO Auto-generated method stub
         return null;
     }
     @Override
@@ -69,52 +65,42 @@ public class CustomResponse extends Response {
     }
     @Override
     public EntityTag getEntityTag() {
-        // TODO Auto-generated method stub
         return null;
     }
     @Override
     public String getHeaderString(String arg0) {
-        // TODO Auto-generated method stub
         return null;
     }
     @Override
     public Locale getLanguage() {
-        // TODO Auto-generated method stub
         return null;
     }
     @Override
     public Date getLastModified() {
-        // TODO Auto-generated method stub
         return null;
     }
     @Override
     public int getLength() {
-        // TODO Auto-generated method stub
         return 0;
     }
     @Override
     public Link getLink(String arg0) {
-        // TODO Auto-generated method stub
         return null;
     }
     @Override
     public Builder getLinkBuilder(String arg0) {
-        // TODO Auto-generated method stub
         return null;
     }
     @Override
     public Set<Link> getLinks() {
-        // TODO Auto-generated method stub
         return null;
     }
     @Override
     public URI getLocation() {
-        // TODO Auto-generated method stub
         return null;
     }
     @Override
     public MediaType getMediaType() {
-        // TODO Auto-generated method stub
         return null;
     }
     @Override
@@ -127,42 +113,34 @@ public class CustomResponse extends Response {
     }
     @Override
     public StatusType getStatusInfo() {
-        // TODO Auto-generated method stub
         return null;
     }
     @Override
     public MultivaluedMap<String, String> getStringHeaders() {
-        // TODO Auto-generated method stub
         return null;
     }
     @Override
     public boolean hasEntity() {
-        // TODO Auto-generated method stub
         return false;
     }
     @Override
     public boolean hasLink(String arg0) {
-        // TODO Auto-generated method stub
         return false;
     }
     @Override
     public <T> T readEntity(Class<T> arg0) {
-        // TODO Auto-generated method stub
         return null;
     }
     @Override
     public <T> T readEntity(GenericType<T> arg0) {
-        // TODO Auto-generated method stub
         return null;
     }
     @Override
     public <T> T readEntity(Class<T> arg0, Annotation[] arg1) {
-        // TODO Auto-generated method stub
         return null;
     }
     @Override
     public <T> T readEntity(GenericType<T> arg0, Annotation[] arg1) {
-        // TODO Auto-generated method stub
         return null;
     }
 }

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRS20ClientServerBookTest.java
----------------------------------------------------------------------
diff --git a/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRS20ClientServerBookTest.java b/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRS20ClientServerBookTest.java
index 14d81d4..1d1fdf5 100644
--- a/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRS20ClientServerBookTest.java
+++ b/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRS20ClientServerBookTest.java
@@ -539,7 +539,6 @@ public class JAXRS20ClientServerBookTest extends AbstractBusClientServerTestBase
 
         @Override
         public void failed(Throwable arg0) {
-            // TODO Auto-generated method stub
 
         }
 
@@ -558,7 +557,6 @@ public class JAXRS20ClientServerBookTest extends AbstractBusClientServerTestBase
 
         @Override
         public void failed(Throwable arg0) {
-            // TODO Auto-generated method stub
 
         }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSAsyncClientTest.java
----------------------------------------------------------------------
diff --git a/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSAsyncClientTest.java b/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSAsyncClientTest.java
index d0dc048..81d981c 100644
--- a/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSAsyncClientTest.java
+++ b/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSAsyncClientTest.java
@@ -404,7 +404,6 @@ public class JAXRSAsyncClientTest extends AbstractBusClientServerTestBase {
 
         @Override
         public long getSize(Book arg0, Class<?> arg1, Type arg2, Annotation[] arg3, MediaType arg4) {
-            // TODO Auto-generated method stub
             return 0;
         }
 
@@ -445,7 +444,6 @@ public class JAXRSAsyncClientTest extends AbstractBusClientServerTestBase {
         @Override
         public void filter(ClientRequestContext requestContext, ClientResponseContext responseContext)
             throws IOException {
-            // TODO Auto-generated method stub
 
         }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/jaxws/src/test/java/org/apache/cxf/systest/jaxws/ClientServerTest.java
----------------------------------------------------------------------
diff --git a/systests/jaxws/src/test/java/org/apache/cxf/systest/jaxws/ClientServerTest.java b/systests/jaxws/src/test/java/org/apache/cxf/systest/jaxws/ClientServerTest.java
index 5003b15..f013933 100644
--- a/systests/jaxws/src/test/java/org/apache/cxf/systest/jaxws/ClientServerTest.java
+++ b/systests/jaxws/src/test/java/org/apache/cxf/systest/jaxws/ClientServerTest.java
@@ -955,7 +955,6 @@ public class ClientServerTest extends AbstractBusClientServerTestBase {
         InvocationHandler handler = new InvocationHandler() {
 
             public Object invoke(Object proxy, Method method, Object[] args) throws Throwable {
-                // TODO Auto-generated method stub
                 return null;
             }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/jaxws/src/test/java/org/apache/cxf/systest/jaxws/cxf5064/SOAPHeaderServiceImpl.java
----------------------------------------------------------------------
diff --git a/systests/jaxws/src/test/java/org/apache/cxf/systest/jaxws/cxf5064/SOAPHeaderServiceImpl.java b/systests/jaxws/src/test/java/org/apache/cxf/systest/jaxws/cxf5064/SOAPHeaderServiceImpl.java
index 792cbab..5faa64e 100644
--- a/systests/jaxws/src/test/java/org/apache/cxf/systest/jaxws/cxf5064/SOAPHeaderServiceImpl.java
+++ b/systests/jaxws/src/test/java/org/apache/cxf/systest/jaxws/cxf5064/SOAPHeaderServiceImpl.java
@@ -28,7 +28,6 @@ public class SOAPHeaderServiceImpl implements SOAPHeaderSEI {
      *
      */
     public SOAPHeaderServiceImpl() {
-        // TODO Auto-generated constructor stub
     }
 
     public String test(HeaderObj headerObj) {

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/jaxws/src/test/java/org/apache/cxf/systest/jaxws/types/BarImpl.java
----------------------------------------------------------------------
diff --git a/systests/jaxws/src/test/java/org/apache/cxf/systest/jaxws/types/BarImpl.java b/systests/jaxws/src/test/java/org/apache/cxf/systest/jaxws/types/BarImpl.java
index 97d0d59..4863ac8 100644
--- a/systests/jaxws/src/test/java/org/apache/cxf/systest/jaxws/types/BarImpl.java
+++ b/systests/jaxws/src/test/java/org/apache/cxf/systest/jaxws/types/BarImpl.java
@@ -36,7 +36,6 @@ public class BarImpl implements Bar {
 
     /** {@inheritDoc}*/
     public String getName() {
-        // TODO Auto-generated method stub
         return name;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/jaxws/src/test/java/org/apache/cxf/systest/provider/HWDOMSourcePayloadProvider.java
----------------------------------------------------------------------
diff --git a/systests/jaxws/src/test/java/org/apache/cxf/systest/provider/HWDOMSourcePayloadProvider.java b/systests/jaxws/src/test/java/org/apache/cxf/systest/provider/HWDOMSourcePayloadProvider.java
index fef9de8..f2e0fcf 100644
--- a/systests/jaxws/src/test/java/org/apache/cxf/systest/provider/HWDOMSourcePayloadProvider.java
+++ b/systests/jaxws/src/test/java/org/apache/cxf/systest/provider/HWDOMSourcePayloadProvider.java
@@ -110,7 +110,6 @@ public class HWDOMSourcePayloadProvider implements Provider<DOMSource> {
 
                     throw new SOAPFaultException(soapFault);
                 } catch (SOAPException e) {
-                    // TODO Auto-generated catch block
                     e.printStackTrace();
                 }
             }

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/transport-jms/src/test/java/org/apache/cxf/systest/jms/TwoWayJMSImplBase.java
----------------------------------------------------------------------
diff --git a/systests/transport-jms/src/test/java/org/apache/cxf/systest/jms/TwoWayJMSImplBase.java b/systests/transport-jms/src/test/java/org/apache/cxf/systest/jms/TwoWayJMSImplBase.java
index 953c789..c131346 100644
--- a/systests/transport-jms/src/test/java/org/apache/cxf/systest/jms/TwoWayJMSImplBase.java
+++ b/systests/transport-jms/src/test/java/org/apache/cxf/systest/jms/TwoWayJMSImplBase.java
@@ -85,32 +85,26 @@ public class TwoWayJMSImplBase implements HelloWorldPortType {
     }
 
     public Response<String> greetMeAsync(String stringParam0) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public Future<?> greetMeAsync(String stringParam0, AsyncHandler<String> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public Response<String> sayHiAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public Future<?> sayHiAsync(AsyncHandler<String> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public Response<TestRpcLitFaultResponse> testRpcLitFaultAsync(String in) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public Future<?> testRpcLitFaultAsync(String in, AsyncHandler<TestRpcLitFaultResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/5f277de6/systests/transport-jms/src/test/java/org/apache/cxf/systest/jms/security/TwoWayJMSImplBase.java
----------------------------------------------------------------------
diff --git a/systests/transport-jms/src/test/java/org/apache/cxf/systest/jms/security/TwoWayJMSImplBase.java b/systests/transport-jms/src/test/java/org/apache/cxf/systest/jms/security/TwoWayJMSImplBase.java
index d143a4e..c35d3a4 100644
--- a/systests/transport-jms/src/test/java/org/apache/cxf/systest/jms/security/TwoWayJMSImplBase.java
+++ b/systests/transport-jms/src/test/java/org/apache/cxf/systest/jms/security/TwoWayJMSImplBase.java
@@ -91,32 +91,26 @@ public class TwoWayJMSImplBase implements HelloWorldPortType {
     }
 
     public Response<String> greetMeAsync(String stringParam0) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public Future<?> greetMeAsync(String stringParam0, AsyncHandler<String> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public Response<String> sayHiAsync() {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public Future<?> sayHiAsync(AsyncHandler<String> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public Response<TestRpcLitFaultResponse> testRpcLitFaultAsync(String in) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     public Future<?> testRpcLitFaultAsync(String in, AsyncHandler<TestRpcLitFaultResponse> asyncHandler) {
-        // TODO Auto-generated method stub
         return null;
     }
 

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[4/4] cxf git commit: Start going through the "TODO Auto-generated catch block"'s to figure out what should be done

Daniel  Kulp
Administrator
In reply to this post by Daniel Kulp
Start going through the "TODO Auto-generated catch block"'s to figure out what should be done


Project: http://git-wip-us.apache.org/repos/asf/cxf/repo
Commit: http://git-wip-us.apache.org/repos/asf/cxf/commit/b69bd427
Tree: http://git-wip-us.apache.org/repos/asf/cxf/tree/b69bd427
Diff: http://git-wip-us.apache.org/repos/asf/cxf/diff/b69bd427

Branch: refs/heads/master
Commit: b69bd427318a9f6db6e6e7940796cb233aa54ef9
Parents: 5f277de
Author: Daniel Kulp <[hidden email]>
Authored: Thu Aug 10 14:07:48 2017 -0400
Committer: Daniel Kulp <[hidden email]>
Committed: Thu Aug 10 14:07:48 2017 -0400

----------------------------------------------------------------------
 .../apache/cxf/jaxws/handler/AnnotationHandlerChainBuilder.java  | 3 +--
 .../src/main/java/org/apache/cxf/jaxws/spi/ProviderImpl.java     | 4 +---
 .../cxf/javascript/fortest/MtoMParameterBeanNoDataHandler.java   | 1 -
 .../cxf/transport/http/asyncclient/AsyncHTTPConduitTest.java     | 4 ----
 .../src/main/java/org/apache/cxf/wsdl11/CatalogWSDLLocator.java  | 2 +-
 .../cxf/systest/jaxws/DocLitWrappedCodeFirstServiceImpl.java     | 1 -
 .../apache/cxf/systest/ws/ut/SecurityHeaderCacheInterceptor.java | 1 -
 7 files changed, 3 insertions(+), 13 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cxf/blob/b69bd427/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/handler/AnnotationHandlerChainBuilder.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/handler/AnnotationHandlerChainBuilder.java b/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/handler/AnnotationHandlerChainBuilder.java
index df4f605..7d32846 100644
--- a/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/handler/AnnotationHandlerChainBuilder.java
+++ b/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/handler/AnnotationHandlerChainBuilder.java
@@ -284,8 +284,7 @@ public class AnnotationHandlerChainBuilder extends HandlerChainBuilder {
             PortComponentHandlerType pt = JAXBUtils.unmarshall(ctx, el, PortComponentHandlerType.class).getValue();
             chain.addAll(buildHandlerChain(pt, classLoader));
         } catch (JAXBException e) {
-            // TODO Auto-generated catch block
-            e.printStackTrace();
+            throw new IllegalArgumentException("Could not unmarshal handler chain", e);
         }
     }
 

http://git-wip-us.apache.org/repos/asf/cxf/blob/b69bd427/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/spi/ProviderImpl.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/spi/ProviderImpl.java b/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/spi/ProviderImpl.java
index 3b91b72..e9691cf 100644
--- a/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/spi/ProviderImpl.java
+++ b/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/spi/ProviderImpl.java
@@ -203,12 +203,10 @@ public class ProviderImpl extends javax.xml.ws.spi.Provider {
                 um = context.createUnmarshaller();
                 return um.unmarshal(reader, EndpointReferenceType.class).getValue();
             } catch (JAXBException e) {
-                // TODO Auto-generated catch block
-                e.printStackTrace();
+                throw new IllegalArgumentException("Could not unmarshal EndpointReference", e);
             } finally {
                 JAXBUtils.closeUnmarshaller(um);
             }
-            return null;
         }
         return null;
     }

http://git-wip-us.apache.org/repos/asf/cxf/blob/b69bd427/rt/javascript/javascript-tests/src/test/java/org/apache/cxf/javascript/fortest/MtoMParameterBeanNoDataHandler.java
----------------------------------------------------------------------
diff --git a/rt/javascript/javascript-tests/src/test/java/org/apache/cxf/javascript/fortest/MtoMParameterBeanNoDataHandler.java b/rt/javascript/javascript-tests/src/test/java/org/apache/cxf/javascript/fortest/MtoMParameterBeanNoDataHandler.java
index 0f0da70..5787c5f 100644
--- a/rt/javascript/javascript-tests/src/test/java/org/apache/cxf/javascript/fortest/MtoMParameterBeanNoDataHandler.java
+++ b/rt/javascript/javascript-tests/src/test/java/org/apache/cxf/javascript/fortest/MtoMParameterBeanNoDataHandler.java
@@ -47,7 +47,6 @@ public class MtoMParameterBeanNoDataHandler {
         try {
             this.notXml10 = new String(notXml10, "utf-8");
         } catch (UnsupportedEncodingException e) {
-            // TODO Auto-generated catch block
             e.printStackTrace();
         }
     }

http://git-wip-us.apache.org/repos/asf/cxf/blob/b69bd427/rt/transports/http-hc/src/test/java/org/apache/cxf/transport/http/asyncclient/AsyncHTTPConduitTest.java
----------------------------------------------------------------------
diff --git a/rt/transports/http-hc/src/test/java/org/apache/cxf/transport/http/asyncclient/AsyncHTTPConduitTest.java b/rt/transports/http-hc/src/test/java/org/apache/cxf/transport/http/asyncclient/AsyncHTTPConduitTest.java
index 36cfc45..f86eff9 100644
--- a/rt/transports/http-hc/src/test/java/org/apache/cxf/transport/http/asyncclient/AsyncHTTPConduitTest.java
+++ b/rt/transports/http-hc/src/test/java/org/apache/cxf/transport/http/asyncclient/AsyncHTTPConduitTest.java
@@ -197,10 +197,8 @@ public class AsyncHTTPConduitTest extends AbstractBusClientServerTestBase {
                 try {
                     res.get().getResponseType();
                 } catch (InterruptedException e) {
-                    // TODO Auto-generated catch block
                     e.printStackTrace();
                 } catch (ExecutionException e) {
-                    // TODO Auto-generated catch block
                     e.printStackTrace();
                 }
             }
@@ -292,10 +290,8 @@ public class AsyncHTTPConduitTest extends AbstractBusClientServerTestBase {
                             System.out.println("Problem " + c + " != " + s);
                         }
                     } catch (InterruptedException e) {
-                        // TODO Auto-generated catch block
                         e.printStackTrace();
                     } catch (ExecutionException e) {
-                        // TODO Auto-generated catch block
                         e.printStackTrace();
                     }
                     wdone[c] = true;

http://git-wip-us.apache.org/repos/asf/cxf/blob/b69bd427/rt/wsdl/src/main/java/org/apache/cxf/wsdl11/CatalogWSDLLocator.java
----------------------------------------------------------------------
diff --git a/rt/wsdl/src/main/java/org/apache/cxf/wsdl11/CatalogWSDLLocator.java b/rt/wsdl/src/main/java/org/apache/cxf/wsdl11/CatalogWSDLLocator.java
index 1801b4d..2d8d6e7 100644
--- a/rt/wsdl/src/main/java/org/apache/cxf/wsdl11/CatalogWSDLLocator.java
+++ b/rt/wsdl/src/main/java/org/apache/cxf/wsdl11/CatalogWSDLLocator.java
@@ -116,7 +116,7 @@ public class CatalogWSDLLocator implements WSDLLocator {
             in = this.resolver.resolve(resolvedImportLocation, null);
         }
 
-        // XXX: If we return null (as per javadoc), a NPE is raised in WSDL4J code.
+        // If we return null (as per javadoc), a NPE is raised in WSDL4J code.
         // So let's return new InputSource() and let WSDL4J fail. Optionally,
         // we can throw a similar exception as in CatalogXmlSchemaURIResolver.
         if (in == null) {

http://git-wip-us.apache.org/repos/asf/cxf/blob/b69bd427/systests/jaxws/src/test/java/org/apache/cxf/systest/jaxws/DocLitWrappedCodeFirstServiceImpl.java
----------------------------------------------------------------------
diff --git a/systests/jaxws/src/test/java/org/apache/cxf/systest/jaxws/DocLitWrappedCodeFirstServiceImpl.java b/systests/jaxws/src/test/java/org/apache/cxf/systest/jaxws/DocLitWrappedCodeFirstServiceImpl.java
index 0187319..c9495ba 100644
--- a/systests/jaxws/src/test/java/org/apache/cxf/systest/jaxws/DocLitWrappedCodeFirstServiceImpl.java
+++ b/systests/jaxws/src/test/java/org/apache/cxf/systest/jaxws/DocLitWrappedCodeFirstServiceImpl.java
@@ -218,7 +218,6 @@ public class DocLitWrappedCodeFirstServiceImpl implements DocLitWrappedCodeFirst
         try {
             Thread.sleep(2000);
         } catch (InterruptedException e) {
-            // TODO Auto-generated catch block
             e.printStackTrace();
         }
     }

http://git-wip-us.apache.org/repos/asf/cxf/blob/b69bd427/systests/ws-security/src/test/java/org/apache/cxf/systest/ws/ut/SecurityHeaderCacheInterceptor.java
----------------------------------------------------------------------
diff --git a/systests/ws-security/src/test/java/org/apache/cxf/systest/ws/ut/SecurityHeaderCacheInterceptor.java b/systests/ws-security/src/test/java/org/apache/cxf/systest/ws/ut/SecurityHeaderCacheInterceptor.java
index 053259d..a41d9cf 100644
--- a/systests/ws-security/src/test/java/org/apache/cxf/systest/ws/ut/SecurityHeaderCacheInterceptor.java
+++ b/systests/ws-security/src/test/java/org/apache/cxf/systest/ws/ut/SecurityHeaderCacheInterceptor.java
@@ -81,7 +81,6 @@ public class SecurityHeaderCacheInterceptor implements PhaseInterceptor<SoapMess
                 }
 
             } catch (SOAPException e) {
-                // TODO Auto-generated catch block
                 e.printStackTrace();
             }
         }

Loading...