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

Comparing deliantra/server/server/c_object.C (file contents):
Revision 1.63 by root, Mon Dec 17 08:35:49 2007 UTC vs.
Revision 1.80 by elmex, Sun Oct 5 12:40:21 2008 UTC

 /*
  * This file is part of Deliantra, the Roguelike Realtime MMORPG.
  *
- * Copyright (©) 2005,2006,2007 Marc Alexander Lehmann / Robin Redeker / the Deliantra 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
  *
  * Deliantra is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
 find_best_object_match (object *pl, const char *params)
 {
   return find_best_apply_object_match (pl, params, AP_TOGGLE);
 }
+static bool
+can_split (object *pl, object *&op, sint32 nrof)
+{
+  if (object *tmp = op->split (nrof ? nrof : op->number_of ()))
+    {
+      op = tmp;
+      return true;
+    }
+  else
+    {
+      if (op->nrof > 1)
+        new_draw_info_format (NDI_UNIQUE, 0, pl, "There are only %d %s.", op->nrof, &op->name_pl);
+      else
+        new_draw_info_format (NDI_UNIQUE, 0, pl, "There is only one %s.", &op->name);
+      return false;
+    }
+}
 int
 command_uskill (object *pl, char *params)
 {
   if (!params)
     {
  * not need to use split_ob and stuff.
  */
 int
 sack_can_hold (object *pl, object *sack, object *op, uint32 nrof)
 {
   if (!QUERY_FLAG (sack, FLAG_APPLIED))
     {
       new_draw_info_format (NDI_UNIQUE, 0, pl, "The %s is not active.", query_name (sack));
       return 0;
     }
   if (sack == op)
     {
       new_draw_info_format (NDI_UNIQUE, 0, pl, "You can't put the %s into itself.", query_name (sack));
       return 0;
     }
   if (sack->race && (sack->race != op->race || op->type == CONTAINER || (sack->stats.food && sack->stats.food != op->type)))
     {
       new_draw_info_format (NDI_UNIQUE, 0, pl, "You can put only %s into the %s.", &sack->race, query_name (sack));
       return 0;
     }
   if (op->type == SPECIAL_KEY && sack->slaying && op->slaying)
     {
       new_draw_info_format (NDI_UNIQUE, 0, pl, "You can't put the key into %s.", query_name (sack));
       return 0;
     }
   if (sack->weight_limit && (sint32) (sack->carrying + (nrof ? nrof : 1) *
                                       (op->weight + (op->type == CONTAINER ? (op->carrying * op->stats.Str) : 0))
                                       * (100 - sack->stats.Str) / 100) > sack->weight_limit)
     {
       new_draw_info_format (NDI_UNIQUE, 0, pl, "That won't fit in the %s!", query_name (sack));
       return 0;
     }
   /* All other checks pass, must be OK */
   return 1;
 }
 /* Pick up commands follow */
  * pick up (0 means all of them)
  */
 static void
 pick_up_object (object *pl, object *op, object *tmp, int nrof)
 {
-  /* buf needs to be big (more than 256 chars) because you can get
-   * very long item names.
-   */
-  char buf[HUGE_BUF];
   object *env = tmp->env;
   uint32 weight, effective_weight_limit;
-  int tmp_nrof = tmp->nrof ? tmp->nrof : 1;
+  int tmp_nrof = tmp->number_of ();
   /* IF the player is flying & trying to take the item out of a container
    * that is in his inventory, let him.  tmp->env points to the container
    * (sack, luggage, etc), tmp->env->env then points to the player (nested
    * containers not allowed as of now)
    */
   if ((pl->move_type & MOVE_FLYING) && !QUERY_FLAG (pl, FLAG_WIZ) && tmp->in_player () != pl)
     {
-      new_draw_info (NDI_UNIQUE, 0, pl, "You are levitating, you can't reach the ground!");
+      pl->failmsg ("You are levitating, you can't reach the ground! "
+                   "H<You have to stop levitating first, if you can, either by using your levitation skill, "
+                   "or waiting till the levitation effect wears off.>");
       return;
     }
   if (QUERY_FLAG (tmp, FLAG_NO_DROP))
     return;
-  if (nrof > tmp_nrof || nrof == 0)
+  if (nrof > tmp_nrof || nrof <= 0)
     nrof = tmp_nrof;
   /* Figure out how much weight this object will add to the player */
   weight = tmp->weight * nrof;
   if (tmp->inv)
     weight += tmp->carrying * (100 - tmp->stats.Str) / 100;
-  if (pl->stats.Str <= MAX_STAT)
-    effective_weight_limit = weight_limit[pl->stats.Str];
+  effective_weight_limit = weight_limit [min (MAX_STAT, pl->stats.Str)];
-  else
-    effective_weight_limit = weight_limit[MAX_STAT];
   if ((pl->weight + pl->carrying + weight) > effective_weight_limit)
     {
       new_draw_info (0, 0, pl, "That item is too heavy for you to pick up.");
       return;
     }
-  if (nrof != tmp_nrof)
+  if (!can_split (pl, tmp, nrof))
-    {
-      object *tmp2 = tmp;
-      tmp = get_split_ob (tmp, nrof);
-      if (!tmp)
-        {
-          new_draw_info (NDI_UNIQUE, 0, pl, errmsg);
-          return;
+    return;
-        }
-      /* Tell the client what happened to the rest of objects */
+  if (QUERY_FLAG (tmp, FLAG_UNPAID))
-      if (pl->contr)
+    {
-        if (tmp2->destroyed ())
+      tmp->flag.reset (FLAG_UNPAID);
-          esrv_del_item (pl->contr, tmp2->count);
+      new_draw_info_format (NDI_UNIQUE, 0, pl, "%s will cost you %s.", query_name (tmp), query_cost_string (tmp, pl, F_BUY | F_SHOP));
-        else
+      tmp->flag.set (FLAG_UNPAID);
-          esrv_update_item (UPD_NROF, pl, tmp2);
     }
   else
-    {
+    new_draw_info_format (NDI_UNIQUE, 0, pl, "You pick up the %s.", query_name (tmp));
-      /* If the object is in a container, send a delete to the client.
-       * - we are moving all the items from the container to elsewhere,
-       * so it needs to be deleted.
-       */
-      if (!QUERY_FLAG (tmp, FLAG_REMOVED))
-        {
-          if (tmp->env && pl->type == PLAYER)
-            esrv_del_item (pl->contr, tmp->count);
-          tmp->remove ();      /* Unlink it */
+  op->insert (tmp);
-        }
-    }
-  if (QUERY_FLAG (tmp, FLAG_UNPAID))
-    sprintf (buf, "%s will cost you %s.", query_name (tmp), query_cost_string (tmp, pl, F_BUY | F_SHOP));
-  else
-    sprintf (buf, "You pick up the %s.", query_name (tmp));
-  new_draw_info (NDI_UNIQUE, 0, pl, buf);
-  tmp = insert_ob_in_ob (tmp, op);
-  /* All the stuff below deals with client/server code, and is only
-   * usable by players
-   */
-  if (pl->contr)
-    {
-      esrv_send_item (pl, tmp);
-      /* These are needed to update the weight for the container we
-       * are putting the object in.
-       */
-      if (op != pl)
-        {
-          esrv_update_item (UPD_WEIGHT, pl, op);
-          esrv_send_item (pl, pl);
-        }
-      /* Update the container the object was in */
-      if (env && env != pl && env != op)
-        esrv_update_item (UPD_WEIGHT, pl, env);
-    }
 }
 /* modified slightly to allow monsters use this -b.t. 5-31-95 */
 void
 pick_up (object *op, object *alt)
       if (!can_pick (op, alt))
         {
           new_draw_info_format (NDI_UNIQUE, 0, op, "You can't pick up the %s.", &alt->name);
           goto leave;
         }
       tmp = alt;
     }
   else
     {
       if (op->below == NULL || !can_pick (op, op->below))
   /* Try to catch it. */
   tmp_map = tmp->map;
   tmp = stop_item (tmp);
   if (tmp == NULL)
     goto leave;
   need_fix_tmp = 1;
   if (!can_pick (op, tmp))
     goto leave;
   if (op->type == PLAYER)
     }
   else
     count = tmp->nrof;
   /* container is open, so use it */
+  if (tmp->flag [FLAG_STARTEQUIP])
+    alt = op;
-  if (op->container)
+  else if (op->container)
     {
       alt = op->container;
       if (alt != tmp->env && !sack_can_hold (op, alt, tmp, count))
         goto leave;
     }
       /* here it could be possible to check rent,
        * if someone wants to implement it
        */
       alt = op;
     }
 #ifdef PICKUP_DEBUG
   LOG (llevDebug, "Pick_up(): %s picks %s (%d) and inserts it %s.\n", op->name, tmp->name, op->contr->count, alt->name);
 #endif
   /* startequip items are not allowed to be put into containers: */
        * but that probably will make it more difficult to read, and
        * not make it any more efficient
        */
       if (params && item_matched_string (op, tmp, params))
         {
+          if (--cnt < 0) break;
           pick_up (op, tmp);
-          if (--cnt <= 0) break;
         }
       else if (can_pick (op, tmp) && !params)
         {
+          if (--cnt < 0) break;
           pick_up (op, tmp);
           break;
         }
       tmp = next;
     }
-  if (cnt <= 0)
+  if (cnt < 0)
     {
       op->failmsg ("Couldn't pick up so many items at once.");
       return 0;
     }
   if (!params && !tmp)
     {
       for (tmp = op->below; tmp; tmp = tmp->below)
         if (!tmp->invisible)
           {
-            char buf[MAX_BUF];
-            sprintf (buf, "You can't pick up a %s.", &tmp->name);
+            new_draw_info_format (NDI_UNIQUE, 0, op, "You can't pick up a %s.", &tmp->name);
-            new_draw_info (NDI_UNIQUE, 0, op, buf);
             break;
           }
       if (!tmp)
         new_draw_info (NDI_UNIQUE, 0, op, "There is nothing to pick up.");
     }
   return 0;
 }
 /*
  *  This function was part of drop, now is own function.
  *  Player 'op' tries to put object 'tmp' into sack 'sack',
  *  if nrof is non zero, then nrof objects is tried to put into sack.
       return;
     }
   if (tmp->type == CONTAINER && tmp->inv)
     {
       /* Eneq(@csd.uu.se): If the object to be dropped is a container
        * we instead move the contents of that container into the active
        * container, this is only done if the object has something in it.
        */
       sack2 = tmp;
       for (tmp2 = tmp->inv; tmp2; tmp2 = tmp)
         {
           tmp = tmp2->below;
           if ((sack->type == CONTAINER && sack_can_hold (op, op->container, tmp2, tmp2->nrof)))
-            {
-              put_object_in_sack (op, sack, tmp2, 0);
+            put_object_in_sack (op, sack, tmp2, 0);
-            }
           else
             {
               sprintf (buf, "Your %s fills up.", query_name (sack));
               new_draw_info (NDI_UNIQUE, 0, op, buf);
               break;
             }
         }
-      esrv_update_item (UPD_WEIGHT, op, sack2);
       return;
     }
   /* Don't worry about this for containers - our caller should have
    * already checked this.
   if (QUERY_FLAG (tmp, FLAG_APPLIED))
     if (apply_special (op, tmp, AP_UNAPPLY | AP_NO_MERGE))
       return;
   /* we want to put some portion of the item into the container */
-  if (nrof && tmp->nrof != nrof)
+  if (!can_split (op, tmp, nrof))
-    {
-      object *tmp2 = tmp;
-      tmp = get_split_ob (tmp, nrof);
-      if (!tmp)
-        {
-          new_draw_info (NDI_UNIQUE, 0, op, errmsg);
-          return;
+    return;
-        }
-      /* Tell a client what happened other objects */
-      if (tmp2->destroyed ())
-        esrv_del_item (op->contr, tmp2->count);
-      else                      /* this can proably be replaced with an update */
-        esrv_send_item (op, tmp2);
-    }
-  else
-    tmp->remove ();
   new_draw_info_format (NDI_UNIQUE, 0, op, "You put the %s in %s.", query_name (tmp), query_name (sack));
-  tmp2 = insert_ob_in_ob (tmp, sack);
+  sack->insert (tmp);
-  // elmex: 2007-11-26: removed update_stats and replaced
-  //                    it by update_after_inventory_change () in the caller
-  //                    of this function
-  /* If an object merged (and thus, different object), we need to
-   * delete the original.
-   */
-  if (tmp2 != tmp)
-    esrv_del_item (op->contr, tmp->count);
-  esrv_send_item (op, tmp2);
-  /* update the sacks weight */
-  esrv_update_item (UPD_WEIGHT, op, sack);
 }
 /*
  *  This function was part of drop, now is own function.
  *  Player 'op' tries to drop object 'tmp', if tmp is non zero, then
  * This is used when dropping objects onto the floor.
  */
 void
 drop_object (object *op, object *tmp, uint32 nrof)
 {
-  object *floor;
   if (QUERY_FLAG (tmp, FLAG_NO_DROP))
     return;
   if (QUERY_FLAG (tmp, FLAG_APPLIED))
     if (apply_special (op, tmp, AP_UNAPPLY | AP_NO_MERGE))
   //fprintf (stderr, "ui, on space is %ld\n", op->ms ().volume ());//D
   /* We are only dropping some of the items.  We split the current object
    * off
    */
-  if (nrof && tmp->nrof != nrof)
+  if (!can_split (op, tmp, nrof))
-    {
-      object *tmp2 = tmp;
-      tmp = get_split_ob (tmp, nrof);
-      if (!tmp)
-        {
-          new_draw_info (NDI_UNIQUE, 0, op, errmsg);
-          return;
+    return;
-        }
-      /* Tell a client what happened rest of objects.  tmp2 is now the
-       * original object
-       */
-      if (op->type == PLAYER)
-        {
-          if (tmp2->destroyed ())
-            esrv_del_item (op->contr, tmp2->count);
-          else
-            esrv_send_item (op, tmp2);
-        }
-    }
-  else
-    tmp->remove ();
   if (INVOKE_OBJECT (DROP, tmp, ARG_OBJECT (op)))
     return;
   if (QUERY_FLAG (tmp, FLAG_STARTEQUIP))
     {
       op->statusmsg (format ("You drop the %s.", query_name (tmp)));
-      op->statusmsg ("The gods who lent it to you retrieves it.");
+      op->statusmsg ("The god who lent it to you retrieves it.");
-      if (op->type == PLAYER)
-        esrv_del_item (op->contr, tmp->count);
       tmp->destroy ();
       op->update_stats ();
       return;
     }
-  if (op->type == PLAYER)
-    esrv_del_item (op->contr, tmp->count);
   /* Call this before we update the various windows/players.  At least
    * that we, we know the weight is correct.
    */
   // 2007-11-26: moved op->update_stats away and calling it later after
   //             all items of a drop command have been processed.
-  for (floor = GET_MAP_OB (op->map, op->x, op->y); floor; floor = floor->above)
+  for (object *floor = GET_MAP_OB (op->map, op->x, op->y); floor; floor = floor->above)
     if (INVOKE_OBJECT (DROP_ON, floor, ARG_OBJECT (tmp), ARG_OBJECT (op)))
       return;
   if (is_in_shop (op) && !QUERY_FLAG (tmp, FLAG_UNPAID) && tmp->type != MONEY)
+    {
-    sell_item (tmp, op);
+      if (!sell_item (tmp, op))
+        {
+          // if we can't sell it we don't drop it, so give it back to the seller
+          op->insert (tmp);
+          return;
+        }
+    }
   tmp->x = op->x;
   tmp->y = op->y;
   insert_ob_in_map (tmp, op->map, op, INS_BELOW_ORIGINATOR);
-}
-void update_after_inventory_change (object *op)
-{
-  /* Call this before we update the various windows/players.  At least
-   * that we, we know the weight is correct.
-   */
-  op->update_stats ();
-  // elmex: kept this for now, i have no idea what happens when i remove it:
-  if (op->type == PLAYER)
-    {
-      /* Need to update the weight for the player */
-      esrv_send_item (op, op);
-      op->contr->ns->floorbox_update ();
-    }
 }
 void
 drop (object *op, object *tmp)
 {
       if (tmp->env && tmp->env->type != PLAYER)
         {
           /* Just toss the object - probably shouldn't be hanging
            * around anyways
            */
-          tmp->remove ();
           tmp->destroy ();
           return;
         }
       else
         {
-          while (tmp != NULL && tmp->invisible)
+          while (tmp && tmp->invisible)
             tmp = tmp->below;
         }
     }
   if (tmp == NULL)
     {
       new_draw_info (NDI_UNIQUE, 0, op, "You don't have anything to drop.");
       return;
     }
   if (QUERY_FLAG (tmp, FLAG_INV_LOCKED))
     {
       new_draw_info (NDI_UNIQUE, 0, op, "This item is locked");
       return;
     }
   if (QUERY_FLAG (tmp, FLAG_NO_DROP))
     {
 #if 0
       /* Eneq(@csd.uu.se): Objects with NO_DROP defined can't be dropped. */
       new_draw_info (NDI_UNIQUE, 0, op, "This item can't be dropped.");
   if (op->type == PLAYER)
     op->contr->count = 0;
 }
 /* Command will drop all items that have not been locked */
 int
 command_dropall (object *op, char *params)
 {
-  object *curinv, *nextinv;
   if (op->inv == NULL)
     {
       new_draw_info (NDI_UNIQUE, 0, op, "Nothing to drop!");
       return 0;
     }
-  curinv = op->inv;
+  object *curinv = op->inv;
+  object *nextinv;
   /*
      This is the default.  Drops everything not locked or considered
      not something that should be dropped.
    */
      in a shop. --Tero.Pelander@utu.fi
    */
   int cnt = MAX_ITEM_PER_DROP;
-  if (params == NULL)
+  if (!params)
     {
-      while (curinv != NULL)
+      while (curinv)
         {
           nextinv = curinv->below;
           while (nextinv && nextinv->type == MONEY)
             nextinv = nextinv->below;
           if (!QUERY_FLAG (curinv, FLAG_INV_LOCKED) && curinv->type != MONEY &&
               curinv->type != FOOD && curinv->type != KEY &&
               curinv->type != SPECIAL_KEY && curinv->type != GEM &&
               !curinv->invisible && (curinv->type != CONTAINER || op->container != curinv))
             {
               drop (op, curinv);
               if (--cnt <= 0) break;
             }
           curinv = nextinv;
         }
     }
   else if (strcmp (params, "weapons") == 0)
     {
-      while (curinv != NULL)
+      while (curinv)
         {
           nextinv = curinv->below;
           while (nextinv && nextinv->type == MONEY)
             nextinv = nextinv->below;
           if (!QUERY_FLAG (curinv, FLAG_INV_LOCKED) && ((curinv->type == WEAPON) || (curinv->type == BOW) || (curinv->type == ARROW)))
             {
               drop (op, curinv);
               if (--cnt <= 0) break;
             }
           curinv = nextinv;
         }
     }
   else if (strcmp (params, "armor") == 0 || strcmp (params, "armour") == 0)
     {
-      while (curinv != NULL)
+      while (curinv)
         {
           nextinv = curinv->below;
           while (nextinv && nextinv->type == MONEY)
             nextinv = nextinv->below;
           if (!QUERY_FLAG (curinv, FLAG_INV_LOCKED) && ((curinv->type == ARMOUR) || curinv->type == SHIELD || curinv->type == HELMET))
             {
               drop (op, curinv);
               if (--cnt <= 0) break;
             }
           curinv = nextinv;
         }
     }
   else if (strcmp (params, "misc") == 0)
     {
-      while (curinv != NULL)
+      while (curinv)
         {
           nextinv = curinv->below;
           while (nextinv && nextinv->type == MONEY)
             nextinv = nextinv->below;
           if (!QUERY_FLAG (curinv, FLAG_INV_LOCKED) && !QUERY_FLAG (curinv, FLAG_APPLIED))
             {
               switch (curinv->type)
                 {
                   case HORN:
                     break;
                   default:
                     curinv = nextinv;
                     break;
                 }
               if (--cnt <= 0) break;
             }
           curinv = nextinv;
         }
     }
   if (cnt <= 0)
     op->failmsg ("Only dropped some items, can't drop that many items at once.");
-  update_after_inventory_change (op);
 /*  draw_look(op);*/
   return 0;
 }
 /* Object op wants to drop object(s) params.  params can be a
  * comma seperated list.
  */
 int
 command_drop (object *op, char *params)
 {
   object *tmp, *next;
   int did_one = 0;
       if (!did_one)
         new_draw_info (NDI_UNIQUE, 0, op, "Nothing to drop.");
       if (cnt <= 0)
         op->failmsg ("Only dropped some items, can't drop that many items at once.");
-      update_after_inventory_change (op);
     }
   return 0;
 }
     {
       object *tmp = op->below;
       while (tmp && !tmp->client_visible ())
         tmp = tmp->below;
       if (tmp)
         examine (op, tmp);
     }
   else
     {
       object *tmp = find_best_object_match (op, params);
       if (tmp)
         examine (op, tmp);
       else
-        new_draw_info_format (NDI_UNIQUE, 0, op, "Could not find an object that matches %s", params);
+        op->contr->infobox (MSG_CHANNEL ("examine"), format ("Could not find an object that matches %s", params));
     }
   return 0;
 }
 /* op should be a player.
  * we return the object the player has marked with the 'mark' command
   dynbuf_text buf (512, 512);
   object *mon = head ? head : this;
   if (QUERY_FLAG (mon, FLAG_UNDEAD))
-    buf << "It is an undead force.\n";
+    buf << "It is an undead force.\r";
   if (mon->level > who->level)
-    buf << "It is likely more powerful than you.\n";
+    buf << "It is likely more powerful than you.\r";
   else if (mon->level < who->level)
-    buf << "It is likely less powerful than you.\n";
+    buf << "It is likely less powerful than you.\r";
   else
-    buf << "It is probably as powerful as you.\n";
+    buf << "It is probably as powerful as you.\r";
   if (mon->attacktype & AT_ACID)
-    buf << "You seem to smell an acrid odor.\n";
+    buf << "You seem to smell an acrid odor.\r";
   /* Anyone know why this used to use the clone value instead of the
    * maxhp field?  This seems that it should give more accurate results.
    */
   switch ((mon->stats.hp + 1) * 4 / (mon->stats.maxhp + 1))
     {                           /* From 1-4 */
       case 1:
-        buf <<  "It is in a bad shape.\n";
+        buf <<  "It is in a bad shape.\r";
         break;
       case 2:
-        buf <<  "It is hurt.\n";
+        buf <<  "It is hurt.\r";
         break;
       case 3:
-        buf <<  "It is somewhat hurt.\n";
+        buf <<  "It is somewhat hurt.\r";
         break;
       case 4:
-        buf <<  "It is in excellent shape.\n";
+        buf <<  "It is in excellent shape.\r";
         break;
     }
   if (present_in_ob (POISONING, mon))
-    buf << "It looks very ill.\n";
+    buf << "It looks very ill.\r";
+  buf << '\n';
   return buf;
 }
 /* tmp is the object being described, pl is who is examing it. */
 std::string
 object::describe (object *who)
 {
   dynbuf_text buf (1024, 1024);
-  buf.printf ("That is: %s.\n", long_desc (who).c_str ());
+  buf.printf ("That is: %s.\r", long_desc (who).c_str ());
   if (custom_name)
-    buf.printf ("You call it %s\n", &custom_name);
+    buf.printf ("You call it %s.\r", &custom_name);
   switch (type)
     {
       case SPELLBOOK:
         if (flag [FLAG_IDENTIFIED] && inv)
-          buf.printf ("%s is a %s %s spell\n", &inv->name, get_levelnumber (inv->level), &inv->skill);
+          buf.printf ("%s is a %s %s spell.\r", &inv->name, get_levelnumber (inv->level), &inv->skill);
         break;
       case BOOK:
         if (msg)
-          buf << "Something is written in it.\n";
+          buf << "Something is written in it.\r";
         break;
       case CONTAINER:
-        if (race != NULL)
+        if (race)
           {
             if (weight_limit && stats.Str < 100)
-              buf.printf ("It can hold only %s and its weight limit is %.1f kg.\n",
+              buf.printf ("It can hold only %s and its weight limit is %.1f kg.\r",
                           &race, weight_limit / (10.0 * (100 - stats.Str)));
             else
-              buf.printf ("It can hold only %s.\n", &race);
+              buf.printf ("It can hold only %s.\r", &race);
           }
         else if (weight_limit && stats.Str < 100)
-          buf.printf ("Its weight limit is %.1f kg.\n", weight_limit / (10.0 * (100 - stats.Str)));
+          buf.printf ("Its weight limit is %.1f kg.\r", weight_limit / (10.0 * (100 - stats.Str)));
         break;
       case WAND:
         if (flag [FLAG_IDENTIFIED])
-          buf.printf ("It has %d charges left.\n", stats.food);
+          buf.printf ("It has %d charges left.\r", stats.food);
         break;
     }
   if (materialname && !msg)
-    buf.printf ("It is made of: %s.\n", &materialname);
+    buf.printf ("It is made of: %s.\r", &materialname);
   if (who)
     /* Where to wear this item */
     for (int i = 0; i < NUM_BODY_LOCATIONS; i++)
       if (slot[i].info)
           buf << (who->slot[i].info ? body_locations[i].use_name : body_locations[i].nonuse_name);
           if (slot[i].info < -1 && who->slot[i].info)
             buf.printf ("(%d)", -slot[i].info);
-          buf << ".\n";
+          buf << ".\r";
         }
   if (weight)
-    buf.printf (nrof > 1 ? "They weigh %3.3f kg.\n" : "It weighs %3.3f kg.\n", weight * (nrof ? nrof : 1) / 1000.0);
+    buf.printf ("%s %3.3f kg.\r", nrof > 1 ? "They weigh" : "It weighs", weight * (nrof ? nrof : 1) / 1000.0);
   if (value && !flag [FLAG_STARTEQUIP] && !flag [FLAG_NO_PICK] && who)
     {
-      buf.printf ("You reckon %s worth %s.\n", nrof > 1 ? "they are" : "it is", query_cost_string (this, who, F_TRUE | F_APPROX));
+      buf.printf ("You reckon %s worth %s.\r", nrof > 1 ? "they are" : "it is", query_cost_string (this, who, F_TRUE | F_APPROX));
       if (is_in_shop (who))
         {
           if (flag [FLAG_UNPAID])
-            buf.printf ("%s would cost you %s.\n", nrof > 1 ? "They" : "It", query_cost_string (this, who, F_BUY | F_SHOP));
+            buf.printf ("%s would cost you %s.\r", nrof > 1 ? "They" : "It", query_cost_string (this, who, F_BUY | F_SHOP));
           else
-            buf.printf ("You are offered %s for %s.\n", query_cost_string (this, who, F_SELL + F_SHOP), nrof > 1 ? "them" : "it");
+            buf.printf ("You are offered %s for %s.\r", query_cost_string (this, who, F_SELL + F_SHOP), nrof > 1 ? "them" : "it");
         }
     }
   if (flag [FLAG_MONSTER])
     buf << describe_monster (who);
   /* Is this item buildable? */
   if (flag [FLAG_IS_BUILDABLE])
-    buf << "This is a buildable item.\n";
+    buf << "This is a buildable item.\r";
   /* Does the object have a message?  Don't show message for all object
    * types - especially if the first entry is a match
    */
-  if (msg && type != EXIT && type != BOOK && type != CORPSE && !move_on && *msg != '@')
+  if (msg && type != EXIT && type != BOOK && type != CORPSE && !move_on && !has_dialogue ())
     {
       /* This is just a hack so when identifying the items, we print
        * out the extra message
        */
       if (need_identify (this) && flag [FLAG_IDENTIFIED])
-        buf << "The object has a story:\n";
+        buf << "The object has a story:\r";
       buf << msg << '\n';
     }
   buf << '\n';

Diff Legend

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

Comparing deliantra/server/server/c_object.C (file contents): Revision 1.63 by root, Mon Dec 17 08:35:49 2007 UTC vs. Revision 1.80 by elmex, Sun Oct 5 12:40:21 2008 UTC

Diff Legend

Comparing deliantra/server/server/c_object.C (file contents):
Revision 1.63 by root, Mon Dec 17 08:35:49 2007 UTC vs.
Revision 1.80 by elmex, Sun Oct 5 12:40:21 2008 UTC