X.26

assertions-disabled/pom.xml

@@ -25,7 +25,7 @@
     <parent>
         <groupId>net.openhft</groupId>
         <artifactId>java-parent-pom</artifactId>
-        <version>1.24.0</version>
+        <version>1.26.0-SNAPSHOT</version>
         <relativePath />
     </parent>
 

assertions-enabled/pom.xml

@@ -25,7 +25,7 @@
     <parent>
         <groupId>net.openhft</groupId>
         <artifactId>java-parent-pom</artifactId>
-        <version>1.24.0</version>
+        <version>1.26.0-SNAPSHOT</version>
         <relativePath />
     </parent>
 

benchmarks/pom.xml

@@ -22,7 +22,7 @@
     <parent>
         <groupId>net.openhft</groupId>
         <artifactId>java-parent-pom</artifactId>
-        <version>1.24.0</version>
+        <version>1.26.0-SNAPSHOT</version>
         <relativePath />
     </parent>
 
@@ -41,15 +41,15 @@
             <dependency>
                 <groupId>net.openhft</groupId>
                 <artifactId>third-party-bom</artifactId>
-                <version>3.24.0</version>
+                <version>3.26.0-SNAPSHOT</version>
                 <type>pom</type>
                 <scope>import</scope>
             </dependency>
 
             <dependency>
                 <groupId>net.openhft</groupId>
                 <artifactId>chronicle-bom</artifactId>
-                <version>2.25ea-SNAPSHOT</version>
+                <version>2.26ea-SNAPSHOT</version>
                 <type>pom</type>
                 <scope>import</scope>
             </dependency>

checked-exceptions/pom.xml

@@ -23,7 +23,7 @@
     <parent>
         <groupId>net.openhft</groupId>
         <artifactId>java-parent-pom</artifactId>
-        <version>1.24.0</version>
+        <version>1.26.0-SNAPSHOT</version>
         <relativePath/>
     </parent>
 

checked-exceptions/src/main/java/java/lang/Appendable.java

@@ -23,7 +23,7 @@ public interface Appendable {
     /**
      * Appends the specified character sequence to this <code>Appendable</code>.
      * <p>
-     * <p> Depending on which class implements the character sequence
+     *  Depending on which class implements the character sequence
      * <code>csq</code>, the entire sequence may not be appended.  For
      * instance, if <code>csq</code> is a {@link java.nio.CharBuffer} then
      * the subsequence to append is defined by the buffer's position and limit.
@@ -40,7 +40,7 @@ public interface Appendable {
      * Appends a subsequence of the specified character sequence to this
      * <code>Appendable</code>.
      * <p>
-     * <p> An invocation of this method of the form <code>out.append(csq, start,
+     *  An invocation of this method of the form <code>out.append(csq, start,
      * end)</code> when <code>csq</code> is not <code>null</code>, behaves in
      * exactly the same way as the invocation
      * <p>

checked-exceptions/src/main/java/java/lang/CharSequence.java

@@ -32,7 +32,7 @@ public interface CharSequence {
      * index zero, the next at index one, and so on, as for array
      * indexing.
      * <p>
-     * <p>If the <code>char</code> value specified by the index is a
+     * If the <code>char</code> value specified by the index is a
      * <a href="{@docRoot}/java/lang/Character.html#unicode">surrogate</a>, the surrogate
      * value is returned.
      *

pom.xml

@@ -21,11 +21,11 @@
     <parent>
         <groupId>net.openhft</groupId>
         <artifactId>java-parent-pom</artifactId>
-        <version>1.25.4</version>
-        <relativePath />
+        <version>1.26.0-SNAPSHOT</version>
+        <relativePath></relativePath>
     </parent>
     <artifactId>chronicle-core</artifactId>
-    <version>2.25ea15-SNAPSHOT</version>
+    <version>2.26ea0-SNAPSHOT</version>
     <packaging>bundle</packaging>
     <name>OpenHFT/Chronicle Core</name>
     <description>Chronicle-Core</description>
@@ -48,15 +48,15 @@
             <dependency>
                 <groupId>net.openhft</groupId>
                 <artifactId>third-party-bom</artifactId>
-                <version>3.25.0</version>
+                <version>3.26.0-SNAPSHOT</version>
                 <type>pom</type>
                 <scope>import</scope>
             </dependency>
 
             <dependency>
                 <groupId>net.openhft</groupId>
                 <artifactId>chronicle-bom</artifactId>
-                <version>2.25ea-SNAPSHOT</version>
+                <version>2.26ea-SNAPSHOT</version>
                 <type>pom</type>
                 <scope>import</scope>
             </dependency>
@@ -165,22 +165,6 @@
 
     <build>
         <plugins>
-            <!-- The parent POM defines this plugin and activates it in a profile. Unfortunately this plugin will generate
-            code at @NotNull call-sites that throws IllegalArgumentException rather than NullPointerException. This will
-            break various core tests. Chronicle have raised this issue before with the maintainers of the plugin. See:
-            https://github.com/osundblad/intellij-annotations-instrumenter-maven-plugin/issues/53
-            -->
-            <plugin>
-                <groupId>se.eris</groupId>
-                <artifactId>notnull-instrumenter-maven-plugin</artifactId>
-                <version>1.1.1</version>
-                <configuration>
-                    <notNull>
-                        <!-- Not ideal but this was the easiest way to effectively disable this plugin -->
-                        <param>Ignore</param>
-                    </notNull>
-                </configuration>
-            </plugin>
             <plugin>
                 <groupId>com.github.ekryd.sortpom</groupId>
                 <artifactId>sortpom-maven-plugin</artifactId>
@@ -234,7 +218,8 @@
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-compiler-plugin</artifactId>
                 <configuration>
-                    <compilerArgument>-Xlint:deprecation</compilerArgument>
+                    <compilerArgument>-Xlint:all,-options</compilerArgument>
+                    <failOnWarning>false</failOnWarning>
                 </configuration>
                 <!-- build a multi-release jar -->
                 <executions>
@@ -335,38 +320,10 @@
                 </executions>
             </plugin>
 
-            <!--        Enable once https://github.com/osundblad/intellij-annotations-instrumenter-maven-plugin/issues/53  has been fixed
-                        <plugin>
-                            <groupId>se.eris</groupId>
-                            <artifactId>notnull-instrumenter-maven-plugin</artifactId>
-                            <executions>
-                                <execution>
-                                    <goals>
-                                        <goal>instrument</goal>
-                                        <goal>tests-instrument</goal>
-                                    </goals>
-                                </execution>
-                            </executions>
-                            <dependencies>
-                                <dependency>
-                                    <groupId>org.jetbrains</groupId>
-                                    <artifactId>annotations</artifactId>
-                                    <version>19.0.0</version>
-                                </dependency>
-                            </dependencies>
-                        </plugin>-->
-
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-install-plugin</artifactId>
             </plugin>
-            <plugin>
-                <groupId>org.apache.maven.plugins</groupId>
-                <artifactId>maven-javadoc-plugin</artifactId>
-                <configuration>
-                    <source>8</source>
-                </configuration>
-            </plugin>
         </plugins>
     </build>
 
@@ -380,7 +337,7 @@
         <profile>
             <id>java11</id>
             <activation>
-                <jdk>[11,</jdk>
+                <jdk>[11,)</jdk>
             </activation>
             <build>
                 <plugins>
@@ -394,6 +351,7 @@
                                 <arg>--add-exports=java.base/jdk.internal.ref=ALL-UNNAMED</arg>
                                 <arg>--add-exports=java.base/sun.nio.ch=ALL-UNNAMED</arg>
                             </compilerArgs>
+                            <failOnWarning>false</failOnWarning>
                         </configuration>
                     </plugin>
                 </plugins>

src/main/java/net/openhft/chronicle/core/ChronicleInit.java

@@ -20,6 +20,8 @@
 
 import java.util.ServiceLoader;
 
+import static net.openhft.chronicle.core.internal.Bootstrap.uncheckedCast;
+
 /**
  * Handles application code which must be loaded first/run and may override system properties.
  * <p>
@@ -50,8 +52,8 @@ private ChronicleInit() {
         String initRunnableClass = System.getProperty(CHRONICLE_INIT_CLASS);
         if (initRunnableClass != null && !initRunnableClass.isEmpty()) {
             try {
-                Class<? extends Runnable> descendant = (Class<? extends Runnable>) Class.forName(initRunnableClass);
-                Runnable chronicleInit = descendant.newInstance();
+                Class<? extends Runnable> descendant = uncheckedCast(Class.forName(initRunnableClass));
+                Runnable chronicleInit = descendant.getConstructor().newInstance();
                 chronicleInit.run();
             } catch (Exception ex) {
                 // System.err since the logging subsystem may not be up at this point
@@ -89,8 +91,8 @@ static void postInit() {
         String initRunnableClass = System.getProperty(CHRONICLE_POSTINIT_CLASS);
         if (initRunnableClass != null && !initRunnableClass.isEmpty()) {
             try {
-                Class<? extends Runnable> descendant = (Class<? extends Runnable>) Class.forName(initRunnableClass);
-                Runnable chronicleInit = descendant.newInstance();
+                Class<? extends Runnable> descendant = uncheckedCast(Class.forName(initRunnableClass));
+                Runnable chronicleInit = descendant.getConstructor().newInstance();
                 chronicleInit.run();
             } catch (Exception ex) {
                 // System.err since the logging subsystem may not be up at this point

src/main/java/net/openhft/chronicle/core/Jvm.java

@@ -574,7 +574,7 @@ public static Field getFieldOrNull(@NotNull final Class<?> clazz, @NotNull final
     @NotNull
     public static Method getMethod(@NotNull final Class<?> clazz,
                                    @NotNull final String methodName,
-                                   final Class... argTypes) {
+                                   final Class<?>... argTypes) {
         return ClassUtil.getMethod0(clazz, methodName, argTypes, true);
     }
 
@@ -618,7 +618,7 @@ public static <V> V getValue(@NotNull Object target, @NotNull final String field
             }
             aClass = target.getClass();
         }
-        return (V) target;
+        return uncheckedCast(target);
     }
 
     /**
@@ -1223,8 +1223,7 @@ public static long parseSize(@NotNull String value) throws IllegalArgumentExcept
             }
         }
         double number = Double.parseDouble(value.trim());
-        factor *= number;
-        return factor;
+        return Math.round(factor * number);
     }
 
     /**
@@ -1310,9 +1309,10 @@ private static void doNotCloseOnInterrupt9(final Class<?> clazz, final FileChann
             final Class<?> interruptibleClass = field.getType();
             ClassUtil.setAccessible(field);
             final CommonInterruptible ci = new CommonInterruptible(clazz, fc);
+            Class<?>[] interfaces = {interruptibleClass};
             field.set(fc, Proxy.newProxyInstance(
                     interruptibleClass.getClassLoader(),
-                    new Class[]{interruptibleClass},
+                    interfaces,
                     (p, m, a) -> {
                         if (m.getDeclaringClass() != Object.class)
                             ci.interrupt();
@@ -1450,7 +1450,7 @@ public static int objectHeaderSize() {
      * @return The object header size or array base offset, depending on the class type.
      */
     @Deprecated(/* to be removed in x.27, use net.openhft.chronicle.bytes.BytesUtil.triviallyCopyableStart for POJOs */)
-    public static int objectHeaderSize(Class type) {
+    public static int objectHeaderSize(Class<?> type) {
         return ObjectHeaderSizeHolder.objectHeaderSize(type);
     }
 
@@ -1505,7 +1505,6 @@ public static <A extends Annotation> A findAnnotation(AnnotatedElement annotated
      * Typically, the name of these classes contains the "$$Lambda" substring.
      * Note that this naming convention is JVM-specific and can change
      * in future versions. The approach is known to work up to Java 21.
-     * </p>
      *
      * @param clazz the class to be checked.
      * @return {@code true} if the class is a lambda, {@code false} otherwise.

src/main/java/net/openhft/chronicle/core/Memory.java

@@ -19,35 +19,18 @@
 package net.openhft.chronicle.core;
 
 import net.openhft.chronicle.core.annotation.Positive;
-import net.openhft.chronicle.core.util.Longs;
 import net.openhft.chronicle.core.util.MisAlignedAssertionError;
 import org.jetbrains.annotations.NotNull;
-import org.jetbrains.annotations.Nullable;
 import sun.misc.Unsafe;
 
 import java.lang.reflect.Field;
 import java.nio.ByteBuffer;
 
-import static java.util.Objects.nonNull;
-import static net.openhft.chronicle.assertions.AssertUtil.SKIP_ASSERTIONS;
-import static net.openhft.chronicle.core.util.Longs.assertIfEnabled;
-
 /**
  * The Memory interface provides low-level memory access methods.
  */
 public interface Memory {
 
-    /**
-     * Retrieves the amount of heap memory currently used by the application.
-     *
-     * @return The number of bytes used in the heap memory.
-     */
-    @Deprecated(/* remove in x.26 */)
-    default long heapUsed() {
-        Runtime runtime = Runtime.getRuntime();
-        return runtime.totalMemory() - runtime.freeMemory();
-    }
-
     /**
      * Ensures that all previous writes to memory are completed before any
      * subsequent memory access instruction.
@@ -368,30 +351,6 @@ default long heapUsed() {
      */
     void copyMemory(long fromAddress, long address, long length);
 
-        /**
-         * Deprecated method. Copies a range of bytes from the given byte array to the specified object at the given offset.
-         *
-         * @param bytes   the source byte array
-         * @param offset  the starting index in the byte array
-         * @param obj2    the destination object
-         * @param offset2 the starting offset in the destination object
-         * @param length  the number of bytes to copy
-         * @deprecated This method is deprecated and will be removed in a future version (x.24).
-         */
-    @Deprecated(/* to be removed in x.26 */)
-    default void copyMemory(byte[] src, int srcOffset, @Nullable Object dest, long destOffset, int length) {
-        assert SKIP_ASSERTIONS || nonNull(src);
-        assert SKIP_ASSERTIONS || assertIfEnabled(Longs.nonNegative(), srcOffset);
-        assert SKIP_ASSERTIONS || assertIfEnabled(Longs.nonNegative(), destOffset);
-        assert SKIP_ASSERTIONS || assertIfEnabled(Longs.nonNegative(), length);
-
-        if (dest instanceof byte[]) {
-            copyMemory(src, srcOffset, (byte[]) dest, Math.toIntExact(destOffset - Unsafe.ARRAY_BYTE_BASE_OFFSET), length);
-        } else {
-            copyMemory(src, Unsafe.ARRAY_BYTE_BASE_OFFSET + srcOffset, dest, destOffset, length);
-        }
-    }
-
     /**
      * Copies a range of memory from the given object to the specified memory address.
      *