summaryrefslogtreecommitdiffstats
path: root/src/UI
diff options
context:
space:
mode:
Diffstat (limited to 'src/UI')
-rw-r--r--src/UI/Window.cpp14
-rw-r--r--src/UI/Window.h10
2 files changed, 13 insertions, 11 deletions
diff --git a/src/UI/Window.cpp b/src/UI/Window.cpp
index 8bbc4f482..9fbc4d988 100644
--- a/src/UI/Window.cpp
+++ b/src/UI/Window.cpp
@@ -1,4 +1,4 @@
-#include "Globals.h" // NOTE: MSVC stupidness requires this to be the same across all modules
+#include "Globals.h" // NOTE: MSVC stupidness requires this to be the same across all modules
#include "Window.h"
#include "WindowOwner.h"
@@ -362,12 +362,12 @@ void cWindow::OwnerDestroyed()
-bool cWindow::ForEachPlayer(cItemCallback<cPlayer> & a_Callback)
+bool cWindow::ForEachPlayer(const cPlayerListCallback & a_Callback)
{
cCSLock Lock(m_CS);
- for (cPlayerList::iterator itr = m_OpenedBy.begin(), end = m_OpenedBy.end(); itr != end; ++itr)
+ for (auto & Player : m_OpenedBy)
{
- if (a_Callback.Item(*itr))
+ if (a_Callback(*Player))
{
return false;
}
@@ -379,12 +379,12 @@ bool cWindow::ForEachPlayer(cItemCallback<cPlayer> & a_Callback)
-bool cWindow::ForEachClient(cItemCallback<cClientHandle> & a_Callback)
+bool cWindow::ForEachClient(cClientHandleCallback & a_Callback)
{
cCSLock Lock(m_CS);
- for (cPlayerList::iterator itr = m_OpenedBy.begin(), end = m_OpenedBy.end(); itr != end; ++itr)
+ for (auto & Player : m_OpenedBy)
{
- if (a_Callback.Item((*itr)->GetClientHandle()))
+ if (a_Callback(*Player->GetClientHandle()))
{
return false;
}
diff --git a/src/UI/Window.h b/src/UI/Window.h
index d7a29dc47..f2023c731 100644
--- a/src/UI/Window.h
+++ b/src/UI/Window.h
@@ -1,4 +1,4 @@
-
+
// Window.h
// Interfaces to the cWindow class representing a UI window for a specific block
@@ -9,6 +9,7 @@
#pragma once
+#include <functional>
#include "../ItemGrid.h"
@@ -31,7 +32,8 @@ class cWorld;
typedef std::list<cPlayer *> cPlayerList;
typedef std::vector<cSlotArea *> cSlotAreas;
-
+using cPlayerListCallback = std::function<bool(cPlayer &)>;
+using cClientHandleCallback = std::function<bool(cClientHandle &)>;
@@ -151,10 +153,10 @@ public:
void OwnerDestroyed(void);
/** Calls the callback safely for each player that has this window open; returns true if all players have been enumerated */
- bool ForEachPlayer(cItemCallback<cPlayer> & a_Callback);
+ bool ForEachPlayer(const cPlayerListCallback & a_Callback);
/** Calls the callback safely for each client that has this window open; returns true if all clients have been enumerated */
- bool ForEachClient(cItemCallback<cClientHandle> & a_Callback);
+ bool ForEachClient(cClientHandleCallback & a_Callback);
/** Called on shift-clicking to distribute the stack into other areas; Modifies a_ItemStack as it is distributed!
if a_ShouldApply is true, the changes are written into the slots;