From 69b4c19414ea1c7a37f3806668583299249b0e46 Mon Sep 17 00:00:00 2001 From: Axel Huebl Date: Fri, 8 Apr 2022 14:31:10 -0700 Subject: Signal: Cleanup (Simplify) (#3031) Simplify the logic by fusing a loop. --- Source/ablastr/utils/SignalHandling.cpp | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) (limited to 'Source/ablastr/utils/SignalHandling.cpp') diff --git a/Source/ablastr/utils/SignalHandling.cpp b/Source/ablastr/utils/SignalHandling.cpp index e56faeb52..f071315ed 100644 --- a/Source/ablastr/utils/SignalHandling.cpp +++ b/Source/ablastr/utils/SignalHandling.cpp @@ -125,6 +125,9 @@ SignalHandling::InitSignalHandling () signal_active |= signal_conf_requests[signal_request][signal_number]; } if (signal_active) { + // at least one signal action is configured + m_any_signal_action_active = true; + if (amrex::ParallelDescriptor::MyProc() == 0) { sa.sa_handler = &SignalHandling::SignalSetFlag; } else { @@ -135,12 +138,6 @@ SignalHandling::InitSignalHandling () "Failed to install signal handler for a configured signal"); } } - - for (int signal_number = 0; signal_number < NUM_SIGNALS; ++signal_number) { - for (int signal_request = 0; signal_request < SIGNAL_REQUESTS_SIZE; ++signal_request) { - m_any_signal_action_active |= signal_conf_requests[signal_request][signal_number]; - } - } #endif } -- cgit v1.2.3