wisp-forest / owo-lib

Open ωorthωhile Operations, yes the acronym was "totally accidental"
https://modrinth.com/mod/owo-lib
MIT License
191 stars 37 forks source link

Crash only reproducible if Minecraft freezes and you remove a parent component #224

Open Zailer43 opened 5 months ago

Zailer43 commented 5 months ago

Description

I was with a friend on a server and he crashed using my mod, I asked him for the crash report, and from what I could notice, I can't reproduce it. Then he told me that Minecraft froze before the crash, which makes sense if we look at where it crashed. When calling the queue parent, it wasn't null, but inside the queue it was because while it froze, he clicked and closed an overlay that was the parent.

https://github.com/wisp-forest/owo-lib/blob/a06c4776d40248f2f897d86a94ea9b48187c603d/src/main/java/io/wispforest/owo/ui/core/Component.java#L314-L319

Crash log

https://gist.github.com/Zailer43/dcac94e1897b86ab30a5eba98a0d54e3

Versions

owo-lib = 0.12.6+1.20.3 fabric-api = 0.96.1+1.20.4

gliscowo commented 5 months ago

I can certainly see how this would happen, I'll probably just capture the parent at queue time and pass that into the actual task. Given how this is a rather non-critical problem though it might be a little while before I actually get around to implementing and publishing that

Either way, thanks for reporting! Cheers