summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
Diffstat (limited to 'src')
-rw-r--r--src/actions/commands.cpp4
-rw-r--r--src/gui/shortcut/itemshortcut.cpp20
-rw-r--r--src/gui/shortcut/itemshortcut.h21
3 files changed, 23 insertions, 22 deletions
diff --git a/src/actions/commands.cpp b/src/actions/commands.cpp
index c74edbe43..03278c47f 100644
--- a/src/actions/commands.cpp
+++ b/src/actions/commands.cpp
@@ -2119,8 +2119,8 @@ impHandler(addSkillShortcut)
}
ItemShortcut *const selShortcut = itemShortcut[num];
- const int index = selShortcut->getFreeIndex();
- if (index < 0)
+ const size_t index = selShortcut->getFreeIndex();
+ if (index == SHORTCUT_ITEMS)
return true;
selShortcut->setItem(index,
diff --git a/src/gui/shortcut/itemshortcut.cpp b/src/gui/shortcut/itemshortcut.cpp
index 2f80f6759..7139fd74e 100644
--- a/src/gui/shortcut/itemshortcut.cpp
+++ b/src/gui/shortcut/itemshortcut.cpp
@@ -147,7 +147,7 @@ void ItemShortcut::clear()
}
}
-void ItemShortcut::useItem(const int index) const
+void ItemShortcut::useItem(const size_t index) const
{
const Inventory *const inv = PlayerInfo::getInventory();
if (inv == nullptr)
@@ -177,7 +177,7 @@ void ItemShortcut::useItem(const int index) const
}
}
-void ItemShortcut::equipItem(const int index) const
+void ItemShortcut::equipItem(const size_t index) const
{
const Inventory *const inv = PlayerInfo::getInventory();
if (inv == nullptr)
@@ -197,7 +197,7 @@ void ItemShortcut::equipItem(const int index) const
}
}
}
-void ItemShortcut::unequipItem(const int index) const
+void ItemShortcut::unequipItem(const size_t index) const
{
const Inventory *const inv = PlayerInfo::getInventory();
if (inv == nullptr)
@@ -232,14 +232,14 @@ void ItemShortcut::setItemSelected(const Item *const item)
}
}
-void ItemShortcut::setItem(const int index)
+void ItemShortcut::setItem(const size_t index)
{
mItems[index] = mItemSelected;
mItemColors[index] = mItemColorSelected;
save();
}
-void ItemShortcut::setItem(const int index,
+void ItemShortcut::setItem(const size_t index,
const int item,
const ItemColor color)
{
@@ -256,11 +256,11 @@ void ItemShortcut::setItemFast(const size_t index,
mItemColors[index] = color;
}
-void ItemShortcut::swap(const int index1, const int index2)
+void ItemShortcut::swap(const size_t index1,
+ const size_t index2)
{
- if (index1 < 0 || index2 < 0
- || CAST_U32(index1) >= SHORTCUT_ITEMS
- || CAST_U32(index2) >= SHORTCUT_ITEMS)
+ if (CAST_U32(index1) >= SHORTCUT_ITEMS ||
+ CAST_U32(index2) >= SHORTCUT_ITEMS)
{
return;
}
@@ -285,5 +285,5 @@ size_t ItemShortcut::getFreeIndex() const
if (mItems[i] < 0)
return i;
}
- return -1;
+ return SHORTCUT_ITEMS;
}
diff --git a/src/gui/shortcut/itemshortcut.h b/src/gui/shortcut/itemshortcut.h
index 8aa7a9efc..5e77e9152 100644
--- a/src/gui/shortcut/itemshortcut.h
+++ b/src/gui/shortcut/itemshortcut.h
@@ -66,10 +66,10 @@ class ItemShortcut final
*
* @param index Index of the shortcut item.
*/
- int getItem(const int index) const A_WARN_UNUSED
+ int getItem(const size_t index) const A_WARN_UNUSED
{ return mItems[index]; }
- ItemColor getItemColor(const int index) const A_WARN_UNUSED
+ ItemColor getItemColor(const size_t index) const A_WARN_UNUSED
{ return mItemColors[index]; }
void setItemData(const size_t index,
@@ -96,9 +96,9 @@ class ItemShortcut final
*
* @param index Index of the items.
*/
- void setItem(const int index);
+ void setItem(const size_t index);
- void setItem(const int index,
+ void setItem(const size_t index,
const int item,
const ItemColor color);
@@ -112,7 +112,7 @@ class ItemShortcut final
* @param index Index of the item.
* @param itemId ID of the item.
*/
- void setItems(const int index,
+ void setItems(const size_t index,
const int itemId,
const ItemColor color)
{ mItems[index] = itemId; mItemColors[index] = color; save(); }
@@ -142,7 +142,7 @@ class ItemShortcut final
/**
* Remove a item from the shortcut.
*/
- void removeItem(const int index)
+ void removeItem(const size_t index)
{ mItems[index] = -1; save(); }
/**
@@ -150,19 +150,20 @@ class ItemShortcut final
*
* @param index Index of the item shortcut.
*/
- void useItem(const int index) const;
+ void useItem(const size_t index) const;
/**
* Equip a item from the shortcut.
*/
- void equipItem(const int index) const;
+ void equipItem(const size_t index) const;
/**
* UnEquip a item from the shortcut.
*/
- void unequipItem(const int index) const;
+ void unequipItem(const size_t index) const;
- void swap(const int index1, const int index2);
+ void swap(const size_t index1,
+ const size_t index2);
void clear();