From: funkman <funkman@13f79535-47bb-0310-9956-ffa450edef68>
Date: Wed, 4 Nov 2009 00:54:02 +0000 (+0000)
Subject: https://issues.apache.org/bugzilla/show_bug.cgi?id=46947
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=a9fa19349dd8b559294ec45f8be35f29d3a59078;p=tomcat7.0

https://issues.apache.org/bugzilla/show_bug.cgi?id=46947

Code cleanup patch that removes some obsolete null checks




git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@832636 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/java/org/apache/catalina/ant/jmx/JMXAccessorTask.java b/java/org/apache/catalina/ant/jmx/JMXAccessorTask.java
index 6dc17ee7e..1fe6da2a2 100644
--- a/java/org/apache/catalina/ant/jmx/JMXAccessorTask.java
+++ b/java/org/apache/catalina/ant/jmx/JMXAccessorTask.java
@@ -430,10 +430,8 @@ public class JMXAccessorTask extends BaseRedirectorHelperTask {
             try {
                 jmxServerConnection = (MBeanServerConnection) pref;
             } catch (ClassCastException cce) {
-                if (project != null) {
-                    project.log("wrong object reference " + refId + " - "
+                project.log("wrong object reference " + refId + " - "
                             + pref.getClass());
-                }
                 return null;
             }
         }
diff --git a/java/org/apache/catalina/tribes/transport/bio/BioReceiver.java b/java/org/apache/catalina/tribes/transport/bio/BioReceiver.java
index 91e73a9d6..bfed62cdc 100644
--- a/java/org/apache/catalina/tribes/transport/bio/BioReceiver.java
+++ b/java/org/apache/catalina/tribes/transport/bio/BioReceiver.java
@@ -30,7 +30,7 @@ import org.apache.catalina.tribes.transport.AbstractRxTask;
 /**
  *
  * @author Filip Hanik
- * @version $Revision:$ $Date:$
+ * @version $Revision$ $Date$
  */
 public class BioReceiver extends ReceiverBase implements Runnable, ChannelReceiver, ListenCallback {
 
diff --git a/java/org/apache/catalina/valves/JDBCAccessLogValve.java b/java/org/apache/catalina/valves/JDBCAccessLogValve.java
index 87f080ae9..ae501646f 100644
--- a/java/org/apache/catalina/valves/JDBCAccessLogValve.java
+++ b/java/org/apache/catalina/valves/JDBCAccessLogValve.java
@@ -462,17 +462,13 @@ public final class JDBCAccessLogValve
         
         getNext().invoke(request, response);
 
-        String remoteHost = EMPTY;
+        String remoteHost;
         if(resolveHosts)
             remoteHost = request.getRemoteHost();
         else
             remoteHost = request.getRemoteAddr();
-        String user = EMPTY;
-        if(request != null)
-            user = request.getRemoteUser();
-        String query="";
-        if(request != null)
-            query = request.getRequestURI();
+        String user = request.getRemoteUser();
+        String query=request.getRequestURI();
         
         long bytes = response.getContentCountLong() ;
         if(bytes < 0)
@@ -482,7 +478,7 @@ public final class JDBCAccessLogValve
         String method = EMPTY;
         String referer = EMPTY;
         String userAgent = EMPTY;
-        if (pattern.equals("combined") && request != null) {
+        if (pattern.equals("combined")) {
             virtualHost = request.getServerName();
             method = request.getMethod();
             referer = request.getHeader("referer");
diff --git a/java/org/apache/tomcat/util/net/NioBlockingSelector.java b/java/org/apache/tomcat/util/net/NioBlockingSelector.java
index 86d83b7f5..73d634efc 100644
--- a/java/org/apache/tomcat/util/net/NioBlockingSelector.java
+++ b/java/org/apache/tomcat/util/net/NioBlockingSelector.java
@@ -350,7 +350,7 @@ public class NioBlockingSelector {
                                 countDown(attachment.getWriteLatch());
                             }
                         }catch (CancelledKeyException ckx) {
-                            if (sk!=null) sk.cancel();
+                            sk.cancel();
                             countDown(attachment.getReadLatch());
                             countDown(attachment.getWriteLatch());
                         }
@@ -391,4 +391,4 @@ public class NioBlockingSelector {
         }
     }
 
-}
\ No newline at end of file
+}