Index: src/java/org/apache/commons/dbutils/BasicRowProcessor.java =================================================================== --- src/java/org/apache/commons/dbutils/BasicRowProcessor.java (revision 752322) +++ src/java/org/apache/commons/dbutils/BasicRowProcessor.java (working copy) @@ -60,7 +60,7 @@ /** * Use this to process beans. */ - private BeanProcessor convert = null; + private final BeanProcessor convert; /** * BasicRowProcessor constructor. Bean processing defaults to a Index: src/java/org/apache/commons/dbutils/QueryLoader.java =================================================================== --- src/java/org/apache/commons/dbutils/QueryLoader.java (revision 752322) +++ src/java/org/apache/commons/dbutils/QueryLoader.java (working copy) @@ -46,7 +46,7 @@ /** * Maps query set names to Maps of their queries. */ - private Map queries = new HashMap(); + private final Map queries = new HashMap(); /** * QueryLoader constructor. Index: src/java/org/apache/commons/dbutils/ResultSetIterator.java =================================================================== --- src/java/org/apache/commons/dbutils/ResultSetIterator.java (revision 752322) +++ src/java/org/apache/commons/dbutils/ResultSetIterator.java (working copy) @@ -37,19 +37,19 @@ /** * The wrapped ResultSet. */ - private ResultSet rs = null; + private final ResultSet rs; /** * The processor to use when converting a row into an Object[]. */ - private RowProcessor convert = new BasicRowProcessor(); + private final RowProcessor convert; /** * Constructor for ResultSetIterator. * @param rs Wrap this ResultSet in an Iterator. */ public ResultSetIterator(ResultSet rs) { - this.rs = rs; + this(rs , new BasicRowProcessor()); } /**