On Tue, Aug 14, 2012 at 09:17:30AM -0700, [email protected] wrote: > james > 2012-08-14 09:17:30 -0700 (Tue, 14 Aug 2012) > 128 > Add check_used_onetime_npcs() which currently just prints out a bunch of > debugging > information when you go into the map editor. > ---
I have to do more testing to be sure, but I think this has exposed two bugs: 1) The cache of used onetime bits in gen() appears to be corrupt in Wandering Hamster and Vikings of Midguard. it might be corrupt in all games, and nobody has complained because nobody has used up ever onetimeuse tag yet. My plan for dealing with this is to abandon the onetime usage cache entirely. Re-checking every NPC on every map can happen so fast that I think it is best to do it on the fly, when you set an NPC to be onetime-usable. I think I will still continue to use the indexer that counts up, so it will prefer using never-before-used onetime tags in favor of re-using old freed ones. 2) Duplicate onetime use tags happen frequently. The only case I know for sure is when you copy a map, the NPCs in the copy keep the same onetime tags. This raises the issue that this is probably intended behavior, and it would be really nice to be able to do it on purpose. My plan for this is to turn it into a feature. I will add a way to explicitly set the onetime tag you want to use, and to see a list of which other NPCs use it already. This will make it possible to use the same onetime tag for a onetime npc on multiple copies of a map, even if you didn't create the NPC before the map. --- James _______________________________________________ Ohrrpgce mailing list [email protected] http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org
