Do not send empty upsert player info packets

There is no harm in this, but seemingly, some common mod blows up on this, and, well, why eat buffer?
PaperMC · May 1, 2024 · 368d1a7 · 368d1a7
1 parent 5cdcf86
commit 368d1a7
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/proxy/src/main/java/com/velocitypowered/proxy/tablist/VelocityTabList.java b/proxy/src/main/java/com/velocitypowered/proxy/tablist/VelocityTabList.java
@@ -166,7 +166,9 @@ public void addEntry(TabListEntry entry1) {
       return entry;
     });
 
-    this.connection.write(new UpsertPlayerInfoPacket(actions, List.of(playerInfoEntry)));
+    if (!actions.isEmpty()) {
+      this.connection.write(new UpsertPlayerInfoPacket(actions, List.of(playerInfoEntry)));
+    }
   }
 
   @Override