From b463ceaef4d82ee7c425c88749b9fd68ea27e5b8 Mon Sep 17 00:00:00 2001 From: jacqueline Date: Tue, 7 Nov 2023 14:53:47 +1100 Subject: [PATCH] use std::list instead of std::deque for events std::deque loves to allocate 512 bytes in internal memory just for fun --- src/events/include/event_queue.hpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/events/include/event_queue.hpp b/src/events/include/event_queue.hpp index 332f7be5..78b21d53 100644 --- a/src/events/include/event_queue.hpp +++ b/src/events/include/event_queue.hpp @@ -29,7 +29,7 @@ class Queue { auto Add(std::function fn) { { std::lock_guard lock{mut_}; - events_.push(fn); + events_.push_back(fn); } xSemaphoreGive(has_events_); } @@ -50,7 +50,7 @@ class Queue { } had_work = true; fn = events_.front(); - events_.pop(); + events_.pop_front(); } std::invoke(fn); } @@ -64,7 +64,7 @@ class Queue { private: SemaphoreHandle_t has_events_; std::mutex mut_; - std::queue> events_; + std::list> events_; }; template