Turn classes into records.

Author: fniephaus

wasm/src/org.graalvm.wasm/src/org/graalvm/wasm/SymbolTable.java

@@ -149,62 +149,27 @@ public String toString() {
         }
     }
 
-    public static class TableInfo {
-        /**
-         * Lower bound on table size.
-         */
-        public final int initialSize;
-
-        /**
-         * Upper bound on table size.
-         * <p>
-         * <em>Note:</em> this is the upper bound defined by the module. A table instance might have
-         * a lower internal max allowed size in practice.
-         */
-        public final int maximumSize;
-
-        /**
-         * The element type of the table.
-         */
-        public final byte elemType;
-
-        public TableInfo(int initialSize, int maximumSize, byte elemType) {
-            this.initialSize = initialSize;
-            this.maximumSize = maximumSize;
-            this.elemType = elemType;
-        }
+    /**
+     * @param initialSize Lower bound on table size.
+     * @param maximumSize Upper bound on table size.
+     *            <p>
+     *            <em>Note:</em> this is the upper bound defined by the module. A table instance
+     *            might have a lower internal max allowed size in practice.
+     * @param elemType The element type of the table.
+     */
+    public record TableInfo(int initialSize, int maximumSize, byte elemType) {
     }
 
-    public static class MemoryInfo {
-        /**
-         * Lower bound on memory size (in pages of 64 kiB).
-         */
-        public final long initialSize;
-
-        /**
-         * Upper bound on memory size (in pages of 64 kiB).
-         * <p>
-         * <em>Note:</em> this is the upper bound defined by the module. A memory instance might
-         * have a lower internal max allowed size in practice.
-         */
-        public final long maximumSize;
-
-        /**
-         * If the memory uses index type 64.
-         */
-        public final boolean indexType64;
-
-        /**
-         * Whether the memory is shared (modifications are visible to other threads).
-         */
-        public final boolean shared;
-
-        public MemoryInfo(long initialSize, long maximumSize, boolean indexType64, boolean shared) {
-            this.initialSize = initialSize;
-            this.maximumSize = maximumSize;
-            this.indexType64 = indexType64;
-            this.shared = shared;
-        }
+    /**
+     * @param initialSize Lower bound on memory size (in pages of 64 kiB).
+     * @param maximumSize Upper bound on memory size (in pages of 64 kiB).
+     *            <p>
+     *            <em>Note:</em> this is the upper bound defined by the module. A memory instance
+     *            might have a lower internal max allowed size in practice.
+     * @param indexType64 If the memory uses index type 64.
+     * @param shared Whether the memory is shared (modifications are visible to other threads).
+     */
+    public record MemoryInfo(long initialSize, long maximumSize, boolean indexType64, boolean shared) {
     }
 
     /**

wasm/src/org.graalvm.wasm/src/org/graalvm/wasm/WasmCustomSection.java

@@ -42,44 +42,10 @@
 
 /**
  * Information about a custom section.
+ *
+ * @param name Name of the custom section.
+ * @param offset Location in the custom sections array.
+ * @param length Length of the custom section.
  */
-public class WasmCustomSection {
-    /**
-     * Name of the custom section.
-     */
-    private final String name;
-
-    /**
-     * Location in the custom sections array.
-     */
-    private final int offset;
-
-    /**
-     * Length of the custom section.
-     */
-    private final int length;
-
-    public WasmCustomSection(String name, int offset, int length) {
-        this.name = name;
-        this.offset = offset;
-        this.length = length;
-    }
-
-    /**
-     * Returns the name of the custom section.
-     * 
-     * @return name of the custom section.
-     */
-    public String name() {
-        return name;
-    }
-
-    public int offset() {
-        return offset;
-    }
-
-    public int length() {
-        return length;
-    }
-
+public record WasmCustomSection(String name, int offset, int length) {
 }

wasm/src/org.graalvm.wasm/src/org/graalvm/wasm/debugging/parser/DebugParseUnit.java

@@ -46,38 +46,10 @@
 /**
  * Represents the result of parsing debug information.
  */
-public final class DebugParseUnit {
-    private final DebugData rootData;
-    private final EconomicMap<Integer, DebugData> entries;
-    private final EconomicMap<Integer, AbbreviationDeclaration> abbreviationTable;
-    private final int entryOffset;
-    private final int compilationUnitOffset;
-
-    public DebugParseUnit(DebugData rootData, EconomicMap<Integer, DebugData> entries, EconomicMap<Integer, AbbreviationDeclaration> abbreviationTable, int entryOffset, int compilationUnitOffset) {
-        this.rootData = rootData;
-        this.entries = entries;
-        this.abbreviationTable = abbreviationTable;
-        this.entryOffset = entryOffset;
-        this.compilationUnitOffset = compilationUnitOffset;
-    }
-
-    public DebugData rootData() {
-        return rootData;
-    }
-
-    public EconomicMap<Integer, DebugData> entries() {
-        return entries;
-    }
-
-    public EconomicMap<Integer, AbbreviationDeclaration> abbreviationTable() {
-        return abbreviationTable;
-    }
-
-    public int entryOffset() {
-        return entryOffset;
-    }
-
-    public int compilationUnitOffset() {
-        return compilationUnitOffset;
-    }
+public record DebugParseUnit(
+                DebugData rootData,
+                EconomicMap<Integer, DebugData> entries,
+                EconomicMap<Integer, AbbreviationDeclaration> abbreviationTable,
+                int entryOffset,
+                int compilationUnitOffset) {
 }

wasm/src/org.graalvm.wasm/src/org/graalvm/wasm/nodes/WasmFunctionNode.java

@@ -189,14 +189,7 @@ private Vector128Ops<V128> vector128Ops() {
     }
 
     // region OSR support
-    private static final class WasmOSRInterpreterState {
-        final int stackPointer;
-        final int line;
-
-        WasmOSRInterpreterState(int stackPointer, int line) {
-            this.stackPointer = stackPointer;
-            this.line = line;
-        }
+    private record WasmOSRInterpreterState(int stackPointer, int line) {
     }
 
     @Override