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

Comparing deliantra/server/server/c_object.C (file contents):
Revision 1.69 by root, Tue Apr 22 07:01:47 2008 UTC vs.
Revision 1.76 by root, Sun Sep 7 21:31:23 2008 UTC

    * (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;
        * 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 (!can_split (op, tmp, nrof))
     return;
   new_draw_info_format (NDI_UNIQUE, 0, op, "You put the %s in %s.", query_name (tmp), query_name (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
 }
 /*
  *  This function was part of drop, now is own function.
  *  Player 'op' tries to drop object 'tmp', if tmp is non zero, then
   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))
+        return; // if we can't sell it we don't drop it
+    }
   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)
 {
           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;
 }
   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.\n";
   if (mon->level > who->level)
-    buf << "It is likely more powerful than you.\n";
+    buf << " - It is likely more powerful than you.\n";
   else if (mon->level < who->level)
-    buf << "It is likely less powerful than you.\n";
+    buf << " - It is likely less powerful than you.\n";
   else
-    buf << "It is probably as powerful as you.\n";
+    buf << " - It is probably as powerful as you.\n";
   if (mon->attacktype & AT_ACID)
-    buf << "You seem to smell an acrid odor.\n";
+    buf << " - You seem to smell an acrid odor.\n";
   /* 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.\n";
         break;
       case 2:
-        buf <<  "It is hurt.\n";
+        buf <<  " - It is hurt.\n";
         break;
       case 3:
-        buf <<  "It is somewhat hurt.\n";
+        buf <<  " - It is somewhat hurt.\n";
         break;
       case 4:
-        buf <<  "It is in excellent shape.\n";
+        buf <<  " - It is in excellent shape.\n";
         break;
     }
   if (present_in_ob (POISONING, mon))
-    buf << "It looks very ill.\n";
+    buf << " - It looks very ill.\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.\n\n", long_desc (who).c_str ());
   if (custom_name)
-    buf.printf ("You call it %s\n", &custom_name);
+    buf.printf ("You call it %s.\n\n", &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.\n", &inv->name, get_levelnumber (inv->level), &inv->skill);
         break;
       case BOOK:
         if (msg)
           buf << "Something is written in it.\n";
     buf << "This is a buildable item.\n";
   /* 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:\n\n";
       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.69 by root, Tue Apr 22 07:01:47 2008 UTC vs. Revision 1.76 by root, Sun Sep 7 21:31:23 2008 UTC

Diff Legend

Comparing deliantra/server/server/c_object.C (file contents):
Revision 1.69 by root, Tue Apr 22 07:01:47 2008 UTC vs.
Revision 1.76 by root, Sun Sep 7 21:31:23 2008 UTC