Fix some warnings and mark some methods as static

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

@@ -1939,7 +1939,7 @@ public class MVMap<K, V> extends AbstractMap<K, V>
         if (notificationRequested) {
             synchronized (lock) {
                 notificationRequested = false;
-                lock.notifyAll();;
+                lock.notifyAll();
             }
         }
         return updatedRootReference;

h2/src/main/org/h2/mvstore/tx/VersionedValueCommitted.java

@@ -27,10 +27,12 @@ class VersionedValueCommitted extends VersionedValue {
         return value instanceof VersionedValue ? (VersionedValue) value : new VersionedValueCommitted(value);
     }
 
+    @Override
     public Object getCurrentValue() {
         return value;
     }
 
+    @Override
     public Object getCommittedValue() {
         return value;
     }

h2/src/test/org/h2/test/auth/TestAuthentication.java

@@ -77,11 +77,11 @@ public class TestAuthentication extends TestBase {
         return externalUserPassword;
     }
 
-    private String getRealmName() {
+    private static String getRealmName() {
         return "testRealm";
     }
 
-    private String getStaticRoleName() {
+    private static String getStaticRoleName() {
         return "staticRole";
     }
 
@@ -117,7 +117,7 @@ public class TestAuthentication extends TestBase {
         return "jdbc:h2:mem:" + getClass().getSimpleName();
     }
 
-    private String getExternalUser() {
+    private static String getExternalUser() {
         return "user";
     }
 

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

@@ -92,12 +92,12 @@ public class TestLIRSMemoryConsumption extends TestDb {
         }
     }
 
-    private Object createValue(long key) {
+    private static Object createValue(long key) {
 //        return new Object();
         return new byte[2540];
     }
 
-    private int getValueSize(long key) {
+    private static int getValueSize(long key) {
 //        return 16;
         return 2560;
     }