diff options
Diffstat (limited to '')
| -rw-r--r-- | src/UI/Window.cpp | 38 |
1 files changed, 5 insertions, 33 deletions
diff --git a/src/UI/Window.cpp b/src/UI/Window.cpp index 55df04798..629db12c2 100644 --- a/src/UI/Window.cpp +++ b/src/UI/Window.cpp @@ -421,43 +421,15 @@ void cWindow::DistributeStackToAreas(cItem & a_ItemStack, cPlayer & a_Player, cS bool cWindow::CollectItemsToHand(cItem & a_Dragging, cSlotArea & a_Area, cPlayer & a_Player, bool a_CollectFullStacks) { - // First ask the slot areas from a_Area till the end of list: - bool ShouldCollect = false; - for (cSlotAreas::iterator itr = m_SlotAreas.begin(), end = m_SlotAreas.end(); itr != end; ++itr) + // Ask to collect items from each slot area in order: + for (auto Area : m_SlotAreas) { - if (&a_Area == *itr) - { - ShouldCollect = true; - } - if (!ShouldCollect) - { - continue; - } - if ((*itr)->CollectItemsToHand(a_Dragging, a_Player, a_CollectFullStacks)) - { - // a_Dragging is full - return true; - } - } - - // a_Dragging still not full, ask slot areas before a_Area in the list: - for (cSlotAreas::iterator itr = m_SlotAreas.begin(), end = m_SlotAreas.end(); itr != end; ++itr) - { - if (*itr == &a_Area) - { - // All areas processed - return false; - } - if ((*itr)->CollectItemsToHand(a_Dragging, a_Player, a_CollectFullStacks)) + if (Area->CollectItemsToHand(a_Dragging, a_Player, a_CollectFullStacks)) { - // a_Dragging is full - return true; + return true; // a_Dragging is full } } - // Shouldn't reach here - // a_Area is expected to be part of m_SlotAreas[], so the "return false" in the loop above should have returned already - ASSERT(!"This branch should not be reached"); - return false; + return false; // All areas processed } |