cleanup

Author: NikolasK-source

cmake_files/warnings.cmake

@@ -63,6 +63,7 @@ function(clangwarn target)
     target_compile_options(${target} PRIVATE -Wno-gnu-anonymous-struct)
     target_compile_options(${target} PRIVATE -Wno-source-uses-openmp)
     target_compile_options(${target} PRIVATE -Wno-disabled-macro-expansion)
+    target_compile_options(${target} PRIVATE -Wno-c++20-designator)
 
 endfunction()
 

include/cxxitimer.hpp

@@ -41,7 +41,7 @@ class ITimer {
 
 protected:
     //* internal use only!
-    explicit ITimer(int type, const timeval &interval = {1, 0}) noexcept;
+    explicit ITimer(int type, const timeval &interval = {.tv_sec = 1, .tv_usec = 0}) noexcept;
 
     //* internal use only!
     ITimer(int type, const timeval &interval, const timeval &value) noexcept;
@@ -192,7 +192,7 @@ class ITimer_Real : public ITimer {
      * @param interval timer interval
      * @exception std::logic_error instance exists
      */
-    explicit ITimer_Real(const timeval &interval = {1, 0});
+    explicit ITimer_Real(const timeval &interval = {.tv_sec = 1, .tv_usec = 0});
 
     /**
      * @brief create ITimer_Real instance
@@ -251,7 +251,7 @@ class ITimer_Virtual : public ITimer {
      * @param interval timer interval
      * @exception std::logic_error instance exists
      */
-    explicit ITimer_Virtual(const timeval &interval = {1, 0});
+    explicit ITimer_Virtual(const timeval &interval = {.tv_sec = 1, .tv_usec = 0});
 
     /**
      * @brief create ITimer_Virtual instance
@@ -313,7 +313,7 @@ class ITimer_Prof : public ITimer {
      * @param interval timer interval
      * @exception std::logic_error instance exists
      */
-    explicit ITimer_Prof(const timeval &interval = {1, 0});
+    explicit ITimer_Prof(const timeval &interval = {.tv_sec = 1, .tv_usec = 0});
 
     /**
      * @brief create ITimer_Prof instance

src/cxxitimer.cpp

@@ -13,7 +13,7 @@
 namespace cxxitimer {
 
 //* timeval to stop timer
-static constexpr itimerval STOP_TIMER = {{0, 0}, {0, 0}};
+static constexpr itimerval STOP_TIMER = {};
 
 //* number of usec per second
 constexpr auto USEC_PER_SEC = static_cast<double>(1000000);
@@ -97,7 +97,7 @@ void ITimer::start() {
     if (running) throw std::logic_error("timer already started");
 
     // create scaled timer value
-    itimerval timer_val {timer_interval / speed_factor, timer_value / speed_factor};
+    const itimerval timer_val = {.it_interval = timer_interval / speed_factor, .it_value = timer_value / speed_factor};
 
     if (timer_val.it_interval.tv_sec < 0 || timer_val.it_interval.tv_usec < 0)
         throw std::runtime_error("timer interval is negative");
@@ -344,7 +344,7 @@ timeval double_to_timeval(const double time) noexcept {
         sec -= 1;
         usec += static_cast<suseconds_t>(USEC_PER_SEC);
     }
-    return timeval {sec, usec};
+    return timeval {.tv_sec = sec, .tv_usec = usec};
 }
 
 }  // namespace cxxitimer

test/test.cpp

@@ -6,7 +6,7 @@
 #include "cxxitimer.hpp"
 
 #include <iostream>
-#include <signal.h>
+#include <csignal>
 #include <thread>
 
 static int  x = 0;