From b32f28d74e9ec5cfe3bc443e2fbffd5de245fd0b Mon Sep 17 00:00:00 2001
From: Björn Steinbrink <B.Steinbrink@gmx.de>
Date: Thu, 29 Dec 2005 12:30:46 +0000
Subject: Removed duplicate pick up code

---
 src/game.cpp          | 17 ++++-------------
 src/gui/popupmenu.cpp |  4 +---
 src/net/protocol.cpp  |  2 +-
 3 files changed, 6 insertions(+), 17 deletions(-)

(limited to 'src')

diff --git a/src/game.cpp b/src/game.cpp
index ab0e73d9..fb130acc 100644
--- a/src/game.cpp
+++ b/src/game.cpp
@@ -557,13 +557,8 @@ void do_input()
                         }
 
                         if (id)
-                        {
-                            // TODO: Remove duplicated code, probably add a
-                            // pick up command
-                            MessageOut outMsg;
-                            outMsg.writeInt16(0x009f);
-                            outMsg.writeInt32(id);
-                        }
+                            pickUp(id);
+
                         used = true;
                     }
                     break;
@@ -803,12 +798,8 @@ void do_input()
         {
             Uint32 id = find_floor_item_by_cor(player_node->x, player_node->y);
 
-            if (id != 0)
-            {
-                MessageOut outMsg;
-                outMsg.writeInt16(CMSG_ITEM_PICKUP);
-                outMsg.writeInt32(id);
-            }
+            if (id)
+                pickUp(id);
         }
         else if (joy[JOY_BTN2] && action_time)
         {
diff --git a/src/gui/popupmenu.cpp b/src/gui/popupmenu.cpp
index b1046553..8d0ace57 100644
--- a/src/gui/popupmenu.cpp
+++ b/src/gui/popupmenu.cpp
@@ -177,9 +177,7 @@ void PopupMenu::handleLink(const std::string& link)
     // Pick Up Floor Item action
     else if ((link == "pickup") && mFloorItem != NULL)
     {
-        MessageOut outMsg;
-        outMsg.writeInt16(CMSG_ITEM_PICKUP);
-        outMsg.writeInt32(mFloorItem->getId());
+        pickUp(mFloorItem->getId());
     }
 
     // Look To action
diff --git a/src/net/protocol.cpp b/src/net/protocol.cpp
index 7a98ffb6..28663827 100644
--- a/src/net/protocol.cpp
+++ b/src/net/protocol.cpp
@@ -93,7 +93,7 @@ void talk(Being *being)
 void pickUp(Uint32 floorItemId)
 {
     MessageOut outMsg;
-    outMsg.writeInt16(0x009f);
+    outMsg.writeInt16(CMSG_ITEM_PICKUP);
     outMsg.writeInt32(floorItemId);
 }
 
-- 
cgit v1.2.3-70-g09d2