Improve tradewindow.

2 files changed, 15 insertions, 24 deletions

diff --git a/src/gui/tradewindow.cpp b/src/gui/tradewindow.cpp
index 88dac7d55..2e5d026b2 100644
--- a/src/gui/tradewindow.cpp
+++ b/src/gui/tradewindow.cpp
@@ -89,8 +89,6 @@ TradeWindow::TradeWindow():
     mOkOther(false),
     mOkMe(false)
 {
-    logger->log1("TradeWindow::TradeWindow nick");
-
     setWindowName("Trade");
     setResizable(true);
     setCloseButton(true);
@@ -182,7 +180,7 @@ void TradeWindow::setMoney(const int amount)
     mGotMoney = amount;
     // TRANSLATORS: trade window money label
     mMoneyLabel->setCaption(strprintf(_("You get %s"),
-                            Units::formatCurrency(amount).c_str()));
+        Units::formatCurrency(amount).c_str()));
     mMoneyLabel->adjustSize();
 }
 
@@ -208,31 +206,24 @@ void TradeWindow::addItem2(const int id, const bool own, const int quantity,
 void TradeWindow::changeQuantity(const int index, const bool own,
                                  const int quantity) const
 {
+    Item *item;
     if (own)
-    {
-        if (mMyInventory->getItem(index))
-            mMyInventory->getItem(index)->setQuantity(quantity);
-    }
+        item = mMyInventory->getItem(index);
     else
-    {
-        if (mPartnerInventory->getItem(index))
-            mPartnerInventory->getItem(index)->setQuantity(quantity);
-    }
+        item = mPartnerInventory->getItem(index);
+    if (item)
+        item->setQuantity(quantity);
 }
 
 void TradeWindow::increaseQuantity(const int index, const bool own,
                                    const int quantity) const
 {
+    Item *item;
     if (own)
-    {
-        if (mMyInventory->getItem(index))
-            mMyInventory->getItem(index)->increaseQuantity(quantity);
-    }
+        item = mMyInventory->getItem(index);
     else
-    {
-        if (mPartnerInventory->getItem(index))
-            mPartnerInventory->getItem(index)->increaseQuantity(quantity);
-    }
+        item = mPartnerInventory->getItem(index);
+    item->increaseQuantity(quantity);
 }
 
 void TradeWindow::reset()
@@ -363,7 +354,6 @@ void TradeWindow::action(const gcn::ActionEvent &event)
         setVisible(false);
         reset();
         PlayerInfo::setTrading(false);
-
         Net::getTradeHandler()->cancel();
     }
     else if (eventId == "ok")
@@ -436,7 +426,7 @@ void TradeWindow::addAutoMoney(const std::string &nick, const int money)
     mAutoMoney = money;
 }
 
-void TradeWindow::initTrade(std::string nick)
+void TradeWindow::initTrade(const std::string &nick)
 {
     if (!player_node)
         return;
diff --git a/src/gui/tradewindow.h b/src/gui/tradewindow.h
index ebf0cf6b2..eb5a2369b 100644
--- a/src/gui/tradewindow.h
+++ b/src/gui/tradewindow.h
@@ -92,7 +92,8 @@ class TradeWindow final : public Window,
         /**
          * Increase quantity of an item.
          */
-        void increaseQuantity(int index, bool own, int quantity) const;
+        void increaseQuantity(const int index, const bool own,
+                              const int quantity) const;
 
         /**
          * Player received ok message from server
@@ -120,7 +121,7 @@ class TradeWindow final : public Window,
          * Closes the Trade Window, as well as telling the server that the
          * window has been closed.
          */
-        void close();
+        void close() override;
 
         /**
          * Clear auto trade items.
@@ -135,7 +136,7 @@ class TradeWindow final : public Window,
 
         void addAutoMoney(const std::string &nick, const int money);
 
-        void initTrade(std::string nick);
+        void initTrade(const std::string &nick);
 
         std::string getAutoTradeNick() const A_WARN_UNUSED
         { return mAutoAddToNick; }