You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by ba...@apache.org on 2011/04/10 07:02:48 UTC

svn commit: r1090741 - /commons/proper/dbutils/trunk/src/java/org/apache/commons/dbutils/BeanProcessor.java

Author: bayard
Date: Sun Apr 10 05:02:48 2011
New Revision: 1090741

URL: http://svn.apache.org/viewvc?rev=1090741&view=rev
Log:
Applying Brandon Atkinson's patch from DBUTILS-65; an if statement had been duplicated. No change in functionality, just inefficient. 

Modified:
    commons/proper/dbutils/trunk/src/java/org/apache/commons/dbutils/BeanProcessor.java

Modified: commons/proper/dbutils/trunk/src/java/org/apache/commons/dbutils/BeanProcessor.java
URL: http://svn.apache.org/viewvc/commons/proper/dbutils/trunk/src/java/org/apache/commons/dbutils/BeanProcessor.java?rev=1090741&r1=1090740&r2=1090741&view=diff
==============================================================================
--- commons/proper/dbutils/trunk/src/java/org/apache/commons/dbutils/BeanProcessor.java (original)
+++ commons/proper/dbutils/trunk/src/java/org/apache/commons/dbutils/BeanProcessor.java Sun Apr 10 05:02:48 2011
@@ -439,10 +439,6 @@ public class BeanProcessor {
         if ( !propType.isPrimitive() && rs.getObject(index) == null ) {
             return null;
         }
-        
-        if ( !propType.isPrimitive() && rs.getObject(index) == null ) {
-            return null;
-        }
 
         if (propType.equals(String.class)) {
             return rs.getString(index);