Addresses some clang-tidy warnings against reference data members

louis-langholtz · May 12, 2023 · 2db57cd · 2db57cd
1 parent 977c2b6
commit 2db57cd
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/PlayRho/Common/FlagGuard.hpp b/PlayRho/Common/FlagGuard.hpp
@@ -64,8 +64,11 @@ namespace playrho {
         FlagGuard() = delete;
 
     private:
-        T& m_flag; ///< Flag.
-        T m_value; ///< Value.
+        /// @brief Flag.
+        T& m_flag; // NOLINT(cppcoreguidelines-avoid-const-or-ref-data-members)
+
+        /// @brief Value.
+        T m_value;
     };
 
 } // namespace playrho

diff --git a/PlayRho/Common/Templates.hpp b/PlayRho/Common/Templates.hpp
@@ -258,7 +258,8 @@ using IsReverseIterable = typename detail::IsReverseIterableImpl<T>;
 /// @see https://stackoverflow.com/a/28139075/7410358
 template <typename T>
 struct ReversionWrapper {
-    T& iterable; ///< Reference to underlying iterable.
+    /// @brief Reference to underlying iterable.
+    T& iterable; // NOLINT(cppcoreguidelines-avoid-const-or-ref-data-members)
 };
 
 /// @brief Begin function for getting a reversed order iterator.