Don't initialize an instance or static variable to its default value
diff --git a/src/conf/checkstyle/checkstyle.xml b/src/conf/checkstyle/checkstyle.xml
index 41c5c77..ab3bcc2 100644
--- a/src/conf/checkstyle/checkstyle.xml
+++ b/src/conf/checkstyle/checkstyle.xml
@@ -59,6 +59,7 @@
<module name="TreeWalker">
+ <module name="ExplicitInitializationCheck" />
<!-- ************************************************************** -->
<!-- Checks that are different from the sun coding conventions ones -->
<!-- ************************************************************** -->
diff --git a/src/main/java/org/apache/commons/dbutils/AbstractQueryRunner.java b/src/main/java/org/apache/commons/dbutils/AbstractQueryRunner.java
index c00fbe8..b1bb78f 100644
--- a/src/main/java/org/apache/commons/dbutils/AbstractQueryRunner.java
+++ b/src/main/java/org/apache/commons/dbutils/AbstractQueryRunner.java
@@ -44,7 +44,7 @@
* Is {@link ParameterMetaData#getParameterType(int)} broken (have we tried
* it yet)?
*/
- private volatile boolean pmdKnownBroken = false;
+ private volatile boolean pmdKnownBroken;
/**
* The DataSource to retrieve connections from.
diff --git a/src/main/java/org/apache/commons/dbutils/OutParameter.java b/src/main/java/org/apache/commons/dbutils/OutParameter.java
index 0489452..5f8a663 100644
--- a/src/main/java/org/apache/commons/dbutils/OutParameter.java
+++ b/src/main/java/org/apache/commons/dbutils/OutParameter.java
@@ -36,7 +36,7 @@
public class OutParameter<T> {
private final int sqlType;
private final Class<T> javaType;
- private T value = null;
+ private T value;
/**
* Construct an {@code OutParameter} for the given JDBC SQL type and
diff --git a/src/test/java/org/apache/commons/dbutils/BaseTestCase.java b/src/test/java/org/apache/commons/dbutils/BaseTestCase.java
index 63a7adb..bb7f7f0 100644
--- a/src/test/java/org/apache/commons/dbutils/BaseTestCase.java
+++ b/src/test/java/org/apache/commons/dbutils/BaseTestCase.java
@@ -103,12 +103,12 @@
/**
* The ResultSet all test methods will use.
*/
- protected ResultSet rs = null;
+ protected ResultSet rs;
/**
* A ResultSet with 0 rows.
*/
- protected ResultSet emptyResultSet = null;
+ protected ResultSet emptyResultSet;
/**
* Creates a freshly initialized ResultSet.
diff --git a/src/test/java/org/apache/commons/dbutils/MockResultSet.java b/src/test/java/org/apache/commons/dbutils/MockResultSet.java
index 55256ae..46be542 100644
--- a/src/test/java/org/apache/commons/dbutils/MockResultSet.java
+++ b/src/test/java/org/apache/commons/dbutils/MockResultSet.java
@@ -48,9 +48,9 @@
private Object[] currentRow = null;
- private Iterator<Object[]> iter = null;
+ private Iterator<Object[]> iter;
- private ResultSetMetaData metaData = null;
+ private ResultSetMetaData metaData;
private Boolean wasNull = Boolean.FALSE;
diff --git a/src/test/java/org/apache/commons/dbutils/TestBean.java b/src/test/java/org/apache/commons/dbutils/TestBean.java
index b7200aa..83fe007 100644
--- a/src/test/java/org/apache/commons/dbutils/TestBean.java
+++ b/src/test/java/org/apache/commons/dbutils/TestBean.java
@@ -27,17 +27,17 @@
}
- private String one = null;
+ private String one;
- private String two = null;
+ private String two;
- private Ordinal three = null;
+ private Ordinal three;
- private int intTest = 0;
+ private int intTest;
private Integer integerTest = Integer.valueOf(0);
- // UNUSED private Timestamp timestamp = null;
+ // UNUSED private Timestamp timestamp;
private String doNotSet = "not set";
diff --git a/src/test/java/org/apache/commons/dbutils/wrappers/SqlNullCheckedResultSetTest.java b/src/test/java/org/apache/commons/dbutils/wrappers/SqlNullCheckedResultSetTest.java
index 6a36d03..ddba460 100644
--- a/src/test/java/org/apache/commons/dbutils/wrappers/SqlNullCheckedResultSetTest.java
+++ b/src/test/java/org/apache/commons/dbutils/wrappers/SqlNullCheckedResultSetTest.java
@@ -230,7 +230,7 @@
}
}
- private SqlNullCheckedResultSet rs2 = null;
+ private SqlNullCheckedResultSet rs2;
/**
* Sets up instance variables required by this test case.
