Don't cache Item

bukkit/src/main/java/us/myles/ViaVersion/bukkit/listeners/protocol1_9to1_8/HandItemCache.java

@@ -35,7 +35,7 @@ public class HandItemCache extends BukkitRunnable {
     }
 
     public static Item convert(ItemStack itemInHand) {
-        if (itemInHand == null) return Item.EMPTY;
+        if (itemInHand == null) new Item((short) 0, (byte) 0, (short) 0, null);
         return new Item((short) itemInHand.getTypeId(), (byte) itemInHand.getAmount(), itemInHand.getDurability(), null);
     }
 }

common/src/main/java/us/myles/ViaVersion/api/minecraft/item/Item.java

@@ -11,7 +11,6 @@ import lombok.*;
 @ToString
 @EqualsAndHashCode
 public class Item {
-    public static final Item EMPTY = new Item((short) 0, (byte) 0, (short) 0, null);
     @SerializedName(value = "identifier", alternate = "id")
     private int identifier;
     private byte amount;

sponge/src/main/java/us/myles/ViaVersion/sponge/listeners/protocol1_9to1_8/HandItemCache.java

@@ -51,7 +51,7 @@ public class HandItemCache implements Runnable {
     }
 
     public static Item convert(ItemStack itemInHand) {
-        if (itemInHand == null) return Item.EMPTY;
+        if (itemInHand == null) return new Item((short) 0, (byte) 0, (short) 0, null);
         if (GET_DAMAGE == null) {
             try {
                 GET_DAMAGE = itemInHand.getClass().getDeclaredField("field_77991_e");