From b7bbb09d62b55bed26d432a6a2af9831c6ec9143 Mon Sep 17 00:00:00 2001 From: Andrei Karas Date: Thu, 24 Mar 2016 20:42:22 +0300 Subject: Change ItemType into strong typed enum. --- src/gui/popups/itempopup.cpp | 2 +- src/gui/popups/itempopup.h | 4 ++-- src/gui/widgets/itemcontainer.cpp | 4 ++-- src/gui/windows/buydialog.cpp | 4 ++-- src/gui/windows/inventorywindow.cpp | 3 ++- 5 files changed, 9 insertions(+), 8 deletions(-) (limited to 'src/gui') diff --git a/src/gui/popups/itempopup.cpp b/src/gui/popups/itempopup.cpp index f0ab5d167..065704cb8 100644 --- a/src/gui/popups/itempopup.cpp +++ b/src/gui/popups/itempopup.cpp @@ -325,7 +325,7 @@ std::string ItemPopup::getCardsString(const int *const cards) return label->setForegroundColorAll(getThemeColor(name2), \ getThemeColor(name2##_OUTLINE)); \ } -void ItemPopup::setLabelColor(Label *label, const ItemType::Type type) const +void ItemPopup::setLabelColor(Label *label, const ItemTypeT type) const { switch (type) { diff --git a/src/gui/popups/itempopup.h b/src/gui/popups/itempopup.h index 7f5481627..fc0320614 100644 --- a/src/gui/popups/itempopup.h +++ b/src/gui/popups/itempopup.h @@ -82,14 +82,14 @@ class ItemPopup final : public Popup TextBox *mItemEffect A_NONNULLPOINTER; TextBox *mItemWeight A_NONNULLPOINTER; TextBox *mItemCards A_NONNULLPOINTER; - ItemType::Type mItemType; + ItemTypeT mItemType; Icon *mIcon A_NONNULLPOINTER; std::string mLastName; int mLastId; ItemColor mLastColor; void setLabelColor(Label *label, - const ItemType::Type type) const A_NONNULL(2); + const ItemTypeT type) const A_NONNULL(2); }; extern ItemPopup *itemPopup; diff --git a/src/gui/widgets/itemcontainer.cpp b/src/gui/widgets/itemcontainer.cpp index 02ad10ee9..96bdb1dcc 100644 --- a/src/gui/widgets/itemcontainer.cpp +++ b/src/gui/widgets/itemcontainer.cpp @@ -158,8 +158,8 @@ namespace const ItemInfo &info1 = pair1->mItem->getInfo(); const ItemInfo &info2 = pair2->mItem->getInfo(); - const ItemType::Type t1 = info1.getType(); - const ItemType::Type t2 = info2.getType(); + const ItemTypeT t1 = info1.getType(); + const ItemTypeT t2 = info2.getType(); if (t1 == t2) { return (info1.getName() < info2.getName()); diff --git a/src/gui/windows/buydialog.cpp b/src/gui/windows/buydialog.cpp index 47ec11db2..c6be959de 100644 --- a/src/gui/windows/buydialog.cpp +++ b/src/gui/windows/buydialog.cpp @@ -160,8 +160,8 @@ namespace if (!item1 || !item2) return false; - const ItemType::Type type1 = item1->getInfo().getType(); - const ItemType::Type type2 = item2->getInfo().getType(); + const ItemTypeT type1 = item1->getInfo().getType(); + const ItemTypeT type2 = item2->getInfo().getType(); if (type1 == type2) return item1->getPrice() < item2->getPrice(); return type1 < type2; diff --git a/src/gui/windows/inventorywindow.cpp b/src/gui/windows/inventorywindow.cpp index 881e10c1a..5485bd98a 100644 --- a/src/gui/windows/inventorywindow.cpp +++ b/src/gui/windows/inventorywindow.cpp @@ -1047,7 +1047,8 @@ void InventoryWindow::combineItems(const int index1, if (!item2) return; - if (item1->getType() != ItemType::CARD) + // IT_CARD + if (item1->getType() != 6) { const Item *tmpItem = item1; item1 = item2; -- cgit v1.2.3-60-g2f50