On Fri, Mar 10, 2006 at 10:39:05PM -0800, Mark Wedel wrote: > Robin Redeker wrote: > > > [...] > > else if (item->type == SCROLL && QUERY_FLAG(mon, FLAG_USE_SCROLL)) { > > if (monster_should_cast_spell(mon, item->inv)) > > [...] > > So we patched it like this: > > > > if (item->inv && monster_should_cast_spell(mon, item->inv)) > > I think you still need the item->type == SCROLL check in there - otherwise, > the monster will try to use spellbooks, wands, rods, containers - anything > with > an inventory, as a spell.
Of course we did not remove the line with the else if :) Sorry if it was misleading. I only meant that we changed the line: if (monster_should_cast_spell(mon, item->inv)) to: if (item->inv && monster_should_cast_spell(mon, item->inv)) Robin -- [EMAIL PROTECTED] / [EMAIL PROTECTED] / [EMAIL PROTECTED] Robin Redeker _______________________________________________ crossfire mailing list crossfire@metalforge.org http://mailman.metalforge.org/mailman/listinfo/crossfire