Fix formatting and add header to iwyu

SpaceTeam · May 27, 2024 · 7ae6ac9 · 7ae6ac9
1 parent e2db5e1
commit 7ae6ac9
Show file tree

Hide file tree

Showing 6 changed files with 12 additions and 5 deletions.
diff --git a/Sts1CobcSw/FileSystem/CMakeLists.txt b/Sts1CobcSw/FileSystem/CMakeLists.txt
@@ -10,7 +10,9 @@ else()
     target_sources(Sts1CobcSw_FileSystem PRIVATE LfsRam.cpp)
     target_sources(Sts1CobcSw_FileSystem_NoRodos PRIVATE LfsRam.cpp)
     target_sources(Sts1CobcSw_FileSystem PRIVATE LfsWrapper.cpp)
-    target_link_libraries(Sts1CobcSw_FileSystem_NoRodos PUBLIC littlefs::littlefs Sts1CobcSw_Outcome)
+    target_link_libraries(
+        Sts1CobcSw_FileSystem_NoRodos PUBLIC littlefs::littlefs Sts1CobcSw_Outcome
+    )
     target_link_libraries(Sts1CobcSw_FileSystem_NoRodos PRIVATE Sts1CobcSw_Serial)
     target_compile_definitions(Sts1CobcSw_FileSystem_NoRodos PRIVATE NO_RODOS LFS_THREADSAFE)
 endif()
diff --git a/Sts1CobcSw/FileSystem/LfsFlash.cpp b/Sts1CobcSw/FileSystem/LfsFlash.cpp
@@ -86,6 +86,7 @@ auto Read(lfs_config const * config,
     return 0;
 }
 
+
 auto Program(lfs_config const * config,
              lfs_block_t blockNo,
              lfs_off_t offset,
@@ -137,7 +138,6 @@ auto Sync([[maybe_unused]] lfs_config const * config) -> int
 auto Lock(const struct lfs_config * config) -> int
 {
     return 0;
-
 }
 
 

diff --git a/Tests/UnitTests/CMakeLists.txt b/Tests/UnitTests/CMakeLists.txt
@@ -13,7 +13,9 @@ target_link_libraries(
 
 # TODO: Enable again once problem with segmentation violation on CI is fixed
 add_program(LfsRam LfsRam.test.cpp)
-target_link_libraries(Sts1CobcSwTests_LfsRam PRIVATE Catch2::Catch2WithMain Sts1CobcSw_FileSystem_NoRodos)
+target_link_libraries(
+    Sts1CobcSwTests_LfsRam PRIVATE Catch2::Catch2WithMain Sts1CobcSw_FileSystem_NoRodos
+)
 
 add_program(Outcome Outcome.test.cpp)
 target_link_libraries(Sts1CobcSwTests_Outcome PRIVATE Catch2::Catch2WithMain)

diff --git a/Tests/UnitTests/NoCatch2/CMakeLists.txt b/Tests/UnitTests/NoCatch2/CMakeLists.txt
@@ -1,6 +1,8 @@
 # No Catch2, since RODOS (main) is needed
 add_program(LfsRamCustomSemaphore LfsRamCustomSemaphore.test.cpp)
-target_link_libraries(Sts1CobcSwTests_LfsRamCustomSemaphore PRIVATE rodos::rodos Sts1CobcSw_FileSystem)
+target_link_libraries(
+    Sts1CobcSwTests_LfsRamCustomSemaphore PRIVATE rodos::rodos Sts1CobcSw_FileSystem
+)
 
 add_program(LockCollision LockCollision.test.cpp)
 target_link_libraries(Sts1CobcSwTests_LockCollision PRIVATE rodos::rodos Sts1CobcSw_FileSystem)
diff --git a/Tests/UnitTests/NoCatch2/LockCollision.test.cpp b/Tests/UnitTests/NoCatch2/LockCollision.test.cpp
@@ -13,7 +13,7 @@ using RODOS::PRINTF;
 class LockingThread : public RODOS::StaticThread<>
 {
 public:
-    explicit LockingThread(const char * threadName) : StaticThread(threadName)
+    explicit LockingThread(char const * threadName) : StaticThread(threadName)
     {
     }
 

diff --git a/iwyu.imp b/iwyu.imp
@@ -85,6 +85,7 @@
   { include: ["\"Sts1CobcSw/Utility/FlatArray.hpp\"",           "public", "<Sts1CobcSw/Utility/FlatArray.hpp>", "public"] },
   { include: ["\"Sts1CobcSw/Utility/Span.hpp\"",                "public", "<Sts1CobcSw/Utility/Span.hpp>", "public"] },
   { include: ["\"Sts1CobcSw/Utility/Time.hpp\"",                "public", "<Sts1CobcSw/Utility/Time.hpp>", "public"] },
+  { include: ["\"Sts1CobcSw/Utility/LinuxSemaphore.hpp\"",      "public", "<Sts1CobcSw/Utility/LinuxSemaphore.hpp>", "public"] },
   { include: ["\"Sts1CobcSw/CommandParser.hpp\"",               "public", "<Sts1CobcSw/CommandParser.hpp>", "public"] },
   { include: ["\"Sts1CobcSw/EduCommunicationErrorThread.hpp\"", "public", "<Sts1CobcSw/EduCommunicationErrorThread.hpp>", "public"] },
   { include: ["\"Sts1CobcSw/EduListenerThread.hpp\"",           "public", "<Sts1CobcSw/EduListenerThread.hpp>", "public"] },