joe-vettek / Fluid-Drawers-Legacy

2 stars 3 forks source link

Server crash #12

Closed DragonMaster14545 closed 1 year ago

DragonMaster14545 commented 1 year ago

Game Version, Launcher Version, Mod Version

MC 1.18.2

Forge 36.2.20

Newest beta

Problem Description

Its the same crash as I already had in Crash #11

Crash Log

https://pastebin.com/7spex8RL

joe-vettek commented 1 year ago

The second repair version has been uploaded. This time, a new safety measure has been added. It was a problem with pumping water before, but this time it is a problem with adding water.

joe-vettek commented 1 year ago

There is no rush to close this issue, you can check the situation first. Because I am busy now, I can only judge by experience.

DragonMaster14545 commented 1 year ago

Ok thanks a lot for your quick reply

DragonMaster14545 commented 1 year ago

https://pastebin.com/WBUqLw2A Just got this crash

DragonMaster14545 commented 1 year ago

https://pastebin.com/ZCJy287d Happened again...

joe-vettek commented 1 year ago

that's a new problem

DragonMaster14545 commented 1 year ago

Ok... Does your mod cause it?

joe-vettek commented 1 year ago

and it also shows that the before efforts isn't meaningless. Tomorrow I'll continue to fix the remaining bugs.

DragonMaster14545 commented 1 year ago

Thanks a lot

joe-vettek commented 1 year ago

Ok... Does your mod cause it?

I think so, I'm actually surprised that create even accesses that function.

joe-vettek commented 1 year ago

Tomorrow I'll check all related functions and fix others may causing crash, currently all crashes are caused by the controller, which is determined by its caching mechanism.

joe-vettek commented 1 year ago

image

test for a while, no crash happens

joe-vettek commented 1 year ago

you can try the latest beta version

DragonMaster14545 commented 1 year ago

Ill do thanks, btw it ran good for to days so...

DragonMaster14545 commented 1 year ago

I got this crash in the night https://pastebin.com/tRt6TXuv

joe-vettek commented 1 year ago

you need use the latest 2.3.3-beta and now the version you use is 2.3.2-fix. sorry I don't say it clearly.

https://beta.curseforge.com/minecraft/mc-mods/fluid-drawers-legacy/files/4382751

DragonMaster14545 commented 1 year ago

Ok ill try that thanks

DragonMaster14545 commented 1 year ago

I have the latest beta version installed and just got this https://pastebin.com/iJfNUSbW Sorry for beeing so annoying

joe-vettek commented 1 year ago

By the way, I found another bug, which was fixed after checking. So version 2.3.5 was released. https://www.curseforge.com/minecraft/mc-mods/fluid-drawers-legacy/files/4385815

joe-vettek commented 1 year ago

The null pointer problem is due to the architecture, the caching mechanism requires that the drawer index collection be updated every five seconds (this is consistent with the storage drawer mod). In the old version, the update is completed by clearing the previous drawer index set and then adding the drawer index again. The new version first checks whether the old and new index sets are identical before considering whether to directly replace the entire index set. In addition, the rebuilt lock has also been updated (it is responsible for finally judging whether index updates can be performed), all adjusted to semaphores instead of boolean locks that are prone to failure during multi-threaded access.

joe-vettek commented 1 year ago

The final defense plan is to use try catch, but this is a more irresponsible way of writing, so I will not consider this. Thank you for your patience. It is difficult for developers to determine the cause of this hidden error

DragonMaster14545 commented 1 year ago

Thanks for investing so much time in fixing my problem