From 4638e5aececfb33d39677d6d4f6241a3953ebe34 Mon Sep 17 00:00:00 2001 From: Luca Fedeli Date: Tue, 1 Aug 2023 00:08:20 +0200 Subject: Clang tidy CI test: add some modernize checks to clang-tidy CI test (#4133) * Clang Tidy: add several modernize checks * eliminate file included in commit by mistake * fix bug --- Source/ablastr/utils/SignalHandling.cpp | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) (limited to 'Source/ablastr/utils/SignalHandling.cpp') diff --git a/Source/ablastr/utils/SignalHandling.cpp b/Source/ablastr/utils/SignalHandling.cpp index 0fdbd834e..feaf69d00 100644 --- a/Source/ablastr/utils/SignalHandling.cpp +++ b/Source/ablastr/utils/SignalHandling.cpp @@ -16,7 +16,7 @@ // For sigaction() et al. #if defined(__linux__) || defined(__APPLE__) -# include +# include #endif namespace ablastr::utils { @@ -121,8 +121,8 @@ SignalHandling::InitSignalHandling () signal_received_flags[signal_number] = false; bool signal_active = false; - for (int signal_request = 0; signal_request < SIGNAL_REQUESTS_SIZE; ++signal_request) { - signal_active |= signal_conf_requests[signal_request][signal_number]; + for (const auto& request : signal_conf_requests) { + signal_active |= request[signal_number]; } if (signal_active) { // at least one signal action is configured @@ -162,8 +162,9 @@ SignalHandling::CheckSignals () const bool signal_received = signal_received_flags[signal_number].exchange(false); if (signal_received) { - for (int signal_request = 0; signal_request < SIGNAL_REQUESTS_SIZE; ++signal_request) { - signal_actions_requested[signal_request] |= signal_conf_requests[signal_request][signal_number]; + int signal_request = 0; + for (const auto& request : signal_conf_requests) { + signal_actions_requested[signal_request++] |= request[signal_number]; } } } -- cgit v1.2.3