You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by dk...@apache.org on 2014/04/04 18:30:14 UTC

git commit: Fix a couple of eclipse warnings

Repository: cxf
Updated Branches:
  refs/heads/master 816ab4c9c -> 0213b0001


Fix a couple of eclipse warnings


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

Branch: refs/heads/master
Commit: 0213b00013706d9e7fa0418a035819ab8e5af5f1
Parents: 816ab4c
Author: Daniel Kulp <dk...@apache.org>
Authored: Fri Apr 4 12:29:59 2014 -0400
Committer: Daniel Kulp <dk...@apache.org>
Committed: Fri Apr 4 12:29:59 2014 -0400

----------------------------------------------------------------------
 .../apache/cxf/transport/jms/util/MessageListenerContainer.java   | 3 +--
 .../apache/cxf/systest/jaxrs/websocket/WebSocketTestClient.java   | 1 -
 2 files changed, 1 insertion(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cxf/blob/0213b000/rt/transports/jms/src/main/java/org/apache/cxf/transport/jms/util/MessageListenerContainer.java
----------------------------------------------------------------------
diff --git a/rt/transports/jms/src/main/java/org/apache/cxf/transport/jms/util/MessageListenerContainer.java b/rt/transports/jms/src/main/java/org/apache/cxf/transport/jms/util/MessageListenerContainer.java
index 6521289..828da94 100644
--- a/rt/transports/jms/src/main/java/org/apache/cxf/transport/jms/util/MessageListenerContainer.java
+++ b/rt/transports/jms/src/main/java/org/apache/cxf/transport/jms/util/MessageListenerContainer.java
@@ -212,7 +212,6 @@ public class MessageListenerContainer implements JMSListenerContainer {
         
     }
     
-    @SuppressWarnings("PMD")
     static class XATransactionalMessageListener implements MessageListener {
         private TransactionManager tm;
         private MessageListener listenerHandler;
@@ -222,7 +221,7 @@ public class MessageListenerContainer implements JMSListenerContainer {
             if (tm == null) {
                 throw new IllegalArgumentException("Must supply a transaction manager");
             }
-            if (session == null || !(session instanceof XASession)) {
+            if (!(session instanceof XASession)) {
                 throw new IllegalArgumentException("Must supply an XASession");
             }
             this.tm = tm;

http://git-wip-us.apache.org/repos/asf/cxf/blob/0213b000/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/websocket/WebSocketTestClient.java
----------------------------------------------------------------------
diff --git a/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/websocket/WebSocketTestClient.java b/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/websocket/WebSocketTestClient.java
index 3525e9f..67835b9 100644
--- a/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/websocket/WebSocketTestClient.java
+++ b/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/websocket/WebSocketTestClient.java
@@ -248,7 +248,6 @@ class WebSocketTestClient {
             return contentType;
         }
         
-        @SuppressWarnings("unused")
         public Object getEntity() {
             return entity;
         }