--- deliantra/server/server/spell_attack.C	2009/01/01 15:43:35	1.78
+++ deliantra/server/server/spell_attack.C	2009/10/12 14:00:59	1.88
@@ -1,22 +1,23 @@
 /*
  * This file is part of Deliantra, the Roguelike Realtime MMORPG.
  * 
- * Copyright (©) 2005,2006,2007,2008 Marc Alexander Lehmann / Robin Redeker / the Deliantra team
+ * Copyright (©) 2005,2006,2007,2008,2009 Marc Alexander Lehmann / Robin Redeker / the Deliantra team
  * Copyright (©) 2002-2003,2007 Mark Wedel & Crossfire Development Team
  * Copyright (©) 1992,2007 Frank Tore Johansen
  * 
- * Deliantra is free software: you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
+ * Deliantra is free software: you can redistribute it and/or modify it under
+ * the terms of the Affero GNU General Public License as published by the
+ * Free Software Foundation, either version 3 of the License, or (at your
+ * option) any later version.
  * 
  * This program is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
  * GNU General Public License for more details.
  * 
- * You should have received a copy of the GNU General Public License
- * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ * You should have received a copy of the Affero GNU General Public License
+ * and the GNU General Public License along with this program. If not, see
+ * <http://www.gnu.org/licenses/>.
  * 
  * The authors can be reached via e-mail to <support@deliantra.net>
  */
@@ -547,10 +548,6 @@
 void
 move_bullet (object *op)
 {
-  sint16 new_x, new_y;
-  int mflags;
-  maptile *m;
-
 #if 0
   /* We need a better general purpose way to do this */
 
@@ -575,18 +572,20 @@
       return;
     }
 
-  new_x = op->x + DIRX (op);
-  new_y = op->y + DIRY (op);
-  m = op->map;
-  mflags = get_map_flags (m, &m, new_x, new_y, &new_x, &new_y);
+  mapxy pos (op);
+  pos.move (op->direction);
 
-  if (mflags & P_OUT_OF_MAP)
+  if (!pos.normalise ())
     {
       op->destroy ();
       return;
     }
 
-  if (!op->direction || OB_TYPE_MOVE_BLOCK (op, GET_MAP_MOVE_BLOCK (m, new_x, new_y)))
+  mapspace &ms = pos.ms ();
+
+  ms.update ();
+
+  if (!op->direction || OB_TYPE_MOVE_BLOCK (op, ms.move_block))
     {
       if (op->other_arch)
         explode_bullet (op);
@@ -596,7 +595,7 @@
       return;
     }
 
-  if (!(op = m->insert (op, new_x, new_y, op)))
+  if (!(op = pos.insert (op, op)))
     return;
 
   if (reflwall (op->map, op->x, op->y, op))
@@ -830,14 +829,14 @@
 
   for (i = range_min; i <= range_max; i++)
     {
-      sint16 x, y, d;
+      sint16 x, y;
 
       /* We can't use absdir here, because it never returns
        * 0. If this is a rune, we want to hit the person on top
        * of the trap (d==0).  If it is not a rune, then we don't want
        * to hit that person.
        */
-      d = (dir + i) % 9;
+      int d = dir ? absdir (dir + i) : i;
 
       /* If it's not a rune, we don't want to blast the caster.
        * In that case, we have to see - if dir is specified,
@@ -949,7 +948,7 @@
 
   if (op->env)
     {
-      if (env->map == NULL)
+      if (!env->map)
         return;
 
       if (!(op = op->insert_at (env, op)))
@@ -959,7 +958,7 @@
   // elmex Tue Aug 15 17:46:51 CEST 2006: Prevent bomb from exploding
   // on a safe map. I don't like this special casing, but it seems to be neccessary
   // as bombs can be carried.
-  if (get_map_flags (op->map, NULL, op->x, op->y, NULL, NULL) & P_SAFE)
+  if (op->ms ().flags () & P_SAFE)
     {
       op->destroy ();
       return;
@@ -1116,8 +1115,8 @@
   if (!target
       || target->flag [FLAG_REFL_SPELL]
       || (!god && spell->stats.grace)
-      || (target->title && god && target->title == god->name)
-      || (target->race  && god && target->race.contains (god->race)))
+      || (god && target->title == god->name)
+      || (god && target->race.contains (god->race)))
     {
       new_draw_info (NDI_UNIQUE, 0, op, "Your request is unheeded.");
       return 0;
@@ -1258,11 +1257,11 @@
   tmp->speed = 0.01;
   tmp->stats.food = time;
   SET_FLAG (tmp, FLAG_IS_USED_UP);
-  tmp->glow_radius = min (MAX_LIGHT_RADIUS, radius);
+  tmp->set_glow_radius (min (MAX_LIGHT_RADIUS, radius));
   tmp = insert_ob_in_ob (tmp, op);
 
   if (tmp->glow_radius > op->glow_radius)
-    op->glow_radius = tmp->glow_radius;
+    op->set_glow_radius (tmp->glow_radius);
 
   return 1;
 }
@@ -1449,9 +1448,9 @@
    */
   if (!spell->race)
     race = NULL;
-  else if (god && !strcmp (spell->race, "GOD_SLAYING"))
+  else if (god && spell->race == shstr_GOD_SLAYING)
     race = god->slaying;
-  else if (god && !strcmp (spell->race, "GOD_FRIEND"))
+  else if (god && spell->race == shstr_GOD_FRIEND)
     race = god->race;
   else
     race = spell->race;
@@ -1909,12 +1908,14 @@
   tmp->stats.food = spell->duration + SP_level_duration_adjust (caster, spell);
 
   if (tmp->glow_radius)
-    tmp->glow_radius = min (MAX_LIGHT_RADIUS, spell->range + SP_level_range_adjust (caster, spell));
+    tmp->set_glow_radius (
+      clamp (spell->range + SP_level_range_adjust (caster, spell), 1, MAX_LIGHT_RADIUS)
+    );
 
   if (dir)
     m->insert (tmp, x, y, op);
   else
-    caster->outer_env ()->insert (tmp);
+    caster->outer_env_or_self ()->insert (tmp);
 
   return 1;
 }