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

Catching a fish with a full inventory causes a crash to desktop #30

Closed
randomname110101 opened this issue Oct 29, 2021 · 4 comments
Closed

Comments

@randomname110101
Copy link

I was fishing when I noticed the game would consistently crash on the third fish caught. After clearing some room in my inventory I tried fishing again and it did not crash. Currently on the latest version.

@randomname110101
Copy link
Author

Oh, I noticed .1.1 released so I tried and successfully reproduced it, and got this error dump:
HspError 7 WinError 0 Line -1 Ver 6070 Mode 0
Message (JP): ”z—ñ‚Ì—v‘f‚ª–³Œø‚Å‚·
Message (EN): Array overflow
efId 185 efP 0 Area 7/1
ci 0 ti -1#574 cc 0#0 tc 0#0 rc 54#41
3. main_init:End ct:0
2. turn_end:pc ct:0

  1. main_init:Begin ct:245
  2. main_init:End ct:0
  • ci is below 0

@Ruin0x11
Copy link
Owner

I can't seem to reproduce this with the latest version of the code. The game correctly prevents fishing and indicates the inventory is full. It might be related to something other than the current inventory size.

If this still happens with 2.07.2.0, do reopen this issue so I can investigate further.

@randomname110101
Copy link
Author

Just tried again and it crashed again. Here's my save if you'd like to try to reproduce it. My steps: walk over to the water tile, mix water fleas with fishing rod, and fish for 6 hours. It eventually crashes when catching a fish with the same array overflow error.
save.zip

@Ruin0x11 Ruin0x11 reopened this Dec 13, 2021
Ruin0x11 added a commit that referenced this issue Dec 14, 2021
@Ruin0x11
Copy link
Owner

Turns out it had to do with the Super Lure. Interestingly, it's also a bug in stock Plus, but I guess nobody managed to trigger it up until now.

Fixed in 65500a3.

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

No branches or pull requests

2 participants