From f99d20ade34a4dcd17e4ad681fec6072a379e951 Mon Sep 17 00:00:00 2001
From: ineanto <pantoine.rochas@gmail.com>
Date: Mon, 19 Jun 2023 15:51:37 +0200
Subject: [PATCH] refactor(wrappers): unused getters

---
 .../wrapper/WrapperPlayServerRespawn.java     | 36 -------------------
 .../WrapperPlayerServerPlayerInfo.java        | 12 -------
 .../WrapperPlayerServerPlayerInfoRemove.java  |  8 -----
 3 files changed, 56 deletions(-)

diff --git a/src/main/java/xyz/atnrch/nicko/wrapper/WrapperPlayServerRespawn.java b/src/main/java/xyz/atnrch/nicko/wrapper/WrapperPlayServerRespawn.java
index 9908724..38f0a84 100644
--- a/src/main/java/xyz/atnrch/nicko/wrapper/WrapperPlayServerRespawn.java
+++ b/src/main/java/xyz/atnrch/nicko/wrapper/WrapperPlayServerRespawn.java
@@ -38,14 +38,6 @@ public class WrapperPlayServerRespawn extends AbstractPacket {
     // your code to change things that were working perfectly fine before
     //.............
 
-    public World getDimension() {
-        if (MinecraftVersion.WILD_UPDATE.atOrAbove()) {
-            return handle.getWorldKeys().read(0);
-        }
-
-        return handle.getDimensionTypes().read(0);
-    }
-
     public void setDimension(World value) {
         if (MinecraftVersion.WILD_UPDATE.atOrAbove()) {
             // 1.19 to 1.19.4
@@ -71,10 +63,6 @@ public class WrapperPlayServerRespawn extends AbstractPacket {
     // GameMode Field
     //.............
 
-    public GameMode getGameMode() {
-        return handle.getGameModes().read(0).toBukkit();
-    }
-
     public void setGameMode(GameMode value) {
         handle.getGameModes().write(0, EnumWrappers.NativeGameMode.fromBukkit(value));
     }
@@ -83,10 +71,6 @@ public class WrapperPlayServerRespawn extends AbstractPacket {
     // Previous GameMode Field
     //.............
 
-    public GameMode getPreviousGameMode() {
-        return handle.getGameModes().read(1).toBukkit();
-    }
-
     public void setPreviousGameMode(GameMode value) {
         handle.getGameModes().write(1, EnumWrappers.NativeGameMode.fromBukkit(value));
     }
@@ -95,10 +79,6 @@ public class WrapperPlayServerRespawn extends AbstractPacket {
     // Copy Metadata Field
     //.............
 
-    public boolean isCopyMetadata() {
-        return handle.getBytes().read(0) != 0;
-    }
-
     public void setCopyMetadata(boolean value) {
         if(MinecraftVersion.FEATURE_PREVIEW_UPDATE.atOrAbove()) {
             handle.getBytes().write(0, ((byte) (value ? 0x01 : 0x00)));
@@ -112,13 +92,6 @@ public class WrapperPlayServerRespawn extends AbstractPacket {
     // Added in 1.15.
     //.............
 
-    public long getSeed() {
-        if (MinecraftVersion.BEE_UPDATE.atOrAbove()) {
-            return handle.getLongs().read(0);
-        }
-        return -1;
-    }
-
     public void setSeed(long value) {
         if (MinecraftVersion.BEE_UPDATE.atOrAbove()) {
             handle.getLongs().write(0, Hashing.sha256().hashLong(value).asLong());
@@ -130,15 +103,6 @@ public class WrapperPlayServerRespawn extends AbstractPacket {
     // Removed in 1.14.
     //.............
 
-    public Difficulty getDifficulty() {
-        if (MinecraftVersion.VILLAGE_UPDATE.atOrAbove()) {
-            return null;
-        }
-
-        final EnumWrappers.Difficulty difficulty = handle.getDifficulties().read(0);
-        return difficulty == null ? null : Difficulty.valueOf(difficulty.name());
-    }
-
     public void setDifficulty(Difficulty difficulty) {
         if (difficulty != null && !MinecraftVersion.VILLAGE_UPDATE.atOrAbove()) {
             handle.getDifficulties().write(0, EnumWrappers.Difficulty.valueOf(difficulty.name()));
diff --git a/src/main/java/xyz/atnrch/nicko/wrapper/WrapperPlayerServerPlayerInfo.java b/src/main/java/xyz/atnrch/nicko/wrapper/WrapperPlayerServerPlayerInfo.java
index 86ee3f5..32d6f07 100644
--- a/src/main/java/xyz/atnrch/nicko/wrapper/WrapperPlayerServerPlayerInfo.java
+++ b/src/main/java/xyz/atnrch/nicko/wrapper/WrapperPlayerServerPlayerInfo.java
@@ -24,14 +24,6 @@ public class WrapperPlayerServerPlayerInfo extends AbstractPacket {
         handle.getModifier().writeDefaults();
     }
 
-    public WrapperPlayerServerPlayerInfo(PacketContainer packet) {
-        super(packet, TYPE);
-    }
-
-    public Set<EnumWrappers.PlayerInfoAction> getActions() {
-        return handle.getPlayerInfoActions().read(0);
-    }
-
     public void setActions(Set<EnumWrappers.PlayerInfoAction> value) {
         if (MinecraftVersion.FEATURE_PREVIEW_UPDATE.atOrAbove()) {
             handle.getPlayerInfoActions().write(0, value);
@@ -40,10 +32,6 @@ public class WrapperPlayerServerPlayerInfo extends AbstractPacket {
         }
     }
 
-    public List<PlayerInfoData> getData() {
-        return handle.getPlayerInfoDataLists().read(1);
-    }
-
     public void setData(List<PlayerInfoData> value) {
         handle.getPlayerInfoDataLists().write(1, value);
     }
diff --git a/src/main/java/xyz/atnrch/nicko/wrapper/WrapperPlayerServerPlayerInfoRemove.java b/src/main/java/xyz/atnrch/nicko/wrapper/WrapperPlayerServerPlayerInfoRemove.java
index a218e67..5c48f9d 100644
--- a/src/main/java/xyz/atnrch/nicko/wrapper/WrapperPlayerServerPlayerInfoRemove.java
+++ b/src/main/java/xyz/atnrch/nicko/wrapper/WrapperPlayerServerPlayerInfoRemove.java
@@ -21,14 +21,6 @@ public class WrapperPlayerServerPlayerInfoRemove extends AbstractPacket {
         handle.getModifier().writeDefaults();
     }
 
-    public WrapperPlayerServerPlayerInfoRemove(PacketContainer packet) {
-        super(packet, TYPE);
-    }
-
-    public List<UUID> getUUIDs() {
-        return handle.getUUIDLists().read(0);
-    }
-
     public void setUUIDs(List<UUID> value) {
         handle.getUUIDLists().write(0, value);
     }