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

Tinning bug: "You make, but cannot pick up, a bag" #1021

Open
paulcee opened this issue May 6, 2023 · 2 comments
Open

Tinning bug: "You make, but cannot pick up, a bag" #1021

paulcee opened this issue May 6, 2023 · 2 comments
Labels

Comments

@paulcee
Copy link

paulcee commented May 6, 2023

NetHack 3.6.6 (tiles) on Windows 10: I was in the Gnomish Mines carrying (among other things) a tinning kit, and a bag called GOOD. I stepped on a hobbit corpse and tinned it. This evident bug occurred:

There is a hobbit corpse here; tin it? [ynq]
You make, but cannot pick up, a bag called GOOD.

I checked the contents of the square and they were a sling and a tin of hobbit meat. No bag -- that was still in my inventory. So this seems like a string pointer bug.

@pat-rankin
Copy link

Do you have perm_invent enabled? Whenever the persistent inventory window gets updated, every item in inventory gets reformatted and that churns through a set of temporary buffers used to hold the formatted text. If something was formatted before the inventory update but not displayed until after, it can end up with the wrong text (from some other item that reused its buffer).

This was fixed for to-be-3.7 nearly two years ago (commit e43ec0c); I don't think that the fixed ever made it into 3.6.x.

@paulcee
Copy link
Author

paulcee commented May 11, 2023

Yes, perm_invent is enabled; so it sounds like a fix is ready for the future. Thanks.

@nhmall nhmall added the 3.6 label May 11, 2023
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

3 participants