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

Crash when debug spawning item clashes with bugged smoker report #76008

Open
PatrikLundell opened this issue Aug 28, 2024 · 3 comments
Open

Crash when debug spawning item clashes with bugged smoker report #76008

PatrikLundell opened this issue Aug 28, 2024 · 3 comments
Labels
(S1 - Need confirmation) Report waiting on confirmation of reproducibility

Comments

@PatrikLundell
Copy link
Contributor

PatrikLundell commented Aug 28, 2024

Describe the bug

The following annoying report has been generated for quite some time, but so far it has just been annoying.

DEBUG : tried to put an item (charcoal) count (137880) in a container (fake_char_smoker) that cannot contain it: pocket unacceptable because tried to put too many charges of ammo in item

FUNCTION : put_in
FILE : C:\Cataclysm-DDA\src\item.cpp
LINE : 1803
VERSION : 0.G-11687-g348453333a

I believe it's caused by the smoking racks. I've seen inconsistencies in how much charcoal they would accept, so there may be an issue with the code determining how much space remains. Also note that the save is a couple of months old.

However now it causes the game to crash when somehow brought up by the debug item spawn menu, presumably because it being subjected to ImGui conversion.

Attach save file

Hawaiian Ocean View-trimmed.tar.gz

Steps to reproduce

  1. Load the save.
  2. Walk out of the basement (Exit to the East. I've walked a couple of steps further onto the path when getting out).
  3. Bring up the debug item spawn menu.
  4. Get it overlayed by the bug report above.
  5. Press up or down arrow to clear out the debug menu and see the error message without the clutter (probably not needed).
  6. 'I'gnore the error report.
  7. Have the game terminate.

Expected behavior

If an error report is overlayed on the debug item spawn menu the game shouldn't crash when the report is dismissed.

Screenshots

No response

Versions and configuration

  • OS: Windows
    • OS Version: 10.0.19045.4780 (22H2)
  • Game Version: 0.G-11687-g348453333a [64-bit]
  • Graphics Version: Tiles
  • Game Language: System language []
  • Mods loaded: [
    Dark Days Ahead [dda],
    Disable NPC Needs [no_npc_food],
    Portal Storms Ignore NPCs [personal_portal_storms],
    Slowdown Fungal Growth [no_fungal_growth]
    ]

Additional context

Edit:
Further info:
If I examine the first smoker I get the error report, and when that is out of the way, the debug spawning menu can be brought up and used successfully.
Thus, the issue is really about the inability of the changed UI system to handle error reports on top of it, rather than this particular trigger, which is easy to work around (but still a pain to get if you haven't saved in a while).

@PatrikLundell PatrikLundell added the (S1 - Need confirmation) Report waiting on confirmation of reproducibility label Aug 28, 2024
@PatrikLundell

This comment was marked as resolved.

@mqrause
Copy link
Contributor

mqrause commented Aug 28, 2024

#59189 for more info on the pocket error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
(S1 - Need confirmation) Report waiting on confirmation of reproducibility
Projects
None yet
Development

No branches or pull requests

3 participants
@PatrikLundell @mqrause and others