tweaks

CMakeLists.txt

@@ -340,9 +340,9 @@ if(FSFW_BUILD_TESTS)
                           -ftest-coverage)
 
       if(WIN32)
-        setup_target_for_coverage_gcovr_html(
-          NAME ${FSFW_TEST_TGT}_coverage EXECUTABLE ${FSFW_TEST_TGT}
-          DEPENDENCIES ${FSFW_TEST_TGT})
+        #setup_target_for_coverage_gcovr_html(
+        #  NAME ${FSFW_TEST_TGT}_coverage EXECUTABLE ${FSFW_TEST_TGT}
+        #  DEPENDENCIES ${FSFW_TEST_TGT})
       else()
         setup_target_for_coverage_lcov(
           NAME ${FSFW_TEST_TGT}_coverage EXECUTABLE ${FSFW_TEST_TGT}

cmake/cmake-modules/bilke/CodeCoverage.cmake

@@ -140,7 +140,7 @@ find_program( GCOV_PATH gcov )
 find_program( LCOV_PATH  NAMES lcov lcov.bat lcov.exe lcov.perl)
 find_program( FASTCOV_PATH NAMES fastcov fastcov.py )
 find_program( GENHTML_PATH NAMES genhtml genhtml.perl genhtml.bat )
-find_program( GCOVR_PATH gcovr PATHS ${CMAKE_SOURCE_DIR}/scripts/test)
+find_program( GCOVR_PATH gcovr )
 find_program( CPPFILT_PATH NAMES c++filt )
 
 if(NOT GCOV_PATH)

src/fsfw/globalfunctions/timevalOperations.cpp

@@ -1,9 +1,12 @@
 #include "fsfw/globalfunctions/timevalOperations.h"
 
+#include <cstdio>
+
 timeval& operator+=(timeval& lhs, const timeval& rhs) {
-  int64_t sum = lhs.tv_sec * 1000000. + lhs.tv_usec;
-  sum += rhs.tv_sec * 1000000. + rhs.tv_usec;
-  lhs.tv_sec = sum / 1000000;
+  int64_t sum = static_cast<int64_t>(lhs.tv_sec) * 1000000. + lhs.tv_usec;
+  sum += static_cast<int64_t>(rhs.tv_sec) * 1000000. + rhs.tv_usec;
+  int64_t tmp = sum / 1000000;
+  lhs.tv_sec = tmp;
   lhs.tv_usec = sum - lhs.tv_sec * 1000000;
   return lhs;
 }

src/fsfw/osal/common/TcpIpBase.cpp

@@ -1,7 +1,7 @@
 #include "fsfw/osal/common/TcpIpBase.h"
 
-#include "fsfw/serviceinterface.h"
 #include "fsfw/platform.h"
+#include "fsfw/serviceinterface.h"
 
 #ifdef PLATFORM_UNIX
 #include <errno.h>

src/fsfw/osal/windows/winTaskHelpers.cpp

@@ -1,8 +1,9 @@
 #include "fsfw/osal/windows/winTaskHelpers.h"
 
-#include <mutex>
 #include <windows.h>
 
+#include <mutex>
+
 TaskPriority tasks::makeWinPriority(PriorityClass prioClass, PriorityNumber prioNumber) {
   return (static_cast<uint16_t>(prioClass) << 16) | static_cast<uint16_t>(prioNumber);
 }

src/fsfw/serviceinterface/ServiceInterfaceBuffer.cpp

@@ -3,7 +3,6 @@
 #if FSFW_CPP_OSTREAM_ENABLED == 1
 
 #include <cinttypes>
-
 #include <cstring>
 
 #include "fsfw/serviceinterface/serviceInterfaceDefintions.h"

src/fsfw/timemanager/TimeReaderIF.h

@@ -2,6 +2,7 @@
 #define FSFW_TIMEMANAGER_TIMEREADERIF_H
 
 #include <cstdlib>
+
 #include "fsfw/platform.h"
 
 #ifdef PLATFORM_WIN

src/fsfw_hal/common/printChar.c

@@ -1,5 +1,5 @@
-#include <stdio.h>
 #include <stdbool.h>
+#include <stdio.h>
 
 void __attribute__((weak)) printChar(const char* character, bool errStream) {
   if (errStream) {

unittests/globalfunctions/testTimevalOperations.cpp

@@ -56,6 +56,7 @@ TEST_CASE("TimevalTest", "[timevalOperations]") {
   }
   SECTION("Operators") {
     timeval t1;
+    if (sizeof(t1.tv_sec) == 8) {
       t1.tv_sec = 1648227422;
       t1.tv_usec = 123456;
       timeval t2;
@@ -76,7 +77,9 @@ TEST_CASE("TimevalTest", "[timevalOperations]") {
       t6.tv_usec = 999999;
 
       timeval t7 = t6 + t1;
+      // Overflow test
       REQUIRE(t7.tv_sec == (1648227422ull + 1648227400ull + 1ull));
+
       REQUIRE(t7.tv_usec == 123455);
 
       timeval t8 = t1 - t6;
@@ -112,6 +115,7 @@ TEST_CASE("TimevalTest", "[timevalOperations]") {
       double scalar5 = t12 / t1;
       REQUIRE(scalar5 == Catch::Approx(0.5));
     }
+  }
 
   SECTION("timevalOperations::toTimeval") {
     double seconds = 1648227422.123456;

unittests/hal/CMakeLists.txt

@@ -1,3 +1,5 @@
-target_sources(${FSFW_TEST_TGT} PRIVATE
+if(UNIX)
+  target_sources(${FSFW_TEST_TGT} PRIVATE
 	  testCommandExecutor.cpp
-)
+  )
+endif()