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

Comparing deliantra/server/server/player.C (file contents):
Revision 1.123 by root, Fri May 11 21:07:14 2007 UTC vs.
Revision 1.133 by root, Sat May 12 22:04:20 2007 UTC

 player::set_object (object *op)
 {
   ob = op;
   ob->contr = this;              /* this aren't yet in archetype */
-  ob->speed_left = 0.5;
+  ob->speed_left = 0.5f;
-  ob->speed      = 1.0;
+  ob->speed      = 1.0f;
   ob->direction  = 5;            /* So player faces south */
 }
 player::player ()
 {
 }
 void
 object::roll_stats ()
 {
-  int statsort [7];
+  int statsort [NUM_STATS];
   for (;;)
     {
       int sum = 0;
-      for (int i = 7; i--; )
+      for (int i = NUM_STATS; i--; )
         sum += statsort [i] = roll_stat ();
       if (sum >= 82 && sum <= 116)
         break;
     }
   // Sort the stats so that rerolling is easier...
-  std::sort (statsort, statsort + 7, std::greater<int>());
+  std::sort (statsort, statsort + NUM_STATS, std::greater<int>());
+  for (int i = 0; i < NUM_STATS; ++i)
-  stats.Str = statsort[0];
+    stats.stat (i) = statsort [i];
-  stats.Dex = statsort[1];
-  stats.Con = statsort[2];
-  stats.Int = statsort[3];
-  stats.Wis = statsort[4];
-  stats.Pow = statsort[5];
-  stats.Cha = statsort[6];
   stats.exp = 0;
   stats.ac  = 0;
   stats.hp    = stats.maxhp;
 }
 void
 object::swap_stats (int a, int b)
 {
-  int tmp = get_attr_value (&contr->orig_stats, a);
+  swap (contr->orig_stats.stat (a), contr->orig_stats.stat (b));
-  set_attr_value (&contr->orig_stats, a, get_attr_value (&contr->orig_stats, b));
-  set_attr_value (&contr->orig_stats, b, tmp);
+  for (int i = 0; i < NUM_STATS; ++i)
-  stats.Str = contr->orig_stats.Str;
+    stats.stat (i) = contr->orig_stats.stat (i);
-  stats.Dex = contr->orig_stats.Dex;
-  stats.Con = contr->orig_stats.Con;
-  stats.Int = contr->orig_stats.Int;
-  stats.Wis = contr->orig_stats.Wis;
-  stats.Pow = contr->orig_stats.Pow;
-  stats.Cha = contr->orig_stats.Cha;
   //TODO: the following code looks so borked and should, at the very least,
   // be merged with the similar code in roll_stats
   stats.ac  = 0;
  */
 int
 fire_bow (object *op, object *part, object *arrow, int dir, int wc_mod, sint16 sx, sint16 sy)
 {
   object *left, *bow;
-  int bowspeed, mflags;
+  int mflags;
   maptile *m;
   if (!dir)
     {
       new_draw_info (NDI_UNIQUE, 0, op, "You can't shoot yourself!");
     }
   if (player *pl = op->contr)
     {
       bow = pl->ranged_ob;
-      op->set_weapon (bow);
+      if (!op->change_weapon (bow))
+        return 0;
     }
   else
     {
       for (bow = op->inv; bow; bow = bow->below)
         /* Don't check for applied - monsters don't apply bows - in that way, they
     {
       new_draw_info_format (NDI_UNIQUE, 0, op, "Your %s is broken.", &bow->name);
       return 0;
     }
-  bowspeed = bow->stats.sp + dex_bonus[op->stats.Dex];
-  /* penalize ROF for bestarrow */
-  if (op->type == PLAYER && op->contr->bowtype == bow_bestarrow)
-    bowspeed -= dex_bonus[op->stats.Dex] + 5;
-  if (bowspeed < 1)
-    bowspeed = 1;
   if (arrow == NULL)
     {
       if ((arrow = find_arrow (op, bow->race)) == NULL)
         {
           if (op->type == PLAYER)
   arrow->set_owner (op);
   arrow->skill = bow->skill;
   arrow->direction = dir;
-  if (op->type == PLAYER)
-    {
-      op->speed_left = 0.01 - (float) FABS (op->speed) * 100 / bowspeed;
-      op->update_stats ();
-    }
-  SET_ANIMATION (arrow, arrow->direction);
-  arrow->stats.sp    = arrow->stats.wc;    /* save original wc and dam */
+  arrow->stats.sp    = arrow->stats.wc;    /* save original wc, dam, attacktype and slaying */
   arrow->stats.hp    = arrow->stats.dam;
   arrow->stats.grace = arrow->attacktype;
   if (arrow->slaying)
     arrow->spellarg = strdup (arrow->slaying);
-  arrow->stats.dam += op->stats.dam + arrow->magic;
+  if (player *pl = op->contr)
+    {
+      pl->has_hit = 1;
+#if 0
+          float speed = pl->weapon_sp;
+          /* penalize ROF for bestarrow */
+          if (pl->bowtype == bow_bestarrow)
+            speed *= .9f;
+          else
+            speed *= 1.f + dex_bonus[op->stats.Dex] * .2f;
+          op->speed_left += speed - op->speed;
+#endif
+    }
+  SET_ANIMATION (arrow, arrow->direction);
   /* update the speed */
-  arrow->speed = (float) ((QUERY_FLAG (bow, FLAG_NO_STRENGTH) ?
+  arrow->speed = ((bow->flag [FLAG_NO_STRENGTH] ? 0 : dam_bonus[op->stats.Str]) + bow->magic + arrow->magic) / 5.0
-                           0 : dam_bonus[op->stats.Str]) + bow->magic + arrow->magic) / 5.0 + (float) bow->stats.dam / 7.0;
+                 + bow->stats.dam / 7.0;
-  arrow->set_speed (max (arrow->speed, 1.0));
+  arrow->set_speed (max (arrow->speed, 2.0));
   arrow->speed_left = 0;
-  arrow->stats.wc = op->stats.wc + wc_mod - arrow->magic - arrow->stats.wc;
+  int wc = op->stats.wc + wc_mod - arrow->magic - arrow->stats.wc;
   if (op->type == PLAYER)
     {
       arrow->level = op->chosen_skill ? op->chosen_skill->level : op->level;
-      arrow->stats.wc -= dex_bonus[op->stats.Dex];
+      wc -= dex_bonus[op->stats.Dex];
       if (!arrow->slaying)
         arrow->slaying = op->slaying;
+      arrow->attacktype |= op->attacktype;
     }
   else
     {
       arrow->level = op->level;
       arrow->stats.wc -= bow->magic;
       if (!arrow->slaying)
         arrow->slaying = bow->slaying;
-    }
-  arrow->stats.wc -= arrow->level;
-  arrow->attacktype |= bow->attacktype;
+      arrow->attacktype |= bow->attacktype;
+    }
+  wc -= arrow->level;
+  arrow->stats.dam = clamp (arrow->stats.dam + op->stats.dam + arrow->magic, MIN_DAM, MAX_DAM);
+  arrow->stats.wc = clamp (wc, MIN_WC, MAX_WC);
   arrow->move_type = MOVE_FLY_LOW;
   arrow->move_on = MOVE_FLY_LOW | MOVE_WALK;
   play_sound_map (op->map, op->x, op->y, SOUND_FIRE_ARROW);
   m->insert (arrow, sx, sy, op);
     {
       LOG (llevError, "Object %s lacks a spell\n", &item->name);
       return;
     }
-  op->set_weapon (item);
+  if (!op->change_weapon (item))
+    return;
   if (item->type == WAND)
     {
       if (item->stats.food <= 0)
         {
        * that party_number -1 is no party, so attacks can still happen.
        */
       else if ((mon->stats.hp >= 0) && QUERY_FLAG (mon, FLAG_ALIVE) &&
                ((mon->type != PLAYER || op->contr->party == NULL || op->contr->party != mon->contr->party)))
         {
-          /* If the player hasn't hit something this tick, and does
-           * so, give them speed boost based on weapon speed.  Doing
-           * it here is better than process_players2, which basically
-           * incurred a 1 tick offset.
-           */
-          if (!op->contr->has_hit)
+          op->contr->has_hit = 1;
-            {
-              op->speed_left += op->speed / op->contr->weapon_sp;
-              op->contr->has_hit = 1;   /* The last action was to hit, so use weapon_sp */
-            }
           skill_attack (mon, op, 0, 0, 0);
           /* If attacking another player, that player gets automatic
            * hitback, and doesn't loose luck either.
 handle_newcs_player (object *op)
 {
   if (QUERY_FLAG (op, FLAG_SCARED))
     {
       flee_player (op);
       /* If player is still scared, that is his action for this tick */
-      if (QUERY_FLAG (op, FLAG_SCARED))
+      if (op->flag [FLAG_SCARED])
         {
-          op->speed_left--;
+          --op->speed_left;
           return 0;
         }
     }
   /* call this here - we also will call this in do_ericserver, but
    * called, so we recheck it here.
    */
   if (op->contr->ns->handle_command ())
     return 1;
-  if (op->speed_left > 0)
+  if (op->speed_left > 0.f)
     {
       if (op->direction && (op->contr->run_on || op->contr->fire_on))
         {
           /* All move commands take 1 tick, at least for now */
-          op->speed_left--;
+          --op->speed_left;
           /* Instead of all the stuff below, let move_player take care
            * of it.  Also, some of the skill stuff is only put in
            * there, as well as the confusion stuff.
            */
           move_player (op, op->direction);
-          return op->speed_left > 0;
+          return op->speed_left > 0.f;
         }
     }
   return 0;
 }
       object *force;
       int at;
       force = get_archetype (FORCE_NAME);
       /* 50 ticks should be enough time for the spell to abate */
-      force->speed = 0.1;
+      force->speed = 0.1f;
-      force->speed_left = -5.0;
+      force->speed_left = -5.f;
       SET_FLAG (force, FLAG_APPLIED);
       for (at = 0; at < NROFATTACKS; at++)
         if (will_kill_again & (1 << at))
           force->resist[at] = 100;

Diff Legend

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

Comparing deliantra/server/server/player.C (file contents): Revision 1.123 by root, Fri May 11 21:07:14 2007 UTC vs. Revision 1.133 by root, Sat May 12 22:04:20 2007 UTC

Diff Legend

Comparing deliantra/server/server/player.C (file contents):
Revision 1.123 by root, Fri May 11 21:07:14 2007 UTC vs.
Revision 1.133 by root, Sat May 12 22:04:20 2007 UTC