Refactor ChunkMetadataTable into ChunkMetadata

Author: Glavo

src/main/java/org/glavo/nbt/chunk/ChunkRegion.java

@@ -19,7 +19,6 @@
 import org.glavo.nbt.NBTElement;
 import org.glavo.nbt.NBTParent;
 import org.glavo.nbt.internal.ChunkMetadata;
-import org.glavo.nbt.internal.ChunkMetadataTable;
 import org.glavo.nbt.internal.ChunkUtils;
 import org.glavo.nbt.internal.input.DataReader;
 import org.glavo.nbt.internal.input.RawDataReader;
@@ -29,7 +28,6 @@
 import org.glavo.nbt.tag.Tag;
 
 import java.io.IOException;
-import java.util.List;
 import java.util.Objects;
 
 /// @see <a href="https://minecraft.wiki/w/Region_file_format">Region file format - Minecraft Wiki</a>
@@ -57,14 +55,10 @@ static ChunkRegion readRegion(RawDataReader rawReader) throws IOException {
             }
         }
 
-        var table = new ChunkMetadataTable(List.of(metadata));
-
-        List<ChunkMetadata> sortedBySectorOffset = table.getSortedBySectorOffset();
-
         var region = new ChunkRegion();
 
         long contentStart = rawReader.position();
-        for (ChunkMetadata chunkMetadata : sortedBySectorOffset) {
+        for (ChunkMetadata chunkMetadata : ChunkMetadata.sortedByOffset(metadata)) {
             long sectorStart = contentStart + (long) chunkMetadata.sectorOffset() * ChunkUtils.SECTOR_BYTES;
             long position = rawReader.position();
             if (position != sectorStart) {

src/main/java/org/glavo/nbt/internal/ChunkMetadata.java

@@ -15,7 +15,16 @@
  */
 package org.glavo.nbt.internal;
 
+import java.util.Comparator;
+
 public record ChunkMetadata(int localIndex, int sectorOffset, int sectorLength, int timestamp) {
+    private static final Comparator<ChunkMetadata> BY_OFFSET = Comparator.comparingInt(ChunkMetadata::sectorOffset).thenComparingInt(ChunkMetadata::localIndex);
+
+    public static ChunkMetadata[] sortedByOffset(ChunkMetadata[] metadata) {
+        ChunkMetadata[] copy = metadata.clone();
+        java.util.Arrays.sort(copy, BY_OFFSET);
+        return copy;
+    }
 
     public ChunkMetadata {
         assert localIndex >= 0 && localIndex < ChunkUtils.CHUNKS_PRE_REGION : "Invalid local index: " + localIndex;

src/main/java/org/glavo/nbt/internal/ChunkMetadataTable.java

@@ -0,0 +1,21 @@
+/*
+ * Copyright 2026 Glavo
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.glavo.nbt.internal.input;
+
+public final class NBTInput {
+    private NBTInput() {
+    }
+}

src/main/java/org/glavo/nbt/internal/input/NBTInput.java

@@ -22,6 +22,7 @@
 import org.glavo.nbt.internal.input.RawDataReader;
 import org.glavo.nbt.internal.input.InputSource;
 import org.glavo.nbt.internal.output.NBTWriter;
+import org.jetbrains.annotations.ApiStatus;
 import org.jetbrains.annotations.Contract;
 import org.jetbrains.annotations.Nullable;
 
@@ -35,6 +36,7 @@
 public sealed abstract class Tag implements NBTElement
         permits ValueTag, ArrayTag, ParentTag {
 
+    @ApiStatus.Internal
     public static @Nullable Tag readTag(DataReader reader) throws IOException {
         byte tagByte = reader.readByte();
         if (tagByte == 0) {