diff --git a/src/main/java/appeng/parts/misc/ItemHandlerAdapter.java b/src/main/java/appeng/parts/misc/ItemHandlerAdapter.java index eaef832a3..5d189de7e 100644 --- a/src/main/java/appeng/parts/misc/ItemHandlerAdapter.java +++ b/src/main/java/appeng/parts/misc/ItemHandlerAdapter.java @@ -99,8 +99,8 @@ class ItemHandlerAdapter implements IMEInventory, IBaseMonitor, IBaseMonitor maxSize, even if we - // request more. So even if it returns a valid stack, it might have more stuff. + // We have to loop here because according to the docs, the handler shouldn't return a stack with size > + // maxSize, even if we request more. So even if it returns a valid stack, it might have more stuff. do { extracted = itemHandler.extractItem( i, remainingSize, simulate ); @@ -127,8 +127,9 @@ class ItemHandlerAdapter implements IMEInventory, IBaseMonitor remainingSize ) { - // Something broke. It should never return more than we requested... We're going to silently eat the remainder - AELog.warn( "Mod that provided item handler {} is broken. Returned {} items, even though we requested {}.", + // Something broke. It should never return more than we requested... + // We're going to silently eat the remainder + AELog.warn( "Mod that provided item handler %1 is broken. Returned %2 items, even though we requested %3.", itemHandler.getClass().getSimpleName(), extracted.stackSize, remainingSize ); extracted.stackSize = remainingSize; } @@ -142,7 +143,7 @@ class ItemHandlerAdapter implements IMEInventory, IBaseMonitor 0 );