Don't initial instance variable to its default value

Author: garydgregory

checkstyle.xml

@@ -45,7 +45,7 @@
   <module name="FileTabCharacterCheck" />
 
   <module name="TreeWalker">
-
+    <module name="ExplicitInitialization" />
     <!--property name="cacheFile" value="${checkstyle.cache.file}"/ -->
 
     <!-- ************************************************************** -->

src/main/java/org/apache/commons/jxpath/JXPathBasicBeanInfo.java

@@ -37,7 +37,8 @@ public class JXPathBasicBeanInfo implements JXPathBeanInfo {
     private static final long serialVersionUID = -3863803443111484155L;
     private static final Comparator PROPERTY_DESCRIPTOR_COMPARATOR = (left, right) -> ((PropertyDescriptor) left).getName()
             .compareTo(((PropertyDescriptor) right).getName());
-    private boolean atomic = false;
+
+    private boolean atomic;
     private final Class clazz;
     private Class dynamicPropertyHandlerClass;
     private transient PropertyDescriptor[] propertyDescriptors;

src/main/java/org/apache/commons/jxpath/JXPathContext.java

@@ -428,8 +428,8 @@ public static JXPathContext newContext(final Object contextBean) {
     /** Decimal format map */
     protected HashMap<String, DecimalFormatSymbols> decimalFormats;
     private Locale locale;
-    private boolean lenientSet = false;
-    private boolean lenient = false;
+    private boolean lenientSet;
+    private boolean lenient;
 
     /**
      * This constructor should remain protected - it is to be overridden by subclasses, but never explicitly invoked by clients.

src/main/java/org/apache/commons/jxpath/JXPathContextFactory.java

@@ -49,7 +49,8 @@ public abstract class JXPathContextFactory {
     /**
      * Temp debug code - this will be removed after we test everything
      */
-    private static boolean debug = false;
+    private static boolean debug;
+
     static {
         try {
             debug = System.getProperty("jxpath.debug") != null;

src/main/java/org/apache/commons/jxpath/ri/EvalContext.java

@@ -47,10 +47,10 @@ public abstract class EvalContext implements ExpressionContext, Iterator {
     /** Root context */
     protected RootContext rootContext;
     /** Position */
-    protected int position = 0;
-    private boolean startedSetIteration = false;
-    private boolean done = false;
-    private boolean hasPerformedIteratorStep = false;
+    protected int position;
+    private boolean startedSetIteration;
+    private boolean done;
+    private boolean hasPerformedIteratorStep;
     private Iterator<Pointer> pointerIterator;
 
     /**

src/main/java/org/apache/commons/jxpath/ri/JXPathContextReferenceImpl.java

@@ -66,8 +66,8 @@ public class JXPathContextReferenceImpl extends JXPathContext {
     public static final boolean USE_SOFT_CACHE = true;
     private static final Compiler COMPILER = new TreeCompiler();
     private static Map compiled = new HashMap();
-    private static int cleanupCount = 0;
-    private static NodePointerFactory[] nodeFactoryArray = null;
+    private static int cleanupCount;
+    private static NodePointerFactory[] nodeFactoryArray;
     // The frequency of the cache cleanup
     private static final int CLEANUP_THRESHOLD = 500;
     private static final Vector<NodePointerFactory> nodeFactories = new Vector<>();

src/main/java/org/apache/commons/jxpath/ri/compiler/Expression.java

@@ -121,7 +121,7 @@ public void remove() {
     protected static final Double ONE = Double.valueOf(1);
     /** NaN */
     protected static final Double NOT_A_NUMBER = Double.valueOf(Double.NaN);
-    private boolean contextDependencyKnown = false;
+    private boolean contextDependencyKnown;
     private boolean contextDependent;
 
     /**

src/main/java/org/apache/commons/jxpath/ri/compiler/ExpressionPath.java

@@ -33,7 +33,7 @@ public class ExpressionPath extends Path {
 
     private final Expression expression;
     private final Expression[] predicates;
-    private boolean basicKnown = false;
+    private boolean basicKnown;
     private boolean basic;
 
     /**

src/main/java/org/apache/commons/jxpath/ri/compiler/Path.java

@@ -41,7 +41,7 @@
 public abstract class Path extends Expression {
 
     private final Step[] steps;
-    private boolean basicKnown = false;
+    private boolean basicKnown;
     private boolean basic;
 
     /**

src/main/java/org/apache/commons/jxpath/ri/model/NodePointer.java

@@ -138,7 +138,7 @@ public static NodePointer verify(final NodePointer nodePointer) {
 
     /** Index for this NodePointer */
     protected int index = WHOLE_COLLECTION;
-    private boolean attribute = false;
+    private boolean attribute;
     private NamespaceResolver namespaceResolver;
     private ExceptionHandler exceptionHandler;
     private transient Object rootNode;