Merge pull request #908 from grandinj/master

remove dead code

h2/src/main/org/h2/engine/Session.java

@@ -124,7 +124,7 @@ public class Session extends SessionWithState {
     private long modificationMetaID = -1;
     private SubQueryInfo subQueryInfo;
     private int parsingView;
-    private Deque<String> viewNameStack = new ArrayDeque<>();
+    private final Deque<String> viewNameStack = new ArrayDeque<>();
     private int preparingQueryExpression;
     private volatile SmallLRUCache<Object, ViewIndex> viewIndexCache;
     private HashMap<Object, ViewIndex> subQueryIndexCache;

h2/src/main/org/h2/engine/SysProperties.java

@@ -56,13 +56,6 @@ public class SysProperties {
     public static final String FILE_SEPARATOR =
             Utils.getProperty("file.separator", "/");
 
-    /**
-     * System property <code>java.specification.version</code>.<br />
-     * It is set by the system. Examples: 0.9 (on Android), 1.7, 1.8, 9, 10.
-     */
-    public static final String JAVA_SPECIFICATION_VERSION =
-            Utils.getProperty("java.specification.version", "1.7");
-
     /**
      * System property <code>line.separator</code> (default: \n).<br />
      * It is usually set by the system, and used by the script and trace tools.

h2/src/main/org/h2/jdbc/JdbcResultSet.java

@@ -29,7 +29,6 @@ import java.util.Calendar;
 import java.util.HashMap;
 import java.util.Map;
 import java.util.UUID;
-
 import org.h2.api.ErrorCode;
 import org.h2.api.TimestampWithTimeZone;
 import org.h2.command.CommandInterface;
@@ -91,7 +90,7 @@ public class JdbcResultSet extends TraceObject implements ResultSet, JdbcResultS
     private HashMap<String, Integer> columnLabelMap;
     private HashMap<Integer, Value[]> patchedRows;
     private JdbcPreparedStatement preparedStatement;
-    private CommandInterface command;
+    private final CommandInterface command;
 
     JdbcResultSet(JdbcConnection conn, JdbcStatement stat, CommandInterface command,
             ResultInterface result, int id, boolean closeStatement,
@@ -101,7 +100,7 @@ public class JdbcResultSet extends TraceObject implements ResultSet, JdbcResultS
         this.stat = stat;
         this.command = command;
         this.result = result;
-        columnCount = result.getVisibleColumnCount();
+        this.columnCount = result.getVisibleColumnCount();
         this.closeStatement = closeStatement;
         this.scrollable = scrollable;
         this.updatable = updatable;

h2/src/main/org/h2/mvstore/MVStore.java

@@ -237,7 +237,7 @@ public final class MVStore {
      * are counted.
      */
     private int unsavedMemory;
-    private int autoCommitMemory;
+    private final int autoCommitMemory;
     private boolean saveNeeded;
 
     /**
@@ -274,7 +274,7 @@ public final class MVStore {
      */
     private int autoCommitDelay;
 
-    private int autoCompactFillRate;
+    private final int autoCompactFillRate;
     private long autoCompactLastFileOpCount;
 
     private final Object compactSync = new Object();
@@ -376,6 +376,9 @@ public final class MVStore {
             // the parameter is different from the old value
             int delay = DataUtils.getConfigParam(config, "autoCommitDelay", 1000);
             setAutoCommitDelay(delay);
+        } else {
+            autoCommitMemory = 0;
+            autoCompactFillRate = 0;
         }
     }
 

h2/src/main/org/h2/mvstore/db/MVTable.java

@@ -118,7 +118,7 @@ public class MVTable extends TableBase {
     private final Trace traceLock;
     private int changesSinceAnalyze;
     private int nextAnalyze;
-    private boolean containsLargeObject;
+    private final boolean containsLargeObject;
     private Column rowIdColumn;
 
     private final MVTableEngine.Store store;
@@ -130,11 +130,14 @@ public class MVTable extends TableBase {
         this.store = store;
         this.transactionStore = store.getTransactionStore();
         this.isHidden = data.isHidden;
+        boolean b = false;
         for (Column col : getColumns()) {
             if (DataType.isLargeObject(col.getType())) {
-                containsLargeObject = true;
+                b = true;
+                break;
             }
         }
+        containsLargeObject = b;
         traceLock = database.getTrace(Trace.LOCK);
     }
 

h2/src/main/org/h2/mvstore/db/TransactionStore.java

@@ -468,7 +468,7 @@ public class TransactionStore {
      * End this transaction
      *
      * @param t the transaction
-     * @param previous status of this transaction
+     * @param oldStatus status of this transaction
      */
     synchronized void endTransaction(Transaction t, int oldStatus) {
         if (oldStatus == Transaction.STATUS_PREPARED) {

h2/src/main/org/h2/result/ResultTempTable.java

@@ -31,16 +31,16 @@ public class ResultTempTable implements ResultExternal {
     private final boolean distinct;
     private final SortOrder sort;
     private Index index;
-    private Session session;
+    private final Session session;
     private Table table;
     private Cursor resultCursor;
     private int rowCount;
-    private int columnCount;
+    private final int columnCount;
 
     private final ResultTempTable parent;
     private boolean closed;
     private int childCount;
-    private boolean containsLob;
+    private final boolean containsLob;
 
     ResultTempTable(Session session, Expression[] expressions, boolean distinct, SortOrder sort) {
         this.session = session;
@@ -49,15 +49,17 @@ public class ResultTempTable implements ResultExternal {
         this.columnCount = expressions.length;
         Schema schema = session.getDatabase().getSchema(Constants.SCHEMA_MAIN);
         CreateTableData data = new CreateTableData();
+        boolean b = false;
         for (int i = 0; i < expressions.length; i++) {
             int type = expressions[i].getType();
             Column col = new Column(COLUMN_NAME + i,
                     type);
             if (type == Value.CLOB || type == Value.BLOB) {
-                containsLob = true;
+                b = true;
             }
             data.columns.add(col);
         }
+        containsLob = b;
         data.id = session.getDatabase().allocateObjectId();
         data.tableName = "TEMP_RESULT_SET_" + data.id;
         data.temporary = true;

h2/src/main/org/h2/result/UpdatableRow.java

@@ -10,7 +10,6 @@ import java.sql.PreparedStatement;
 import java.sql.ResultSet;
 import java.sql.SQLException;
 import java.util.ArrayList;
-
 import org.h2.api.ErrorCode;
 import org.h2.jdbc.JdbcConnection;
 import org.h2.message.DbException;

h2/src/main/org/h2/table/RegularTable.java

@@ -64,7 +64,7 @@ public class RegularTable extends TableBase {
     private final Trace traceLock;
     private final ArrayList<Index> indexes = New.arrayList();
     private long lastModificationId;
-    private boolean containsLargeObject;
+    private final boolean containsLargeObject;
     private final PageDataIndex mainIndex;
     private int changesSinceAnalyze;
     private int nextAnalyze;
@@ -74,11 +74,14 @@ public class RegularTable extends TableBase {
         super(data);
         nextAnalyze = database.getSettings().analyzeAuto;
         this.isHidden = data.isHidden;
+        boolean b = false;
         for (Column col : getColumns()) {
             if (DataType.isLargeObject(col.getType())) {
-                containsLargeObject = true;
+                b = true;
+                break;
             }
         }
+        containsLargeObject = b;
         if (data.persistData && database.isPersistent()) {
             mainIndex = new PageDataIndex(this, data.id,
                     IndexColumn.wrap(getColumns()),

h2/src/main/org/h2/table/TableBase.java

@@ -5,7 +5,7 @@
  */
 package org.h2.table;
 
-import java.util.ArrayList;
+import java.util.Collections;
 import java.util.List;
 import org.h2.command.ddl.CreateTableData;
 import org.h2.engine.Database;
@@ -27,7 +27,7 @@ public abstract class TableBase extends Table {
      */
     private final String tableEngine;
     /** Provided table parameters */
-    private List<String> tableEngineParams = new ArrayList<>();
+    private final List<String> tableEngineParams;
 
     private final boolean globalTemporary;
 
@@ -38,6 +38,8 @@ public abstract class TableBase extends Table {
         this.globalTemporary = data.globalTemporary;
         if (data.tableEngineParams != null) {
             this.tableEngineParams = data.tableEngineParams;
+        } else {
+            this.tableEngineParams = Collections.EMPTY_LIST;
         }
         setTemporary(data.temporary);
         Column[] cols = data.columns.toArray(new Column[0]);

h2/src/main/org/h2/util/ToDateTokenizer.java

@@ -6,7 +6,6 @@
 package org.h2.util;
 
 import static java.lang.String.format;
-
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Collections;
@@ -113,11 +112,6 @@ class ToDateTokenizer {
      */
     static final InlineParslet PARSLET_INLINE = new InlineParslet();
 
-    /**
-     * The number of milliseconds in a day.
-     */
-    static final int MILLIS_IN_HOUR = 60 * 60 * 1000;
-
     /**
      * Interface of the classes that can parse a specialized small bit of the
      * TO_DATE format-string.

h2/src/test/org/h2/test/db/TestOptimizations.java

@@ -5,18 +5,22 @@
  */
 package org.h2.test.db;
 
-import org.h2.api.ErrorCode;
-import org.h2.test.TestBase;
-import org.h2.tools.SimpleResultSet;
-import org.h2.util.StringUtils;
-import org.h2.util.Task;
-
-import java.sql.*;
+import java.sql.Connection;
+import java.sql.PreparedStatement;
+import java.sql.ResultSet;
+import java.sql.SQLException;
+import java.sql.Statement;
+import java.sql.Types;
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.Random;
 import java.util.TreeSet;
 import java.util.concurrent.TimeUnit;
+import org.h2.api.ErrorCode;
+import org.h2.test.TestBase;
+import org.h2.tools.SimpleResultSet;
+import org.h2.util.StringUtils;
+import org.h2.util.Task;
 
 /**
  * Test various optimizations (query cache, optimization for MIN(..), and
@@ -818,7 +822,7 @@ public class TestOptimizations extends TestBase {
         }
     }
 
-    private long measureQuerySpeed(Statement stat, String sql, boolean optimized) throws SQLException {
+    private static long measureQuerySpeed(Statement stat, String sql, boolean optimized) throws SQLException {
         stat.execute("set OPTIMIZE_REUSE_RESULTS " + (optimized ? "1" : "0"));
         stat.execute(sql);
         long time = System.nanoTime();

h2/src/test/org/h2/test/store/TestMVStoreBenchmark.java

@@ -8,12 +8,10 @@ package org.h2.test.store;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.HashMap;
-import java.util.LinkedList;
 import java.util.List;
 import java.util.Map;
 import java.util.TreeMap;
 import java.util.concurrent.TimeUnit;
-
 import org.h2.mvstore.MVStore;
 import org.h2.test.TestBase;
 import org.h2.util.New;

h2/src/tools/org/h2/jaqu/TableDefinition.java

@@ -19,8 +19,8 @@ import org.h2.jaqu.Table.JQColumn;
 import org.h2.jaqu.Table.JQIndex;
 import org.h2.jaqu.Table.JQSchema;
 import org.h2.jaqu.Table.JQTable;
-import org.h2.jaqu.util.StatementLogger;
 import org.h2.jaqu.util.ClassUtils;
+import org.h2.jaqu.util.StatementLogger;
 import org.h2.util.New;
 import org.h2.util.StatementBuilder;
 import org.h2.util.StringUtils;