Merge pull request #2275 from eclipse-iceoryx/iox-2274-fix-clang-tidy-18-warnings

iox-#2274 Fix clang-tidy-18 warnings

Author: elBoberido

.clang-tidy

@@ -21,7 +21,7 @@ concurrency-*,
 performance-*,
 hicpp-*,
 
--cppcoreguidelines-avoid-do-while
+-cppcoreguidelines-avoid-do-while,
 -cppcoreguidelines-non-private-member-variables-in-classes,
 
 -hicpp-named-parameter,
@@ -30,13 +30,16 @@ hicpp-*,
 -readability-identifier-length,
 -readability-redundant-access-specifiers,
 -readability-redundant-declaration,
+-readability-redundant-inline-specifier,
 -readability-static-accessed-through-instance,
 
 -readability-identifier-naming,
 -readability-use-anyofallof,
 -readability-named-parameter,
 
--cert-dcl21-cpp
+-performance-avoid-endl,
+
+-cert-dcl21-cpp,
 
 '
 
@@ -98,6 +101,9 @@ hicpp-*,
 #        // in cpp file
 #        void bar(Foo& foo) {...}
 #
+# * rule: readability-redundant-inline-specifier
+#   justicfication: there are many false positives
+#
 # * rule: readability-use-anyofallof
 #   justification: requires C++20 and std::ranges but we only use C++17
 #
@@ -132,6 +138,12 @@ hicpp-*,
 #       Tag1 tag;
 #       foo(tag); // calls (1)
 #
+# * rule: performance-avoid-endl
+#   justification: 'iostream' is only used in a few places where the logger either cannot be used, likt the platform layer, or
+#                  we want to print immediatelly something on the screen like the command line parser or the examples. In this
+#                  cases we would immediatelly after the '\n' calls 'std::flush'. Therefore we keep using 'std::endl' in this
+#                  limited places.
+#
 ## Those warnings should be enabled
 ## They are disabled since they require a heavy API refactoring and when we enable it we clutter the code with // NOLINT comments
 # -bugprone-easily-swappable-parameters

.clang-tidy-diff-scans.txt

@@ -1,7 +1,7 @@
 ./iceoryx_hoofs/**/*
 
-./iceoryx_posh/include/iox/**/*
-./iceoryx_posh/source/posh/**/*
+./iceoryx_posh/experimental/include/iox/**/*
+./iceoryx_posh/experimental/source/**/*
 
 # IMPORTANT:
 #    after the first # everything is considered a comment, add new files and

doc/website/release-notes/iceoryx-unreleased.md

@@ -217,6 +217,7 @@
 - Split `iceoryx_hoofs` into logical modules [#1391](https://github.com/eclipse-iceoryx/iceoryx/issues/1391)
 - Create a flat include structure for `iceoryx_hoofs` [#1593](https://github.com/eclipse-iceoryx/iceoryx/issues/1593)
 - Handle 'strerror_r' idiosyncrasies in the platform layer [#1616](https://github.com/eclipse-iceoryx/iceoryx/issues/1616)
+- Fix new clang-tidy-18 warnings [#2274](https://github.com/eclipse-iceoryx/iceoryx/issues/2274)
 
 **Workflow:**
 

iceoryx_hoofs/buffer/include/iox/detail/stack.inl

@@ -47,6 +47,7 @@ inline stack<T, Capacity>& stack<T, Capacity>::copy(const stack& rhs) noexcept
 }
 
 template <typename T, uint64_t Capacity>
+// NOLINTNEXTLINE(cppcoreguidelines-rvalue-reference-param-not-moved) false positive, the elements are moved
 inline stack<T, Capacity>& stack<T, Capacity>::move(stack&& rhs) noexcept
 {
     uint64_t i{0};

iceoryx_hoofs/cli/include/iox/cli/arguments.hpp

@@ -37,7 +37,7 @@ namespace cli
 class Arguments
 {
   public:
-    enum class Error
+    enum class Error : uint8_t
     {
         UNABLE_TO_CONVERT_VALUE,
         NO_SUCH_VALUE

iceoryx_hoofs/concurrent/buffer/include/iox/detail/mpmc_lockfree_queue/mpmc_lockfree_queue.inl

@@ -59,6 +59,7 @@ inline bool MpmcLockFreeQueue<ElementType, Capacity>::tryPush(const ElementType&
 }
 
 template <typename ElementType, uint64_t Capacity>
+// NOLINTNEXTLINE(cppcoreguidelines-rvalue-reference-param-not-moved) perfect forwarding is used
 inline bool MpmcLockFreeQueue<ElementType, Capacity>::tryPush(ElementType&& value) noexcept
 {
     uint64_t index{0};
@@ -120,6 +121,7 @@ inline iox::optional<ElementType> MpmcLockFreeQueue<ElementType, Capacity>::push
 }
 
 template <typename ElementType, uint64_t Capacity>
+// NOLINTNEXTLINE(cppcoreguidelines-rvalue-reference-param-not-moved) perfect forwarding is used
 inline iox::optional<ElementType> MpmcLockFreeQueue<ElementType, Capacity>::push(ElementType&& value) noexcept
 {
     return pushImpl(std::forward<ElementType>(value));

iceoryx_hoofs/concurrent/buffer/include/iox/detail/mpmc_lockfree_queue/mpmc_resizeable_lockfree_queue.inl

@@ -53,8 +53,10 @@ inline bool MpmcResizeableLockFreeQueue<ElementType, MaxCapacity>::setCapacity(c
 
 template <typename ElementType, uint64_t MaxCapacity>
 template <typename Function, typename>
-inline bool MpmcResizeableLockFreeQueue<ElementType, MaxCapacity>::setCapacity(const uint64_t newCapacity,
-                                                                               Function&& removeHandler) noexcept
+inline bool MpmcResizeableLockFreeQueue<ElementType, MaxCapacity>::setCapacity(
+    const uint64_t newCapacity,
+    // NOLINTNEXTLINE(cppcoreguidelines-missing-std-forward) false positive, this is used as a universal reference
+    Function&& removeHandler) noexcept
 {
     if (newCapacity > MAX_CAPACITY)
     {
@@ -121,9 +123,10 @@ MpmcResizeableLockFreeQueue<ElementType, MaxCapacity>::increaseCapacity(const ui
 
 template <typename ElementType, uint64_t MaxCapacity>
 template <typename Function>
-inline uint64_t
-MpmcResizeableLockFreeQueue<ElementType, MaxCapacity>::decreaseCapacity(const uint64_t toDecrease,
-                                                                        Function&& removeHandler) noexcept
+inline uint64_t MpmcResizeableLockFreeQueue<ElementType, MaxCapacity>::decreaseCapacity(
+    const uint64_t toDecrease,
+    // NOLINTNEXTLINE(cppcoreguidelines-missing-std-forward) false positive, this is used as a universal reference
+    Function&& removeHandler) noexcept
 {
     uint64_t decreased = 0U;
     while (decreased < toDecrease)
@@ -191,6 +194,7 @@ iox::optional<ElementType> inline MpmcResizeableLockFreeQueue<ElementType, MaxCa
 
 template <typename ElementType, uint64_t MaxCapacity>
 inline iox::optional<ElementType>
+// NOLINTNEXTLINE(cppcoreguidelines-rvalue-reference-param-not-moved) perfect forwarding is used
 MpmcResizeableLockFreeQueue<ElementType, MaxCapacity>::push(ElementType&& value) noexcept
 {
     return pushImpl(std::forward<ElementType>(value));

iceoryx_hoofs/container/include/iox/detail/forward_list.inl

@@ -354,6 +354,7 @@ inline bool forward_list<T, Capacity>::push_front(const T& data) noexcept
 }
 
 template <typename T, uint64_t Capacity>
+// NOLINTNEXTLINE(cppcoreguidelines-rvalue-reference-param-not-moved) perfect forwarding is used
 inline bool forward_list<T, Capacity>::push_front(T&& data) noexcept
 {
     auto sizeBeforePush = m_size;
@@ -380,8 +381,10 @@ inline typename forward_list<T, Capacity>::iterator forward_list<T, Capacity>::i
 }
 
 template <typename T, uint64_t Capacity>
-inline typename forward_list<T, Capacity>::iterator forward_list<T, Capacity>::insert_after(const_iterator citer,
-                                                                                            T&& data) noexcept
+inline typename forward_list<T, Capacity>::iterator forward_list<T, Capacity>::insert_after(
+    const_iterator citer,
+    // NOLINTNEXTLINE(cppcoreguidelines-rvalue-reference-param-not-moved) perfect forwarding is used
+    T&& data) noexcept
 {
     return emplace_after(citer, std::forward<T>(data));
 }

iceoryx_hoofs/container/include/iox/detail/list.inl

@@ -24,28 +24,28 @@
 
 namespace iox
 {
-/// @NOLINTJUSTIFICATION m_data fields are explicitly initialized whenever a new element is inserted
-/// into the list
-/// @NOLINTNEXTLINE(cppcoreguidelines-pro-type-member-init,hicpp-member-init)
+// NOLINTJUSTIFICATION m_data fields are explicitly initialized whenever a new element is inserted
+// into the list
+// NOLINTNEXTLINE(cppcoreguidelines-pro-type-member-init,hicpp-member-init)
 template <typename T, uint64_t Capacity>
 inline list<T, Capacity>::list() noexcept
 {
     init();
 }
 
-/// @NOLINTJUSTIFICATION m_data fields are explicitly initialized whenever a new element is inserted
-/// into the list
-/// @NOLINTNEXTLINE(cppcoreguidelines-pro-type-member-init,hicpp-member-init)
+// NOLINTJUSTIFICATION m_data fields are explicitly initialized whenever a new element is inserted
+// into the list
+// NOLINTNEXTLINE(cppcoreguidelines-pro-type-member-init,hicpp-member-init)
 template <typename T, uint64_t Capacity>
 inline list<T, Capacity>::list(const list& rhs) noexcept
 {
     init();
     *this = rhs;
 }
 
-/// @NOLINTJUSTIFICATION m_data fields are explicitly initialized whenever a new element is inserted
-/// into the list
-/// @NOLINTNEXTLINE(cppcoreguidelines-pro-type-member-init,hicpp-member-init)
+// NOLINTJUSTIFICATION m_data fields are explicitly initialized whenever a new element is inserted
+// into the list
+// NOLINTNEXTLINE(cppcoreguidelines-pro-type-member-init,hicpp-member-init)
 template <typename T, uint64_t Capacity>
 inline list<T, Capacity>::list(list&& rhs) noexcept
 {
@@ -371,6 +371,7 @@ inline bool list<T, Capacity>::push_front(const T& data) noexcept
 }
 
 template <typename T, uint64_t Capacity>
+// NOLINTNEXTLINE(cppcoreguidelines-rvalue-reference-param-not-moved) perfect forwarding is used
 inline bool list<T, Capacity>::push_front(T&& data) noexcept
 {
     auto sizeBeforePush = m_size;
@@ -394,6 +395,7 @@ inline bool list<T, Capacity>::push_back(const T& data) noexcept
 }
 
 template <typename T, uint64_t Capacity>
+// NOLINTNEXTLINE(cppcoreguidelines-rvalue-reference-param-not-moved) perfect forwarding is used
 inline bool list<T, Capacity>::push_back(T&& data) noexcept
 {
     auto sizeBeforePush = m_size;
@@ -427,6 +429,7 @@ inline typename list<T, Capacity>::iterator list<T, Capacity>::insert(const_iter
 }
 
 template <typename T, uint64_t Capacity>
+// NOLINTNEXTLINE(cppcoreguidelines-rvalue-reference-param-not-moved) perfect forwarding is used
 inline typename list<T, Capacity>::iterator list<T, Capacity>::insert(const_iterator citer, T&& data) noexcept
 {
     return emplace(citer, std::forward<T>(data));
@@ -466,9 +469,9 @@ template <bool IsConstIterator>
 inline typename list<T, Capacity>::template IteratorBase<IsConstIterator>&
 list<T, Capacity>::IteratorBase<IsConstIterator>::operator=(const IteratorBase<false>& rhs) noexcept
 {
-    /// @NOLINTJUSTIFICATION ensure that this and rhs are not the same object without taking the type into account
-    ///                      required since it is possible to assign a non const iterator to a const iterator
-    /// @NOLINTNEXTLINE(cppcoreguidelines-pro-type-reinterpret-cast)
+    // NOLINTJUSTIFICATION ensure that this and rhs are not the same object without taking the type into account
+    //                      required since it is possible to assign a non const iterator to a const iterator
+    // NOLINTNEXTLINE(cppcoreguidelines-pro-type-reinterpret-cast)
     if (reinterpret_cast<const void*>(this) != reinterpret_cast<const void*>(&rhs))
     {
         m_list = rhs.m_list;
@@ -594,9 +597,9 @@ inline const typename list<T, Capacity>::size_type& list<T, Capacity>::getPrevId
 template <typename T, uint64_t Capacity>
 inline typename list<T, Capacity>::size_type& list<T, Capacity>::getPrevIdx(const size_type idx) noexcept
 {
-    /// @NOLINTJUSTIFICATION const_cast avoids code duplication, is safe since the constness of the return value is
-    /// restored
-    /// @NOLINTNEXTLINE(cppcoreguidelines-pro-type-const-cast)
+    // NOLINTJUSTIFICATION const_cast avoids code duplication, is safe since the constness of the return value is
+    // restored
+    // NOLINTNEXTLINE(cppcoreguidelines-pro-type-const-cast)
     return const_cast<size_type&>(const_cast<const list<T, Capacity>*>(this)->getPrevIdx(idx));
 }
 
@@ -608,9 +611,9 @@ inline const typename list<T, Capacity>::size_type& list<T, Capacity>::getNextId
 template <typename T, uint64_t Capacity>
 inline typename list<T, Capacity>::size_type& list<T, Capacity>::getNextIdx(const size_type idx) noexcept
 {
-    /// @NOLINTJUSTIFICATION const_cast avoids code duplication, is safe since the constness of the return value is
-    /// restored
-    /// @NOLINTNEXTLINE(cppcoreguidelines-pro-type-const-cast)
+    // NOLINTJUSTIFICATION const_cast avoids code duplication, is safe since the constness of the return value is
+    // restored
+    // NOLINTNEXTLINE(cppcoreguidelines-pro-type-const-cast)
     return const_cast<size_type&>(const_cast<const list<T, Capacity>*>(this)->getNextIdx(idx));
 }
 
@@ -656,17 +659,17 @@ inline const T* list<T, Capacity>::getDataPtrFromIdx(const size_type idx) const
 {
     IOX_ENFORCE(isValidElementIdx(idx), "invalid list element");
 
-    /// @NOLINTJUSTIFICATION provide type safe access to the encapsulated untyped m_data array
-    /// @NOLINTNEXTLINE(cppcoreguidelines-pro-type-reinterpret-cast)
+    // NOLINTJUSTIFICATION provide type safe access to the encapsulated untyped m_data array
+    // NOLINTNEXTLINE(cppcoreguidelines-pro-type-reinterpret-cast)
     return &(reinterpret_cast<const T*>(&m_data)[idx]);
 }
 
 template <typename T, uint64_t Capacity>
 inline T* list<T, Capacity>::getDataPtrFromIdx(const size_type idx) noexcept
 {
-    /// @NOLINTJUSTIFICATION const_cast avoids code duplication, is safe since the constness of the return value is
-    /// restored
-    /// @NOLINTNEXTLINE(cppcoreguidelines-pro-type-const-cast)
+    // NOLINTJUSTIFICATION const_cast avoids code duplication, is safe since the constness of the return value is
+    // restored
+    // NOLINTNEXTLINE(cppcoreguidelines-pro-type-const-cast)
     return const_cast<T*>(const_cast<const list<T, Capacity>*>(this)->getDataPtrFromIdx(idx));
 }
 

iceoryx_hoofs/container/include/iox/detail/vector.inl

@@ -247,6 +247,7 @@ inline bool vector<T, Capacity>::push_back(const T& value) noexcept
 }
 
 template <typename T, uint64_t Capacity>
+// NOLINTNEXTLINE(cppcoreguidelines-rvalue-reference-param-not-moved) perfect forwarding is used
 inline bool vector<T, Capacity>::push_back(T&& value) noexcept
 {
     // AXIVION Next Construct AutosarC++19_03-A18.9.2: we use idiomatic perfect forwarding