Mercurial > hg > early-roguelike
view urogue/TODO @ 298:5a94c9b3181e
UltraRogue: clear the next_obj field when removing items from the floor.
The next_obj field is a pointer which the top item in a stack uses to
keep a list of the other items. When removing an item from the stack,
rem_obj() failed to set next_obj to NULL, which can cause items in
monster inventory to point to items earlier in the inventory list.
That causes infinite co-recursion when saving or restoring.
| author | John "Elwin" Edwards |
|---|---|
| date | Thu, 08 Feb 2018 20:54:34 -0500 |
| parents | c495a4f288c6 |
| children |
line wrap: on
line source
UltraRogue: The Ultimate Adventure in the Dungeons of Doom Copyright (C) 1993, 1995 Herb Chong All rights reserved. %W% %G% add_pack() calls ur_free() on the linked list pointer passed to it. Need to go through code and look for cases where we use that pointer after calling add_pack(). Electric fence in guard-free() mode should help. Similary throw_away(), discard(), discard_pack() free items... there will be more like this... Bug in genocide scroll: don't request list, type letter, etc... Implement potion of true sight. Bug: apparently when creating a new familiar after a player level change, something isn't begin reset because ur SEGVs when running through the monster list in do_chase() right after the new familiar is created. It appears that the familiar isn't being removed from the monster list.