[ViewVC] Diff of: cvs/deliantra/server/server/spell

Comparing deliantra/server/server/spell_attack.C (file contents):
Revision 1.59 by root, Sat May 17 15:25:19 2008 UTC vs.
Revision 1.68 by elmex, Mon Oct 6 08:46:33 2008 UTC

   new_bolt->duration++;
   new_bolt->stats.dam /= 2;     /* reduce daughter bolt damage */
   new_bolt->stats.dam++;
   tmp->stats.dam /= 2;          /* reduce father bolt damage */
   tmp->stats.dam++;
   if ((new_bolt = m->insert (new_bolt, sx, sy, op)))
     update_turn_face (new_bolt);
 }
 /* move_bolt: moves bolt 'op'.  Basically, it just advances a space,
   sint16 x, y;
   maptile *m;
   if (--op->duration < 0)
     {
-      op->destroy ();
+      op->drop_and_destroy ();
       return;
     }
   hit_map (op, 0, op->attacktype, 1);
   maptile *newmap;
   mflags = get_map_flags (tmp->map, &newmap, tmp->x, tmp->y, &tmp->x, &tmp->y);
   if (mflags & P_OUT_OF_MAP)
     {
-      tmp->destroy ();
+      tmp->drop_and_destroy ();
       return 0;
     }
   tmp->map = newmap;
   if (OB_TYPE_MOVE_BLOCK (tmp, GET_MAP_MOVE_BLOCK (tmp->map, tmp->x, tmp->y)))
     {
       if (!QUERY_FLAG (tmp, FLAG_REFLECTING))
         {
-          tmp->destroy ();
+          tmp->drop_and_destroy ();
           return 0;
         }
       tmp->x = op->x;
       tmp->y = op->y;
 void
 explode_bullet (object *op)
 {
   object *tmp, *owner;
-  if (op->other_arch == NULL)
+  if (!op->other_arch)
     {
       LOG (llevError, "BUG: explode_bullet(): op without other_arch\n");
       op->destroy ();
       return;
     }
   tmp->set_owner (op);
   tmp->skill = op->skill;
   owner = op->owner;
-  if ((tmp->attacktype & AT_HOLYWORD || tmp->attacktype & AT_GODPOWER) && owner && !tailor_god_spell (tmp, owner))
+  if ((tmp->attacktype & AT_HOLYWORD
+       || tmp->attacktype & AT_GODPOWER)
+      && owner
+      && !tailor_god_spell (tmp, owner))
     {
       op->destroy ();
       return;
     }
     {
       if (QUERY_FLAG (tmp, FLAG_ALIVE))
         {
           dam = hit_player (tmp, op->stats.dam, op, op->attacktype, 1);
+          // TODO: can't understand the following if's
           if (op->destroyed () || !tmp->destroyed () || (op->stats.dam -= dam) < 0)
             {
               if (!QUERY_FLAG (op, FLAG_REMOVED))
                 {
                   op->destroy ();
     }
 #endif
   hit_map (op, 0, op->attacktype, 0);
+  if (!op->is_on_map ())
+    return;
   /* Check to see if we should push anything.
    * Spell objects with weight push whatever they encounter to some
    * degree.
    */
   if (op->weight)
+    {
-    check_spell_knockback (op);
+      check_spell_knockback (op);
-  if (op->destroyed ())
+      if (!op->is_on_map ())
-    return;
+        return;
+    }
-  if ((op->duration--) < 0)
+  if (op->duration-- < 0)
     {
       op->destroy ();
       return;
     }
   /* Object has hit maximum range, so don't have it move
       success = 1;
       tmp = arch_to_object (spell->other_arch);
       tmp->set_owner (op);
       set_spell_skill (op, caster, spell, tmp);
-      tmp->level = caster_level (caster, spell);
+      tmp->level = casting_level (caster, spell);
       tmp->attacktype = spell->attacktype;
       /* holy word stuff */
       if ((tmp->attacktype & AT_HOLYWORD) || (tmp->attacktype & AT_GODPOWER))
         if (!tailor_god_spell (tmp, op))
     effect = arch_to_object (spell->other_arch);
   else
     return 0;
   /* tailor the effect by priest level and worshipped God */
-  effect->level = caster_level (caster, spell);
+  effect->level = casting_level (caster, spell);
   effect->attacktype = spell->attacktype;
   if (effect->attacktype & (AT_HOLYWORD | AT_GODPOWER))
     {
       if (tailor_god_spell (effect, op))
         new_draw_info_format (NDI_UNIQUE, 0, op, "%s answers your call!", determine_god (op));
 void
 move_missile (object *op)
 {
   if (op->range-- <= 0)
     {
-      op->destroy ();
+      op->drop_and_destroy ();
       return;
     }
   mapxy pos (op);
   pos.move (op->direction);
   /* We precompute some values here so that we don't have to keep
    * doing it over and over again.
    */
   god = find_god (determine_god (op));
-  level = caster_level (caster, spell);
+  level = casting_level (caster, spell);
   range = spell->range + SP_level_range_adjust (caster, spell);
   /* On the bright side, no monster should ever have a race of GOD_...
    * so even if the player doesn't worship a god, if race=GOD_.., it
    * won't ever match anything.
  * about.  was called move_ball_lightning, but since more than the ball
  * lightning spell used it, that seemed misnamed.
  * op is the spell effect.
  * note that duration is handled by process_object() in time.c
  */
 void
 move_ball_spell (object *op)
 {
   int i, j, dam_save, dir, mflags;
   sint16 nx, ny, hx, hy;
       return;
     }
   if (!op->duration || !owner->is_on_map ())
     {
-      op->destroy ();
+      op->drop_and_destroy ();
       return;
     }
   op->duration--;
   int basedir = op->direction;
   if (!basedir)
+    {
-    /* spray in all directions! 8) */
+      /* spray in all directions! 8) */
-    basedir = (op->facing += op->state) % 8 + 1;
+      op->facing = (op->facing + op->state) & 7;
+      basedir = op->facing + 1;
+    }
 #if 0
   // this is bogus: it causes wrong places to be checked below
   // (a wall 2 cells away will block the effect...) and
   // doesn't work for SP_BULLET anyhow, so again tests the wrong
 {
   if (!spell->other_arch)
     return 0;
   object *tmp = archetype::get (SWARM_SPELL);
   set_spell_skill (op, caster, spell, tmp);
-  tmp->level      = caster_level (caster, spell);    /* needed later, to get level dep. right. */
+  tmp->level      = casting_level (caster, spell);    /* needed later, to get level dep. right. */
   tmp->spell      = spell->other_arch->instance ();
   tmp->attacktype = tmp->spell->attacktype;
   if (tmp->attacktype & AT_HOLYWORD || tmp->attacktype & AT_GODPOWER)
     if (!tailor_god_spell (tmp, op))
                 object *disease = arch_to_object (spell->other_arch);
                 disease->set_owner (op);
                 set_spell_skill (op, caster, spell, disease);
                 disease->stats.exp = 0;
-                disease->level = caster_level (caster, spell);
+                disease->level = casting_level (caster, spell);
                 /* do level adjustments */
                 if (disease->stats.wc)
                   disease->stats.wc += dur_mod / 2;

Diff Legend

-–
+Removed lines
-+
+Added lines
-<
+Changed lines
->
+Changed lines

Comparing deliantra/server/server/spell_attack.C (file contents): Revision 1.59 by root, Sat May 17 15:25:19 2008 UTC vs. Revision 1.68 by elmex, Mon Oct 6 08:46:33 2008 UTC

Diff Legend

Comparing deliantra/server/server/spell_attack.C (file contents):
Revision 1.59 by root, Sat May 17 15:25:19 2008 UTC vs.
Revision 1.68 by elmex, Mon Oct 6 08:46:33 2008 UTC