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

Game crashes when opening inventory after picking up specific item(s) #38

Closed
FelixMoelder opened this issue Mar 14, 2019 · 8 comments
Closed
Labels
bug Something isn't working

Comments

@FelixMoelder
Copy link
Contributor

While it is possible to open the inventory all the time I have the issue, breaking the game when opening the inventory after picking up a "Book of Holy Bold".
After that happened the first time, I started a new run and checked if I can open the inventory in the dungeons. While that was no problem it crashed directly after picking up that book and opening the inventory.

This issue occurred for me playing with a 32bit build on macOS Mojave.

@AJenbo
Copy link
Member

AJenbo commented Mar 14, 2019

Could you please provide a console log from the crash and possibly a save game that makes this easy to replicate?

@FelixMoelder
Copy link
Contributor Author

Sure!
I just tried it again and created a save directly in front of a book lieing on the floor. That bug seems to affect different (all?) kind of books.
single_0.sv.zip
devilutionX_crashreport.txt

@AJenbo
Copy link
Member

AJenbo commented Mar 14, 2019

Thanks this is very helpful, the log makes it clear that the issue is with rendering red items (higher stats required the the player has), it should be easy enough to solve.

@AJenbo
Copy link
Member

AJenbo commented Mar 16, 2019

This patch should fix the issue:
diasurgical/devilution#657
I will update this issue once it is merged into master, and again once we have merged 32bit Mac.

@FelixMoelder
Copy link
Contributor Author

Sounds great!
I was able to reach level 13 on macOS without any new issues besides the ones that were already reported.
I might be able to test the patch tomorrow and check if the crashes are solved.

@FelixMoelder
Copy link
Contributor Author

The patch solves this issue indeed.
Picking up items which the character can not use and opening the inventory is now possible.

@AJenbo
Copy link
Member

AJenbo commented Mar 17, 2019

Thank for testing, that's good hear I'll prioritize getting this merged when ready and then close this issue.

@AJenbo AJenbo closed this as completed in d5a06dc Mar 17, 2019
@AJenbo
Copy link
Member

AJenbo commented Mar 17, 2019

It's now in master, now we just need the 32bit Mac part :)

@AJenbo AJenbo added the bug Something isn't working label Mar 23, 2019
AJenbo pushed a commit that referenced this issue Sep 27, 2019
kphoenix137 added a commit to kphoenix137/devilutionX that referenced this issue Apr 6, 2024
* Add music

* Add music
kphoenix137 added a commit to kphoenix137/devilutionX that referenced this issue Apr 6, 2024
* Add music

* Add music
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants