Fixed PlayerDigging status reading, as it's an unsigned byte, not signed.

Fixed WindowConfirmation, OpenHorseWindow, WindowConfirmation window id reading as it's an unsigned byte, not signed.

api/src/main/java/com/github/retrooper/packetevents/wrapper/play/client/WrapperPlayClientPlayerDigging.java

@@ -59,7 +59,7 @@ public class WrapperPlayClientPlayerDigging extends PacketWrapper<WrapperPlayCli
             int z = readInt();
             blockPosition = new Vector3i(x, y, z);
         }
-        byte face = readByte();
+        short face = readUnsignedByte();
         blockFace = BlockFace.getBlockFaceByValue(face);
     }
 

api/src/main/java/com/github/retrooper/packetevents/wrapper/play/client/WrapperPlayClientWindowConfirmation.java

@@ -55,7 +55,7 @@ public class WrapperPlayClientWindowConfirmation extends PacketWrapper<WrapperPl
 
     @Override
     public void readData() {
-        windowId = readByte();
+        windowId = readUnsignedByte();
         actionId = readShort();
         accepted = readBoolean();
     }

api/src/main/java/com/github/retrooper/packetevents/wrapper/play/server/WrapperPlayServerOpenHorseWindow.java

@@ -5,7 +5,7 @@ import com.github.retrooper.packetevents.protocol.packettype.PacketType;
 import com.github.retrooper.packetevents.wrapper.PacketWrapper;
 
 public class WrapperPlayServerOpenHorseWindow extends PacketWrapper<WrapperPlayServerOpenHorseWindow> {
-    byte windowId;
+    int windowId;
     int slotCount;
     int entityId;
 
@@ -13,7 +13,7 @@ public class WrapperPlayServerOpenHorseWindow extends PacketWrapper<WrapperPlayS
         super(event);
     }
 
-    public WrapperPlayServerOpenHorseWindow(byte windowId, int slotCount, int entityId) {
+    public WrapperPlayServerOpenHorseWindow(int windowId, int slotCount, int entityId) {
         super(PacketType.Play.Server.OPEN_HORSE_WINDOW);
         this.windowId = windowId;
         this.slotCount = slotCount;
@@ -22,7 +22,7 @@ public class WrapperPlayServerOpenHorseWindow extends PacketWrapper<WrapperPlayS
 
     @Override
     public void readData() {
-        this.windowId = (byte) readUnsignedByte();
+        this.windowId = readUnsignedByte();
         this.slotCount = readVarInt();
         this.entityId = readInt();
     }
@@ -41,11 +41,11 @@ public class WrapperPlayServerOpenHorseWindow extends PacketWrapper<WrapperPlayS
         writeInt(entityId);
     }
 
-    public byte getWindowId() {
+    public int getWindowId() {
         return windowId;
     }
 
-    public void setWindowId(byte windowId) {
+    public void setWindowId(int windowId) {
         this.windowId = windowId;
     }
 

api/src/main/java/com/github/retrooper/packetevents/wrapper/play/server/WrapperPlayServerWindowConfirmation.java

@@ -63,7 +63,7 @@ public class WrapperPlayServerWindowConfirmation extends PacketWrapper<WrapperPl
 
     @Override
     public void readData() {
-        this.windowId = readByte();
+        this.windowId = readUnsignedByte();
         this.actionId = readShort();
         this.accepted = readBoolean();
     }