On Sun, Oct 01, 2000 at 05:25:42PM +0100, Chris Gilbert wrote: > On Sun, 01 Oct 2000, you wrote: > > note that this is unrelated to the question, but just something I spotted > > > He has the following code at the end of the hit function, to call the > > double_hit stuff: > > > > else { > > damage(ch, victim, dam, w_type); > > if (GET_EQ(ch, WEAR_DWIELD) && GET_HIT(victim) > 0) > > double_hit(ch, FIGHTING(ch), TYPE_UNDEFINED); > > } > > > > It's worth noting that if you're using bpl 14 (or higher) you want a check on > the return value of damage. If you don't there's the possibility that victim > may no long be valid, check fight.c for the return codes. If you don't you > may crash when you dereference victim... Actually, I checked through the code, and this isn't checked very much. BUT, it did help me solve a very annoying ranged-attack button that was crashing on this EXACT scenario! Thanks Chris! Adam +------------------------------------------------------------+ | Ensure that you have read the CircleMUD Mailing List FAQ: | | http://qsilver.queensu.ca/~fletchra/Circle/list-faq.html | +------------------------------------------------------------+
This archive was generated by hypermail 2b30 : 04/10/01 PDT