[ViewVC] Diff of: cvs/deliantra/server/common/living.C

Comparing deliantra/server/common/living.C (file contents):
Revision 1.74 by root, Tue Aug 7 21:58:25 2007 UTC vs.
Revision 1.91 by root, Thu Dec 4 01:07:34 2008 UTC

 /*
- * This file is part of Crossfire TRT, the Roguelike Realtime MORPG.
+ * This file is part of Deliantra, the Roguelike Realtime MMORPG.
  *
- * Copyright (©) 2005,2006,2007 Marc Alexander Lehmann / Robin Redeker / the Crossfire TRT team
+ * Copyright (©) 2005,2006,2007,2008 Marc Alexander Lehmann / Robin Redeker / the Deliantra team
  * Copyright (©) 2002,2007 Mark Wedel & Crossfire Development Team
  * Copyright (©) 1992,2007 Frank Tore Johansen
  *
- * Crossfire TRT is free software: you can redistribute it and/or modify
+ * 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.
  *
  * This program is distributed in the hope that it will be useful,
  * 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/>.
  *
- * The authors can be reached via e-mail to <crossfire@schmorp.de>
+ * The authors can be reached via e-mail to <support@deliantra.net>
  */
 #include <global.h>
-#include <funcpoint.h>
 /* Handy little macro that adds exp and keeps it within bounds.  Since
  * we are now using 64 bit values, I'm not all concerned about overflow issues
  * with exptotal wrapping.  exptotal is typically op->exp, or op->perm_exp
  */
  * that gives them that ability.
  */
 int
 change_abil (object *op, object *tmp)
 {
-  int flag = QUERY_FLAG (tmp, FLAG_APPLIED) ? 1 : -1, success = 0;
+  int flag = tmp->flag [FLAG_APPLIED] ? 1 : -1;
+  int success = 0;
   char message[MAX_BUF];
   int potion_max = 0;
-  /* remember what object was like before it was changed.  note that
+  // keep some stats for comparison purposes
-   * refop is a local copy of op only to be used for detecting changes
+  object::flags_t prev_flag = op->flag;
-   * found by update_stats. refop is not a real object
+  MoveType prev_move_type = op->move_type;
-   */
+  sint16 prev_resist [NROFATTACKS]; // clumsy
-  object_copy refop = *op;
+  assert (sizeof (prev_resist) == sizeof (op->resist));
+  memcpy (prev_resist, op->resist, sizeof (prev_resist));
   if (op->type == PLAYER)
     {
       if (tmp->type == POTION)
         {
           potion_max = 1;
           for (int j = 0; j < NUM_STATS; j++)
             {
               int ostat = op->contr->orig_stats.stat (j);
               int i = tmp->stats.stat (j);
               /* nstat is what the stat will be after use of the potion */
               int nstat = flag * i + ostat;
-              /* Do some bounds checking.  While I don't think any
+              /* Do some bounds checking. There is the potential for potions
-               * potions do so right now, there is the potential for potions
                * that adjust that stat by more than one point, so we need
                * to allow for that.
                */
               if (nstat < 1 && i * flag < 0)
                 nstat = 1;
     {
       success = 1;
       DIFF_MSG (flag, "Your hands begin to glow red.", "Your hands stop glowing red.");
     }
-  if (QUERY_FLAG (op, FLAG_LIFESAVE) != QUERY_FLAG (&refop, FLAG_LIFESAVE))
+  if (op->flag [FLAG_LIFESAVE] != prev_flag [FLAG_LIFESAVE])
     {
       success = 1;
       DIFF_MSG (flag, "You feel very protected.", "You don't feel protected anymore.");
     }
-  if (QUERY_FLAG (op, FLAG_REFL_MISSILE) != QUERY_FLAG (&refop, FLAG_REFL_MISSILE))
+  if (op->flag [FLAG_REFL_MISSILE] != prev_flag [FLAG_REFL_MISSILE])
     {
       success = 1;
       DIFF_MSG (flag, "A magic force shimmers around you.", "The magic force fades away.");
     }
-  if (QUERY_FLAG (op, FLAG_REFL_SPELL) != QUERY_FLAG (&refop, FLAG_REFL_SPELL))
+  if (op->flag [FLAG_REFL_SPELL] != prev_flag [FLAG_REFL_SPELL])
     {
       success = 1;
       DIFF_MSG (flag, "You feel more safe now, somehow.", "Suddenly you feel less safe, somehow.");
     }
    * user has multiple items giving the same type appled like we
    * used to - that is more work than what we gain, plus messages
    * can be misleading (a little higher could be miscontrued from
    * from fly high)
    */
-  if (tmp->move_type && op->move_type != refop.move_type)
+  if (tmp->move_type && op->move_type != prev_move_type)
     {
       success = 1;
       /* MOVE_FLY_HIGH trumps MOVE_FLY_LOW - changing your move_fly_low
        * status doesn't make a difference if you are flying high
        */
       if (tmp->move_type & MOVE_FLY_LOW && !(op->move_type & MOVE_FLY_HIGH))
         {
-          DIFF_MSG (flag, "You start to float in the air!.", "You float down to the ground.");
+          DIFF_MSG (flag, "You start to float in the air!", "You float down to the ground.");
         }
       if (tmp->move_type & MOVE_FLY_HIGH)
         {
           /* double conditional - second case covers if you have move_fly_low -
     }
   /* becoming UNDEAD... a special treatment for this flag. Only those not
    * originally undead may change their status
    */
-  if (!QUERY_FLAG (op->arch, FLAG_UNDEAD))
+  if (!op->arch->flag [FLAG_UNDEAD])
-    if (QUERY_FLAG (op, FLAG_UNDEAD) != QUERY_FLAG (&refop, FLAG_UNDEAD))
+    if (op->flag [FLAG_UNDEAD] != prev_flag [FLAG_UNDEAD])
       {
         success = 1;
         if (flag > 0)
           {
             op->race = "undead";
             op->race = op->arch->race;
             new_draw_info (NDI_UNIQUE, 0, op, "Your lifeforce returns!");
           }
       }
-  if (QUERY_FLAG (op, FLAG_STEALTH) != QUERY_FLAG (&refop, FLAG_STEALTH))
+  if (op->flag [FLAG_STEALTH] != prev_flag [FLAG_STEALTH])
     {
       success = 1;
       DIFF_MSG (flag, "You walk more quietly.", "You walk more noisily.");
     }
-  if (QUERY_FLAG (op, FLAG_MAKE_INVIS) != QUERY_FLAG (&refop, FLAG_MAKE_INVIS))
+  if (op->flag [FLAG_MAKE_INVIS] != prev_flag [FLAG_MAKE_INVIS])
     {
       success = 1;
       DIFF_MSG (flag, "You become transparent.", "You can see yourself.");
     }
   /* blinded you can tell if more blinded since blinded player has minimal
    * vision
    */
-  if (QUERY_FLAG (tmp, FLAG_BLIND))
+  if (tmp->flag [FLAG_BLIND])
     {
       success = 1;
       if (flag > 0)
         {
-          if (QUERY_FLAG (op, FLAG_WIZ))
+          if (op->flag [FLAG_WIZ])
             new_draw_info (NDI_UNIQUE, 0, op, "Your mortal self is blinded.");
           else
             {
               new_draw_info (NDI_UNIQUE, 0, op, "You are blinded.");
               SET_FLAG (op, FLAG_BLIND);
                 op->contr->do_los = 1;
             }
         }
       else
         {
-          if (QUERY_FLAG (op, FLAG_WIZ))
+          if (op->flag [FLAG_WIZ])
             new_draw_info (NDI_UNIQUE, 0, op, "Your mortal self can now see again.");
           else
             {
               new_draw_info (NDI_UNIQUE, 0, op, "Your vision returns.");
               CLEAR_FLAG (op, FLAG_BLIND);
                 op->contr->do_los = 1;
             }
         }
     }
-  if (QUERY_FLAG (op, FLAG_SEE_IN_DARK) != QUERY_FLAG (&refop, FLAG_SEE_IN_DARK))
+  if (op->flag [FLAG_SEE_IN_DARK] != prev_flag [FLAG_SEE_IN_DARK])
     {
       success = 1;
       if (op->type == PLAYER)
         op->contr->do_los = 1;
       DIFF_MSG (flag, "Your vision is better in the dark.", "You see less well in the dark.");
     }
-  if (QUERY_FLAG (op, FLAG_XRAYS) != QUERY_FLAG (&refop, FLAG_XRAYS))
+  if (op->flag [FLAG_XRAYS] != prev_flag [FLAG_XRAYS])
     {
       success = 1;
       if (flag > 0)
         {
-          if (QUERY_FLAG (op, FLAG_WIZ))
+          if (op->flag [FLAG_WIZ])
             new_draw_info (NDI_UNIQUE, 0, op, "Your vision becomes a little clearer.");
           else
             {
               new_draw_info (NDI_UNIQUE, 0, op, "Everything becomes transparent.");
               if (op->type == PLAYER)
                 op->contr->do_los = 1;
             }
         }
       else
         {
-          if (QUERY_FLAG (op, FLAG_WIZ))
+          if (op->flag [FLAG_WIZ])
             new_draw_info (NDI_UNIQUE, 0, op, "Your vision becomes a bit out of focus.");
           else
             {
               new_draw_info (NDI_UNIQUE, 0, op, "Everything suddenly looks very solid.");
               if (op->type == PLAYER)
   for (int i = 0; i < NROFATTACKS; i++)
     {
       if (i == ATNR_PHYSICAL)
         continue;               /* Don't display about armour */
-      if (op->resist[i] != refop.resist[i])
+      if (op->resist [i] != prev_resist [i])
         {
           success = 1;
-          if (op->resist[i] > refop.resist[i])
+          if (op->resist [i] > prev_resist [i])
-            sprintf (message, "Your resistance to %s rises to %d%%.", change_resist_msg[i], op->resist[i]);
+            sprintf (message, "Your resistance to %s rises to %d%%.", change_resist_msg [i], op->resist [i]);
           else
-            sprintf (message, "Your resistance to %s drops to %d%%.", change_resist_msg[i], op->resist[i]);
+            sprintf (message, "Your resistance to %s drops to %d%%.", change_resist_msg [i], op->resist [i]);
           new_draw_info (NDI_UNIQUE | NDI_BLUE, 0, op, message);
         }
     }
   if (!potion_max)
-    {
-      for (int j = 0; j < NUM_STATS; j++)
+    for (int j = 0; j < NUM_STATS; j++)
-        {
-          if (int i = tmp->stats.stat (j))
+      if (int i = tmp->stats.stat (j))
-            {
+        {
-              success = 1;
+          success = 1;
-              DIFF_MSG (i * flag, gain_msg[j], lose_msg[j]);
+          DIFF_MSG (i * flag, gain_msg[j], lose_msg[j]);
-            }
+        }
-        }
-    }
   return success;
 }
 /*
  * spell system split, grace points now added to system  --peterm
  */
 void
 object::update_stats ()
 {
-  int i, j;
   float f, max_speed = 9, added_speed = 0, bonus_speed = 0, speed_reduce_from_disease = 1;
   int weapon_weight = 0, weapon_speed = 0;
   int best_wc = 0, best_ac = 0, wc = 0, ac = 0;
   int prot[NROFATTACKS], vuln[NROFATTACKS], potion_resist[NROFATTACKS];
   object *grace_obj = NULL, *mana_obj = NULL, *tmp;
   float old_speed = speed;
+  int stat_sum [NUM_STATS];
   /* First task is to clear all the values back to their original values */
   if (type == PLAYER)
     {
-      for (i = 0; i < NUM_STATS; i++)
+      for (int i = 0; i < NUM_STATS; i++)
-        stats.stat (i) = contr->orig_stats.stat (i);
+        stat_sum [i] = contr->orig_stats.stat (i);
       if (settings.spell_encumbrance == TRUE)
         contr->encumbrance = 0;
       attacktype = 0;
   /* initializing resistances from the values in player/monster's
    * archetype clone
    */
   memcpy (&resist, &arch->resist, sizeof (resist));
-  for (i = 0; i < NROFATTACKS; i++)
+  for (int i = 0; i < NROFATTACKS; i++)
     {
       if (resist[i] > 0)
         prot[i] = resist[i], vuln[i] = 0;
       else
         vuln[i] = -(resist[i]), prot[i] = 0;
         continue;
       /* See note in map.c:update_position about making this additive
        * since light sources are never applied, need to put check here.
        */
-      if (tmp->glow_radius > glow_radius)
-        glow_radius = tmp->glow_radius;
+      glow_radius = max (glow_radius, tmp->glow_radius);
       /* For some things, we don't care what is equipped */
       if (tmp->type == SKILL)
         {
           /* Want to take the highest skill here. */
               else if (tmp->level > grace_obj->level)
                 grace_obj = tmp;
             }
         }
-      /* Container objects are not meant to adjust a players, but other applied
+      /* Container objects are not meant to adjust players, but other applied
        * objects need to make adjustments.
        * This block should handle all player specific changes
        * The check for Praying is a bit of a hack - god given bonuses are put
        * in the praying skill, and the player should always get those.
        * It also means we need to put in additional checks for applied below,
                     && !tmp->flag [FLAG_CURSED]
                     && !tmp->flag [FLAG_DAMNED])
                   continue;
               for (int i = 0; i < NUM_STATS; i++)
-                if (expect_false (tmp->stats.stat (i)))
+                stat_sum [i] = stat_sum [i] + tmp->stats.stat (i);
-                  stats.stat (i) = clamp (stats.stat (i) + tmp->stats.stat (i), MIN_STAT, MAX_STAT);
               if (digest_types [tmp->type])
                 {
-                  contr->digestion     = clamp (int (contr->digestion) + tmp->stats.food, MIN_DIGESTION, MAX_DIGESTION);
+                  contr->digestion     += tmp->stats.food;
                   contr->gen_hp        += tmp->stats.hp;
+                  if (tmp->type != BOW) // ugly exception for bows
-                  contr->gen_sp        += tmp->stats.sp;
+                    contr->gen_sp      += tmp->stats.sp;
                   contr->gen_grace     += tmp->stats.grace;
                   contr->gen_sp_armour += tmp->gen_sp_armour;
                 }
             }                   /* if this is a player */
           else
                 current_weapon = tmp;
             }
           /* Update slots used for items */
           if (QUERY_FLAG (tmp, FLAG_APPLIED))
-            for (i = 0; i < NUM_BODY_LOCATIONS; i++)
+            for (int i = 0; i < NUM_BODY_LOCATIONS; i++)
               slot[i].used += tmp->slot[i].info;
           if (tmp->type == SYMPTOM)
-            {
-              speed_reduce_from_disease = tmp->last_sp / 100.f;
-              if (speed_reduce_from_disease == 0)
-                speed_reduce_from_disease = 1;
+            speed_reduce_from_disease =
-            }
+              min (speed_reduce_from_disease, tmp->last_sp ? tmp->last_sp / 100.f : 1.f);
-          /* Pos. and neg. protections are counted seperate (-> pro/vuln).
+          /* Pos. and neg. protections are counted separate (-> pro/vuln).
            * (Negative protections are calculated exactly like positive.)
            * Resistance from potions are treated special as well. If there's
            * more than one potion-effect, the bigger prot.-value is taken.
            */
           if (tmp->type != POTION)
             {
-              for (i = 0; i < NROFATTACKS; i++)
+              for (int i = 0; i < NROFATTACKS; i++)
                 {
                   /* Potential for cursed potions, in which case we just can use
                    * a straight MAX, as potion_resist is initialised to zero.
                    */
                   if (tmp->type == POTION_EFFECT)
                 break;
             }                   /* switch tmp->type */
         }                       /* item is equipped */
     }                           /* for loop of items */
+  fprintf (stderr, "gr %s %d\n", &name, glow_radius);//D
   /* We've gone through all the objects the player has equipped.  For many things, we
    * have generated intermediate values which we now need to assign.
    */
    * If there is an uncursed potion in effect, granting more protection
    * than that, we take: 'total resistance = resistance from potion'.
    * If there is a cursed (and no uncursed) potion in effect, we take
    * 'total resistance = vulnerability from cursed potion'.
    */
-  for (i = 0; i < NROFATTACKS; i++)
+  for (int i = 0; i < NROFATTACKS; i++)
     {
       resist[i] = prot[i] - vuln[i];
       if (potion_resist[i] && ((potion_resist[i] > resist[i]) || (potion_resist[i] < 0)))
         resist[i] = potion_resist[i];
     }
-  /* Figure out the players sp/mana/hp totals. */
   if (type == PLAYER)
     {
+      // clamp various player stats
+      for (int i = 0; i < NUM_STATS; ++i)
+        stats.stat (i) = clamp (stat_sum [i], MIN_STAT, MAX_STAT);
+      contr->digestion = clamp (contr->digestion, MIN_DIGESTION, MAX_DIGESTION);
+      /* Figure out the players sp/mana/hp totals. */
       int pl_level;
       check_stat_bounds (&(stats));
       pl_level = level;
         pl_level = 1;           /* safety, we should always get 1 levels worth of hp! */
       /* You basically get half a con bonus/level.  But we do take into account rounding,
        * so if your bonus is 7, you still get 7 worth of bonus every 2 levels.
        */
-      for (i = 1, stats.maxhp = 0; i <= pl_level && i <= 10; i++)
+      stats.maxhp = 0;
+      for (int i = 1; i <= min (10, pl_level); i++)
         {
-          j = contr->levhp[i] + con_bonus[stats.Con] / 2;
+          int j = contr->levhp[i] + con_bonus[stats.Con] / 2;
           if (i % 2 && con_bonus[stats.Con] % 2)
             if (con_bonus[stats.Con] > 0)
               j++;
             else
               j--;
           stats.maxhp += j > 1 ? j : 1;     /* always get at least 1 hp/level */
         }
-      for (i = 11; i <= level; i++)
+      stats.maxhp += 2 * max (0, level - 10);
-        stats.maxhp += 2;
       if (stats.hp > stats.maxhp)
         stats.hp = stats.maxhp;
       /* Sp gain is controlled by the level of the player's
         stats.maxsp = 1;
       else
         {
           float sp_tmp = 0.f;
-          for (i = 1; i <= min (10, mana_obj->level); i++)
+          for (int i = 1; i <= min (10, mana_obj->level); i++)
             {
               float stmp;
               /* Got some extra bonus at first level */
               if (i < 2)
            * becomes big jumps when the sums of the bonuses jump to the next
            * step of 8 - with floats, even fractional ones are useful.
            */
           float sp_tmp = 0.f;
-          for (i = 1; i <= min (10, grace_obj->level); i++)
+          for (int i = 1; i <= min (10, grace_obj->level); i++)
             {
               float grace_tmp = 0.f;
               /* Got some extra bonus at first level */
               if (i < 2)
       if (contr && wc_obj && wc_obj->level > 1)
         {
           wc -= wc_obj->level + thaco_bonus[stats.Str];
-          for (i = 1; i < wc_obj->level; i++)
+          for (int i = 1; i < wc_obj->level; i++)
             {
               /* additional wc every 6 levels */
               if (!(i % 6))
                 wc--;
       /* f is a number the represents the number of kg above (positive num)
        * or below (negative number) that the player is carrying.  If above
        * weight limit, then player suffers a speed reduction based on how
        * much above he is, and what is max carry is
        */
-      f = (carrying / 1000) - max_carry[stats.Str];
+      float f = (carrying / 1000) - max_carry[stats.Str];
-      if (f > 0)
+      if (f > 0.f)
         speed = speed / (1.f + f / max_carry[stats.Str]);
     }
   speed += bonus_speed / 10.f;      /* Not affected by limits */
+  speed *= speed_reduce_from_disease;
   /* Put a lower limit on speed.  Note with this speed, you move once every
-   * 100 ticks or so.  This amounts to once every 12 seconds of realtime.
+   * 25 ticks or so. This amounts to once every 3 seconds of realtime.
    */
-  speed = speed * speed_reduce_from_disease;
-  if (speed < 0.01f && type == PLAYER)
+  if (speed < 0.04f && type == PLAYER)
-    speed = 0.01f;
+    speed = 0.04f;
+  if (speed != old_speed)
+    set_speed (speed);
   if (type == PLAYER)
     {
       /* (This formula was made by vidarl@ifi.uio.no)
        * Note that we never used these values again - basically
   if (move_type == 0)
     move_type = MOVE_WALK;
   else if (move_type & (MOVE_FLY_LOW | MOVE_FLY_HIGH))
     move_type &= ~MOVE_WALK;
-  if (speed != old_speed)
-    set_speed (speed);
   /* It is quite possible that a player's spell costing might have changed,
    * so we will check that now.
    */
   if (type == PLAYER)
     {
 /*
  * set the new dragon name after gaining levels or
  * changing ability focus (later this can be extended to
  * eventually change the player's face and animation)
- *
- * Note that the title is written to 'own_title' in the
- * player struct. This should be changed to 'ext_title'
- * as soon as clients support this!
- * Please, anyone, write support for 'ext_title'.
  */
 void
 set_dragon_name (object *pl, const object *abil, const object *skin)
 {
   int atnr = -1;                /* attacknumber of highest level */
   /* now if there are equals at highest level, pick the one with focus,
      or else at random */
   if (atnr_is_dragon_enabled (abil->stats.exp) && abil->resist[abil->stats.exp] >= level)
     atnr = abil->stats.exp;
-  level = (int) (level / 5.);
   /* now set the new title */
-  if (pl->contr != NULL)
-    {
-      if (level == 0)
-        sprintf (pl->contr->title, "%s hatchling", attacks[atnr]);
+  if      (level <  25) sprintf (pl->contr->title, "%s hatchling", attacks[atnr]);
-      else if (level == 1)
-        sprintf (pl->contr->title, "%s wyrm", attacks[atnr]);
+  else if (level <  50) sprintf (pl->contr->title, "%s wyrm"     , attacks[atnr]);
-      else if (level == 2)
-        sprintf (pl->contr->title, "%s wyvern", attacks[atnr]);
+  else if (level <  75) sprintf (pl->contr->title, "%s wyvern"   , attacks[atnr]);
-      else if (level == 3)
-        sprintf (pl->contr->title, "%s dragon", attacks[atnr]);
+  else if (level < 100) sprintf (pl->contr->title, "%s dragon"   , attacks[atnr]);
-      else
+  else
-        {
+    {
-          /* special titles for extra high resistance! */
+      /* special titles for extra high resistance! */
-          if (skin->resist[atnr] > 80)
-            sprintf (pl->contr->title, "legendary %s dragon", attacks[atnr]);
+      if      (skin->resist[atnr] > 80) sprintf (pl->contr->title, "legendary %s dragon", attacks[atnr]);
-          else if (skin->resist[atnr] > 50)
-            sprintf (pl->contr->title, "ancient %s dragon", attacks[atnr]);
+      else if (skin->resist[atnr] > 50) sprintf (pl->contr->title, "ancient %s dragon"  , attacks[atnr]);
-          else
-            sprintf (pl->contr->title, "big %s dragon", attacks[atnr]);
+      else                              sprintf (pl->contr->title, "big %s dragon"      , attacks[atnr]);
-        }
     }
   strcpy (pl->contr->own_title, "");
 }
   object *skill_obj;
   skill_obj = get_archetype_by_skill_name (skill_name, SKILL);
   if (!skill_obj)
     {
-      LOG (llevError, "add_player_exp: couldn't find skill %s\n", skill_name);
+      LOG (llevError, "give_skill_by_name: couldn't find skill %s\n", skill_name);
       return NULL;
     }
   /* clear the flag - exp goes into this bucket, but player
    * still doesn't know it.
    */
   CLEAR_FLAG (skill_obj, FLAG_CAN_USE_SKILL);
   skill_obj->stats.exp = 0;
   skill_obj->level = 1;
-  insert_ob_in_ob (skill_obj, op);
+  op->insert (skill_obj);
   if (player *pl = op->contr)
-    {
+    pl->link_skills ();
-      pl->last_skill_ob [skill_obj->subtype] = skill_obj;
-      if (pl->ns)
-        pl->ns->last_skill_exp[skill_obj->subtype] = -1;//TODO: should be made superfluous
-    }
   return skill_obj;
 }
 /* player_lvl_adj() - for the new exp system. we are concerned with
  * flag is what to do if the player doesn't have the skill:
  */
 static void
 add_player_exp (object *op, sint64 exp, const char *skill_name, int flag)
 {
-  object *skill_obj = NULL;
+  object *skill_obj;
   sint64 limit, exp_to_add;
   int i;
   /* prevents some forms of abuse. */
   if (op->contr->braced)
   /* Try to find the matching skill.
    * We do a shortcut/time saving mechanism first - see if it matches
    * chosen_skill. This means we don't need to search through
    * the players inventory.
    */
+  skill_obj = 0;
   if (skill_name)
     {
-      if (op->chosen_skill && op->chosen_skill->type == SKILL && !strcmp (skill_name, op->chosen_skill->skill))
+      skill_obj = op->contr->find_skill (skill_name);
-        skill_obj = op->chosen_skill;
-      else
-        {
-          for (i = 0; i < NUM_SKILLS; i++)
-            if (op->contr->last_skill_ob[i] && !strcmp (op->contr->last_skill_ob[i]->skill, skill_name))
-              {
-                skill_obj = op->contr->last_skill_ob[i];
-                break;
-              }
-          /* Player doesn't have the skill.  Check to see what to do, and give
+      /* Player doesn't have the skill.  Check to see what to do, and give
-           * it to the player if necessary
+       * it to the player if necessary
-           */
+       */
-          if (!skill_obj)
+      if (!skill_obj)
-            {
+        {
-              if (flag == SK_EXP_NONE)
+          if (flag == SK_EXP_NONE)
-                return;
+            return;
-              else if (flag == SK_EXP_ADD_SKILL)
+          if (flag == SK_EXP_ADD_SKILL)
-                give_skill_by_name (op, skill_name);
+            skill_obj = give_skill_by_name (op, skill_name);
-            }
         }
     }
   if (flag != SK_EXP_SKILL_ONLY)
     {
   if (exp < 0)
     return check_exp_loss (op, exp);
   else
     return MIN (exp, (sint64) MAX_EXPERIENCE - op->stats.exp);
 }
 /* Subtracts experience from player.
  * if skill is set and flag == SK_SUBTRACT_SKILL_EXP, then we
  * only subtract from the matching skill.  Otherwise,
  * this subtracts a portion from all
   sint64 del_exp;
   for (tmp = op->inv; tmp; tmp = tmp->below)
     if (tmp->type == SKILL && tmp->stats.exp)
       {
-        if (flag == SK_SUBTRACT_SKILL_EXP && skill && !strcmp (tmp->skill, skill))
+        if (flag == SK_SUBTRACT_SKILL_EXP && skill && !strcmp (&tmp->skill, skill))
           {
             del_exp = check_exp_loss (tmp, exp);
             tmp->stats.exp -= del_exp;
             player_lvl_adj (op, tmp);
           }

Diff Legend

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

Comparing deliantra/server/common/living.C (file contents): Revision 1.74 by root, Tue Aug 7 21:58:25 2007 UTC vs. Revision 1.91 by root, Thu Dec 4 01:07:34 2008 UTC

Diff Legend

Comparing deliantra/server/common/living.C (file contents):
Revision 1.74 by root, Tue Aug 7 21:58:25 2007 UTC vs.
Revision 1.91 by root, Thu Dec 4 01:07:34 2008 UTC