Skip to content
This repository has been archived by the owner on Apr 28, 2021. It is now read-only.

Crash in CropBlock (apparently), something to do with agglo plate matching #29

Closed
quat1024 opened this issue Oct 28, 2018 · 2 comments
Closed

Comments

@quat1024
Copy link
Owner

TheLimePixel/Crop-Block#48

time to investigate this and see if it still happens (this is a kinda old issue), since thats not good lol

@quat1024
Copy link
Owner Author

On phone rn but it looks like this is probably a bug involving agglomeration recipes where more than one ingredient is provided at a time. Which is a case I don't think I tested. The recipe matching code works by creating a copy of the user's input item stacks and a copy of the recipe's item stacks, and then attempting to consume items from the user input list that match the recipe (the goal being to consume all of the items) The problem is that one stack from the recipe actually consumes all the matching user inputs, which due to the really shitty code at play here lmao, can cause one of the lists to become empty.

I should redo that code anyways. Making a bunch of list copies for every recipe every tick doesn't sound... efficient.

It also sounds like there could be bugs involving tricking the recipe matcher into giving you items with the wrong combination of inputs. Whoops.

@quat1024
Copy link
Owner Author

Fixed by rewriting the recipe matcher, hopefully it's more robust now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant