Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Crash report: chisel bag & moving bits = crash #159

Closed
Kaisoul opened this issue Nov 1, 2016 · 5 comments
Closed

Crash report: chisel bag & moving bits = crash #159

Kaisoul opened this issue Nov 1, 2016 · 5 comments
Labels

Comments

@Kaisoul
Copy link

Kaisoul commented Nov 1, 2016

I made an iron chisel & a bit bag gathered a few stacks of granite bits when to move the bits into the bag then crash.....

crash-2016-11-01_18.15.30-server.txt

@Kaisoul
Copy link
Author

Kaisoul commented Nov 1, 2016

sorry about just dropping the crash log on here like this atm my pastbin login is being emo...

@AlgorithmX2
Copy link
Collaborator

Kinda weird that xreliquary is listening for stuff in the bit bag, but whatever I can test for it.


If you want to give the fix a try, https://dvs1.progwml6.com/jenkins/job/Chisels-and-Bits/36/artifact/build/libs/chiselsandbits-12.0.36.jar

Other wise I'll push out an official build in a little bit.

@P3pp3rF1y
Copy link

@AlgorithmX2 the reason that I am listening is so that I can actually display void tear contents when these are in inventories (because I need to update item handler caps on client side for that).
Yes it doesn't make sense to listen to bit bag as void tear will never go in there and I could add a blacklist of container possibly, but in general I don't know what kind of containers players can put void tear in and thus need to listen to all.
Honestly I would rather like to see caps have a way to sync with the packets so that I wouldn't have to do this listener magic at all.

@AlgorithmX2
Copy link
Collaborator

AlgorithmX2 commented Nov 3, 2016

Yeah, that would probably make more sense. Well its not like it was hard to fix, I guess its just one of those things that people will have to patch if they are sending custom packets to listeners. ( if they are not already testing for the player )

@AEnterprise
Copy link

@P3pp3rF1y could this be what you are looking for? MinecraftForge/MinecraftForge#3342

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

No branches or pull requests

4 participants