some more style and cast fixes.

6 files changed, 22 insertions, 27 deletions

diff --git a/src/gui/buyselldialog.cpp b/src/gui/buyselldialog.cpp
index e84982737..7db437c4f 100644
--- a/src/gui/buyselldialog.cpp
+++ b/src/gui/buyselldialog.cpp
@@ -32,7 +32,7 @@
 
 #include "debug.h"
 
-BuySellDialog::DialogList BuySellDialog::instances;
+BuySellDialog::DialogList BuySellDialog::dialogInstances;
 
 BuySellDialog::BuySellDialog(const int npcId) :
     Window(_("Shop"), false, nullptr, "buysell.xml"),
@@ -86,13 +86,13 @@ void BuySellDialog::init()
     loadWindowState();
     enableVisibleSound(true);
 
-    instances.push_back(this);
+    dialogInstances.push_back(this);
     setVisible(true);
 }
 
 BuySellDialog::~BuySellDialog()
 {
-    instances.remove(this);
+    dialogInstances.remove(this);
 }
 
 void BuySellDialog::setVisible(bool visible)
@@ -133,7 +133,7 @@ void BuySellDialog::action(const gcn::ActionEvent &event)
 
 void BuySellDialog::closeAll()
 {
-    FOR_EACH (DialogList::const_iterator, it, instances)
+    FOR_EACH (DialogList::const_iterator, it, dialogInstances)
     {
         if (*it)
             (*it)->close();
diff --git a/src/gui/buyselldialog.h b/src/gui/buyselldialog.h
index 371e8bdd9..ef3a98f18 100644
--- a/src/gui/buyselldialog.h
+++ b/src/gui/buyselldialog.h
@@ -64,7 +64,7 @@ class BuySellDialog final : public Window, public gcn::ActionListener
          * Returns true if any instances exist.
          */
         static bool isActive() A_WARN_UNUSED
-        { return !instances.empty(); }
+        { return !dialogInstances.empty(); }
 
         /**
          * Closes all instances.
@@ -73,7 +73,7 @@ class BuySellDialog final : public Window, public gcn::ActionListener
 
     private:
         typedef std::list<BuySellDialog*> DialogList;
-        static DialogList instances;
+        static DialogList dialogInstances;
 
         int mNpcId;
         std::string mNick;
diff --git a/src/gui/inventorywindow.cpp b/src/gui/inventorywindow.cpp
index fe51018bf..22fea71c7 100644
--- a/src/gui/inventorywindow.cpp
+++ b/src/gui/inventorywindow.cpp
@@ -86,7 +86,7 @@ public:
     }
 };
 
-InventoryWindow::WindowList InventoryWindow::instances;
+InventoryWindow::WindowList InventoryWindow::invInstances;
 
 InventoryWindow::InventoryWindow(Inventory *const inventory):
     Window("Inventory", false, nullptr, "inventory.xml"),
@@ -103,7 +103,7 @@ InventoryWindow::InventoryWindow(Inventory *const inventory):
     mEquipmentButton(nullptr),
     mStoreButton(nullptr),
     mRetrieveButton(nullptr),
-    mCloseButton(nullptr),
+    mInvCloseButton(nullptr),
     mWeightBar(nullptr),
     mSlotsBar(new ProgressBar(this, 0.0f, 100, 0, Theme::PROG_INVY_SLOTS)),
     mFilter(nullptr),
@@ -205,7 +205,7 @@ InventoryWindow::InventoryWindow(Inventory *const inventory):
     {
         mStoreButton = new Button(this, _("Store"), "store", this);
         mRetrieveButton = new Button(this, _("Retrieve"), "retrieve", this);
-        mCloseButton = new Button(this, _("Close"), "close", this);
+        mInvCloseButton = new Button(this, _("Close"), "close", this);
 
         mSlotsBarCell = &place(0, 0, mSlotsBar, 6);
         mSortDropDownCell = &place(6, 0, mSortDropDown, 1);
@@ -216,7 +216,7 @@ InventoryWindow::InventoryWindow(Inventory *const inventory):
         place(0, 2, invenScroll, 7, 4);
         place(0, 6, mStoreButton);
         place(1, 6, mRetrieveButton);
-        place(6, 6, mCloseButton);
+        place(6, 6, mInvCloseButton);
     }
 
     Layout &layout = getLayout();
@@ -224,7 +224,7 @@ InventoryWindow::InventoryWindow(Inventory *const inventory):
 
     mInventory->addInventoyListener(this);
 
-    instances.push_back(this);
+    invInstances.push_back(this);
 
     if (inventory && inventory->isMainInventory())
     {
@@ -232,8 +232,8 @@ InventoryWindow::InventoryWindow(Inventory *const inventory):
     }
     else
     {
-        if (!instances.empty())
-            instances.front()->updateDropButton();
+        if (!invInstances.empty())
+            invInstances.front()->updateDropButton();
     }
 
     loadWindowState();
@@ -247,10 +247,10 @@ InventoryWindow::InventoryWindow(Inventory *const inventory):
 
 InventoryWindow::~InventoryWindow()
 {
-    instances.remove(this);
+    invInstances.remove(this);
     mInventory->removeInventoyListener(this);
-    if (!instances.empty())
-        instances.front()->updateDropButton();
+    if (!invInstances.empty())
+        invInstances.front()->updateDropButton();
     delete mSortModel;
     mSortModel = nullptr;
 }
@@ -707,7 +707,7 @@ bool InventoryWindow::isInputFocused() const
 
 bool InventoryWindow::isAnyInputFocused()
 {
-    FOR_EACH (WindowList::const_iterator, it, instances)
+    FOR_EACH (WindowList::const_iterator, it, invInstances)
     {
         if ((*it) && (*it)->isInputFocused())
             return true;
diff --git a/src/gui/inventorywindow.h b/src/gui/inventorywindow.h
index 83e26948b..903f15254 100644
--- a/src/gui/inventorywindow.h
+++ b/src/gui/inventorywindow.h
@@ -131,7 +131,7 @@ class InventoryWindow final : public Window,
          * Returns true if any instances exist.
          */
         static bool isStorageActive() A_WARN_UNUSED
-        { return instances.size() > 1; }
+        { return invInstances.size() > 1; }
 
         void updateDropButton();
 
@@ -154,7 +154,7 @@ class InventoryWindow final : public Window,
 
 
         typedef std::list<InventoryWindow*> WindowList;
-        static WindowList instances;
+        static WindowList invInstances;
 
         Inventory *mInventory;
         ItemContainer *mItems;
@@ -169,7 +169,7 @@ class InventoryWindow final : public Window,
         Button *mEquipmentButton;
         Button *mStoreButton;
         Button *mRetrieveButton;
-        Button *mCloseButton;
+        Button *mInvCloseButton;
 
         ProgressBar *mWeightBar;
         ProgressBar *mSlotsBar;
diff --git a/src/gui/widgets/avatarlistbox.cpp b/src/gui/widgets/avatarlistbox.cpp
index 372218fe6..fe46feba2 100644
--- a/src/gui/widgets/avatarlistbox.cpp
+++ b/src/gui/widgets/avatarlistbox.cpp
@@ -48,8 +48,7 @@ AvatarListBox::AvatarListBox(const Widget2 *const widget,
     ListBox(widget, model, "avatarlistbox.xml"),
     mShowGender(config.getBoolValue("showgender")),
     mShowLevel(config.getBoolValue("showlevel")),
-    mImagePadding(mSkin ? mSkin->getOption("imagePadding") : 0),
-    mHighlightColor(getThemeColor(Theme::HIGHLIGHT))
+    mImagePadding(mSkin ? mSkin->getOption("imagePadding") : 0)
 {
     instances++;
 
@@ -99,13 +98,10 @@ void AvatarListBox::draw(gcn::Graphics *gcnGraphics)
 
     AvatarListModel *const model = static_cast<AvatarListModel *const>(
         mListModel);
-//    Guild *guild = dynamic_cast<Guild*>(model);
-
     updateAlpha();
-
     Graphics *const graphics = static_cast<Graphics *const>(gcnGraphics);
 
-    mHighlightColor.a = static_cast<int>(mAlpha * 255.0f);
+//    mHighlightColor.a = static_cast<int>(mAlpha * 255.0f);
     gcn::Font *const font = getFont();
 
     const int fontHeight = getFont()->getHeight();
diff --git a/src/gui/widgets/avatarlistbox.h b/src/gui/widgets/avatarlistbox.h
index 7d283abea..579651d29 100644
--- a/src/gui/widgets/avatarlistbox.h
+++ b/src/gui/widgets/avatarlistbox.h
@@ -65,7 +65,6 @@ private:
     bool mShowGender;
     bool mShowLevel;
     int mImagePadding;
-    gcn::Color mHighlightColor;
 
     static int instances;
     static Image *onlineIcon;