diff options
Diffstat (limited to 'src')
| -rw-r--r-- | src/actions/commands.cpp | 4 | ||||
| -rw-r--r-- | src/gui/shortcut/itemshortcut.cpp | 20 | ||||
| -rw-r--r-- | src/gui/shortcut/itemshortcut.h | 21 |
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(); |