Replaced unnecessary @overwrite mixin with cancellable Inject at HEAD

briansemrau · Jul 30, 2019 · e788ce9 · e788ce9
1 parent cc1cae3
commit e788ce9
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/...ain/java/com/github/briansemrau/cosmeticarmorslots/mixin/common/PlayerInventoryMixin.java b/...ain/java/com/github/briansemrau/cosmeticarmorslots/mixin/common/PlayerInventoryMixin.java
@@ -93,16 +93,13 @@ public void onDeserialize(ListTag listTag_1, CallbackInfo ci) {
         }
     }
 
-    /**
-     * @author brian_flakes
-     */
-    @Overwrite
-    public int getInvSize() {
+    @Inject(method = "getInvSize", at = @At("HEAD"), cancellable = true)
+    public void onGetInvSize(CallbackInfoReturnable<Integer> cir) {
         int size = 0;
         for (DefaultedList list : combinedInventory) {
             size += list.size();
         }
-        return size;
+        cir.setReturnValue(size);
     }
 
     @Inject(method = "isInvEmpty", at = @At("HEAD"), cancellable = true)