From 05e796723ffb782c9402e5d505b62c889338cc1f Mon Sep 17 00:00:00 2001 From: Samuel Gaist Date: Sun, 18 Feb 2024 10:43:54 +0100 Subject: [PATCH] fix(userspace): remove unread variable in restart_handler When hitting that part, the restart signal is triggered and the code leaves the loop, hence setting should_restart as false makes no sense in this context. Signed-off-by: Samuel Gaist --- userspace/falco/app/restart_handler.cpp | 1 - 1 file changed, 1 deletion(-) diff --git a/userspace/falco/app/restart_handler.cpp b/userspace/falco/app/restart_handler.cpp index fe1e0d86..0647b5b0 100644 --- a/userspace/falco/app/restart_handler.cpp +++ b/userspace/falco/app/restart_handler.cpp @@ -158,7 +158,6 @@ void falco::app::restart_handler::watcher_loop() noexcept // at least we don't make users wait for the timeout. if (should_restart) { - should_restart = false; // todo(jasondellaluce): make this a callback too maybe? g_restart_signal.trigger(); return;