From e50b5fe13775f98e337b5521719813736cb9fe18 Mon Sep 17 00:00:00 2001 From: Maxime Coste Date: Sun, 20 Dec 2020 11:32:15 +1100 Subject: [PATCH] Add Timer::disable() to be more explicit than set_next_date(TimePoint::max()) --- src/event_manager.hh | 3 ++- src/input_handler.cc | 8 ++++---- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/src/event_manager.hh b/src/event_manager.hh index f4bece31..7503e64a 100644 --- a/src/event_manager.hh +++ b/src/event_manager.hh @@ -66,7 +66,8 @@ public: ~Timer(); TimePoint next_date() const { return m_date; } - void set_next_date(TimePoint date) { m_date = date; } + void set_next_date(TimePoint date) { m_date = date; } + void disable() { m_date = TimePoint::max(); } void run(EventMode mode); private: diff --git a/src/input_handler.cc b/src/input_handler.cc index 14f9240b..b75efd94 100644 --- a/src/input_handler.cc +++ b/src/input_handler.cc @@ -214,8 +214,8 @@ public: void on_disabled(bool temporary) override { - m_idle_timer.set_next_date(TimePoint::max()); - m_fs_check_timer.set_next_date(TimePoint::max()); + m_idle_timer.disable(); + m_fs_check_timer.disable(); if (not temporary and m_hooks_disabled) { @@ -1136,7 +1136,7 @@ private: if (not temporary) context().print_status({}); - m_idle_timer.set_next_date(TimePoint::max()); + m_idle_timer.disable(); if (context().has_client()) context().client().menu_hide(); } @@ -1239,7 +1239,7 @@ public: void on_disabled(bool temporary) override { - m_idle_timer.set_next_date(TimePoint::max()); + m_idle_timer.disable(); if (not temporary) {