You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2018/05/06 16:23:59 UTC

commons-dbutils git commit: Don't need to nest in an else clause.

Repository: commons-dbutils
Updated Branches:
  refs/heads/master 4f1c5824a -> 9d336010c


Don't need to nest in an else clause.

Project: http://git-wip-us.apache.org/repos/asf/commons-dbutils/repo
Commit: http://git-wip-us.apache.org/repos/asf/commons-dbutils/commit/9d336010
Tree: http://git-wip-us.apache.org/repos/asf/commons-dbutils/tree/9d336010
Diff: http://git-wip-us.apache.org/repos/asf/commons-dbutils/diff/9d336010

Branch: refs/heads/master
Commit: 9d336010cdd41aa47b3941c81f99aabe13ef0d20
Parents: 4f1c582
Author: Gary Gregory <ga...@gmail.com>
Authored: Sun May 6 10:23:57 2018 -0600
Committer: Gary Gregory <ga...@gmail.com>
Committed: Sun May 6 10:23:57 2018 -0600

----------------------------------------------------------------------
 src/test/java/org/apache/commons/dbutils/MockResultSet.java | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/commons-dbutils/blob/9d336010/src/test/java/org/apache/commons/dbutils/MockResultSet.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/dbutils/MockResultSet.java b/src/test/java/org/apache/commons/dbutils/MockResultSet.java
index 8815bd8..9831fde 100644
--- a/src/test/java/org/apache/commons/dbutils/MockResultSet.java
+++ b/src/test/java/org/apache/commons/dbutils/MockResultSet.java
@@ -340,10 +340,9 @@ public class MockResultSet implements InvocationHandler {
     protected Boolean next() throws SQLException {
         if (!this.iter.hasNext()) {
             return Boolean.FALSE;
-        } else {
-            this.currentRow = iter.next();
-            return Boolean.TRUE;
         }
+        this.currentRow = iter.next();
+        return Boolean.TRUE;
     }
 
     /**