remove canHold check. #2119
remove canHold check. #2119
Conversation
src/Inventory.zig
Outdated
| if(sourceStack.item == null) return; | ||
| if(self.amount > sourceStack.amount) return; | ||
| if(!self.dest.canHold(.{.item = sourceStack.item, .amount = self.amount})) return; | ||
| if(!self.dest.canHold(.{.item = sourceStack.item, .amount = 1})) return; |
There was a problem hiding this comment.
I think this check can be entirely removed. The code that follows it seems to loop over every slot in the inventory and attempt to transfer as many items from the source as possible, it doesn't look like it will fail even if there isn't any room at all. I could be wrong though
There was a problem hiding this comment.
I see your point. You are right that it doesn't fail. I mean even before if we had as an example one full stack and then a second one which was almost full. It would execute a move operation on the full stack with amount 0 and then a move operation with whatever amount for the second slot. So now we just do it for every slot that is the same item. So at the end it is a question do we want to to make a move operation with amount 0 to all slots with the same item or do we just want to do a before check.
There was a problem hiding this comment.
Couldn't we just remove the check even?
There was a problem hiding this comment.
yes you are both right. I thought this could run the move command a lot. But my mind just ignored the if(remainingAmount == 0) break; line. So Fixed
Wunka
changed the title
3 participants
Fixes: #1881
just a little change that makes the amount requirement of shifting to an inventory not be the stack size