[ci skip] Remove nullable from SyncedEntityData#packAll (#11949)

PaperMC · Jan 11, 2025 · 775002a · 775002a
1 parent ac3aaa2
commit 775002a
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/paper-server/patches/sources/net/minecraft/network/syncher/SynchedEntityData.java.patch b/paper-server/patches/sources/net/minecraft/network/syncher/SynchedEntityData.java.patch
@@ -23,17 +23,16 @@
      public boolean isDirty() {
          return this.isDirty;
      }
-@@ -169,6 +_,20 @@
+@@ -169,6 +_,19 @@
              return new SynchedEntityData(this.entity, this.itemsById);
          }
      }
 +
 +    // Paper start
 +    // We need to pack all as we cannot rely on "non default values" or "dirty" ones.
 +    // Because these values can possibly be desynced on the client.
-+    @Nullable
 +    public List<SynchedEntityData.DataValue<?>> packAll() {
-+        final List<SynchedEntityData.DataValue<?>> list = new ArrayList<>();
++        final List<SynchedEntityData.DataValue<?>> list = new ArrayList<>(this.itemsById.length);
 +        for (final DataItem<?> dataItem : this.itemsById) {
 +            list.add(dataItem.value());
 +        }