You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by si...@apache.org on 2012/09/21 16:19:38 UTC

svn commit: r1388518 - /commons/proper/dbutils/trunk/src/main/java/org/apache/commons/dbutils/handlers/ScalarHandler.java

Author: simonetripodi
Date: Fri Sep 21 14:19:38 2012
New Revision: 1388518

URL: http://svn.apache.org/viewvc?rev=1388518&view=rev
Log:
removed useless 'else' branches

Modified:
    commons/proper/dbutils/trunk/src/main/java/org/apache/commons/dbutils/handlers/ScalarHandler.java

Modified: commons/proper/dbutils/trunk/src/main/java/org/apache/commons/dbutils/handlers/ScalarHandler.java
URL: http://svn.apache.org/viewvc/commons/proper/dbutils/trunk/src/main/java/org/apache/commons/dbutils/handlers/ScalarHandler.java?rev=1388518&r1=1388517&r2=1388518&view=diff
==============================================================================
--- commons/proper/dbutils/trunk/src/main/java/org/apache/commons/dbutils/handlers/ScalarHandler.java (original)
+++ commons/proper/dbutils/trunk/src/main/java/org/apache/commons/dbutils/handlers/ScalarHandler.java Fri Sep 21 14:19:38 2012
@@ -102,12 +102,9 @@ public class ScalarHandler<T> implements
         if (rs.next()) {
             if (this.columnName == null) {
                 return (T) rs.getObject(this.columnIndex);
-            } else {
-                return (T) rs.getObject(this.columnName);
             }
-
-        } else {
-            return null;
+            return (T) rs.getObject(this.columnName);
         }
+        return null;
     }
 }