Add fixes for cppcheck

43dc4ccf · Paul · 97d5b3ca · 43dc4ccf · 43dc4ccf
Commit 43dc4ccf authored Jun 19, 2018 by Paul
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 1 deletion

CMakeLists.txt CMakeLists.txt +2 -0

src/include/rtg/requires.hpp src/include/rtg/requires.hpp +5 -1

No files found.
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -106,6 +106,8 @@ rocm_enable_cppcheck(
        ${CMAKE_CURRENT_SOURCE_DIR}/src/include
        ${CMAKE_CURRENT_SOURCE_DIR}/src/targets/cpu/include
        ${CMAKE_CURRENT_SOURCE_DIR}/src/targets/miopen/include
+    DEFINE
+        CPPCHECK=1
 )

 add_subdirectory(src)

--- a/src/include/rtg/requires.hpp
+++ b/src/include/rtg/requires.hpp
@@ -6,11 +6,15 @@
 namespace rtg {

 template <bool... Bs>
-struct and_ : std::is_same<and_<Bs...>, and_<(Bs || true)...>>
+struct and_ : std::is_same<and_<Bs...>, and_<(Bs || true)...>> // NOLINT
 {
 };

+#ifdef CPPCHECK
+#define RTG_REQUIRES(...) class=void
+#else
 #define RTG_REQUIRES(...) class = typename std::enable_if<and_<__VA_ARGS__, true>{}>::type
+#endif

 } // namespace rtg