diff options
author | LaG1924 <12997935+LaG1924@users.noreply.github.com> | 2018-01-05 16:12:13 +0100 |
---|---|---|
committer | LaG1924 <12997935+LaG1924@users.noreply.github.com> | 2018-01-13 03:39:34 +0100 |
commit | f090b30e587c7af51fde86f36c67ef139ed2ce6f (patch) | |
tree | 727f1f77da4a5a630aa18083713989aaf1a919a9 | |
parent | All usages of previous event-system replaced with new event-system (diff) | |
download | AltCraft-f090b30e587c7af51fde86f36c67ef139ed2ce6f.tar AltCraft-f090b30e587c7af51fde86f36c67ef139ed2ce6f.tar.gz AltCraft-f090b30e587c7af51fde86f36c67ef139ed2ce6f.tar.bz2 AltCraft-f090b30e587c7af51fde86f36c67ef139ed2ce6f.tar.lz AltCraft-f090b30e587c7af51fde86f36c67ef139ed2ce6f.tar.xz AltCraft-f090b30e587c7af51fde86f36c67ef139ed2ce6f.tar.zst AltCraft-f090b30e587c7af51fde86f36c67ef139ed2ce6f.zip |
-rw-r--r-- | src/Event.cpp | 33 | ||||
-rw-r--r-- | src/Event.hpp | 28 | ||||
-rw-r--r-- | src/GlobalState.cpp | 3 |
3 files changed, 37 insertions, 27 deletions
diff --git a/src/Event.cpp b/src/Event.cpp index c857947..e8531d4 100644 --- a/src/Event.cpp +++ b/src/Event.cpp @@ -1,50 +1,55 @@ #include "Event.hpp" std::list<EventListener*> EventSystem::listeners; -std::mutex EventSystem::listenersMutex; +std::recursive_mutex EventSystem::listenersMutex; EventListener::EventListener() { - std::lock_guard<std::mutex> listenersLock(EventSystem::listenersMutex); + EventSystem::listenersMutex.lock(); EventSystem::listeners.push_back(this); + EventSystem::listenersMutex.unlock(); } EventListener::~EventListener() { - std::lock_guard<std::mutex> listenersLock(EventSystem::listenersMutex); + EventSystem::listenersMutex.lock(); EventSystem::listeners.remove(this); + EventSystem::listenersMutex.unlock(); } void EventListener::HandleEvent() { - std::lock_guard<std::mutex> lock(eventsQueueMutex); - std::lock_guard<std::mutex> lockHandlers(handlersMutex); + mutex.lock(); Event event = events.front(); events.pop(); if (handlers[event.id]) { handlers[event.id](event); } + mutex.unlock(); } void EventListener::HandleAllEvents() { - std::lock_guard<std::mutex> lock(eventsQueueMutex); - std::lock_guard<std::mutex> lockHandlers(handlersMutex); + if (!NotEmpty()) + return; + + mutex.lock(); while (!events.empty()) { Event event = events.front(); events.pop(); if (handlers[event.id]) { handlers[event.id](event); } - } + } + mutex.unlock(); } bool EventListener::NotEmpty() { - std::lock_guard<std::mutex> lock(eventsQueueMutex); - return !events.empty(); + bool ret = !events.empty(); + return ret; } void EventListener::WaitEvent() { - eventsQueueMutex.lock(); + mutex.lock(); while (events.empty()) { - eventsQueueMutex.unlock(); - eventsQueueMutex.lock(); + mutex.unlock(); + mutex.lock(); } - eventsQueueMutex.unlock(); + mutex.unlock(); }
\ No newline at end of file diff --git a/src/Event.hpp b/src/Event.hpp index 752fda4..38541a1 100644 --- a/src/Event.hpp +++ b/src/Event.hpp @@ -63,8 +63,7 @@ class EventListener { using HandlerType = std::function<void(const Event&)>; std::queue<Event> events; std::map<size_t, HandlerType> handlers; - std::mutex eventsQueueMutex; - std::mutex handlersMutex; + std::recursive_mutex mutex; public: EventListener(); @@ -79,8 +78,9 @@ public: void WaitEvent(); void RegisterHandler(size_t eventId, const HandlerType &data) { - std::lock_guard<std::mutex> lock(handlersMutex); + mutex.lock(); handlers[eventId] = data; + mutex.unlock(); } void RegisterHandler(const char *eventId, const HandlerType & data) { @@ -91,22 +91,25 @@ public: class EventSystem { friend class EventListener; static std::list<EventListener*> listeners; - static std::mutex listenersMutex; + static std::recursive_mutex listenersMutex; public: template <typename T> static void PushEvent(size_t eventId, T data) { Event event(eventId, data); - std::lock_guard<std::mutex> listenersLock(listenersMutex); for (auto& listener : listeners) { - std::lock_guard<std::mutex> lock(listener->eventsQueueMutex); - std::lock_guard<std::mutex> lockHandlers(listener->handlersMutex); + //if (!listener->mutex.try_lock()) throw std::runtime_error("WHY?!"); + listener->mutex.lock(); auto it = listener->handlers.find(eventId); - if (it == listener->handlers.end()) - continue; + if (it == listener->handlers.end()) { + listener->mutex.unlock(); + continue; + } listener->events.push(event); + + listener->mutex.unlock(); } } @@ -114,16 +117,17 @@ public: static void DirectEventCall(size_t eventId, T data) { Event event(eventId, data); - std::lock_guard<std::mutex> listenersLock(listenersMutex); + listenersMutex.lock(); for (auto & listener : listeners) { - std::lock_guard<std::mutex> lock(listener->eventsQueueMutex); - std::lock_guard<std::mutex> lockHandlers(listener->handlersMutex); + listener->mutex.lock(); auto it = listener->handlers.find(eventId); if (it == listener->handlers.end()) continue; it->second(event); + listener->mutex.unlock(); } + listenersMutex.unlock(); } }; diff --git a/src/GlobalState.cpp b/src/GlobalState.cpp index 2bd4943..e947e01 100644 --- a/src/GlobalState.cpp +++ b/src/GlobalState.cpp @@ -45,7 +45,7 @@ void InitEvents() { } catch (std::exception &e) { LOG(WARNING) << "Connection failed"; - PUSH_EVENT("ConnectionFailed", e.what()); + PUSH_EVENT("ConnectionFailed", std::string(e.what())); return; } LOG(INFO) << "Connected to server"; @@ -177,6 +177,7 @@ void PhysExec() { } void GsExec() { + el::Helpers::setThreadName("Game"); LoopExecutionTimeController timer(std::chrono::milliseconds(16)); while (isRunning) { |