From 17868940ee6e977a511cdf88bd2fb1abff2b6d76 Mon Sep 17 00:00:00 2001
From: Igor Krivenko <igor.s.krivenko@gmail.com>
Date: Mon, 13 May 2024 17:06:37 +0200
Subject: [PATCH] clang-tidy: Suppres `modernize-type-traits`

---
 include/libcommute/expression/expression.hpp | 1 +
 include/libcommute/expression/monomial.hpp   | 3 ++-
 include/libcommute/metafunctions.hpp         | 1 +
 include/libcommute/scalar_traits.hpp         | 1 +
 include/libcommute/utility.hpp               | 7 +++++++
 test/check_ordering.hpp                      | 2 ++
 test/factories_dyn.cpp                       | 1 +
 7 files changed, 15 insertions(+), 1 deletion(-)

diff --git a/include/libcommute/expression/expression.hpp b/include/libcommute/expression/expression.hpp
index b01897c..b2a5893 100644
--- a/include/libcommute/expression/expression.hpp
+++ b/include/libcommute/expression/expression.hpp
@@ -65,6 +65,7 @@ template <typename ScalarType, typename... IndexTypes> class expression {
   // Disable overload for expressions
   template <typename T>
   using disable_for_expression =
+      // NOLINTNEXTLINE(modernize-type-traits)
       typename std::enable_if<!is_expression<T>::value>::type;
 
 public:
diff --git a/include/libcommute/expression/monomial.hpp b/include/libcommute/expression/monomial.hpp
index 87ba88b..4ebe323 100644
--- a/include/libcommute/expression/monomial.hpp
+++ b/include/libcommute/expression/monomial.hpp
@@ -40,6 +40,7 @@ template <typename... IndexTypes> class monomial {
   // Helper method for one of constructors
   template <typename GenType1, typename... GenTypesTail>
   void constructor_impl(GenType1&& generator, GenTypesTail&&... more_gens) {
+    // NOLINTNEXTLINE(modernize-type-traits)
     using gen1_t = typename std::remove_reference<GenType1>::type;
     generators_.emplace_back(
         make_unique<gen1_t>(std::forward<GenType1>(generator)));
@@ -62,7 +63,7 @@ template <typename... IndexTypes> class monomial {
 
   // Construct from a list of >=1 generators
   template <typename... GenTypes,
-            typename = typename std::enable_if<
+            typename = typename std::enable_if< // NOLINT(modernize-type-traits)
                 all_are_generators<GenTypes...>::value>::type>
   explicit monomial(GenTypes&&... generators) {
     constructor_impl(std::forward<GenTypes>(generators)...);
diff --git a/include/libcommute/metafunctions.hpp b/include/libcommute/metafunctions.hpp
index 302456b..32f11ca 100644
--- a/include/libcommute/metafunctions.hpp
+++ b/include/libcommute/metafunctions.hpp
@@ -43,6 +43,7 @@ using std::make_unique;
 
 template <typename T> struct remove_cvref {
   using type =
+      // NOLINTNEXTLINE(modernize-type-traits)
       typename std::remove_cv<typename std::remove_reference<T>::type>::type;
 };
 template <typename T> using remove_cvref_t = typename remove_cvref<T>::type;
diff --git a/include/libcommute/scalar_traits.hpp b/include/libcommute/scalar_traits.hpp
index 1a33b24..f79557c 100644
--- a/include/libcommute/scalar_traits.hpp
+++ b/include/libcommute/scalar_traits.hpp
@@ -36,6 +36,7 @@ template <typename T> struct is_complex<std::complex<T>> : std::true_type {};
 
 // Enable template instantiation if Trait<T>::value is true
 template <template <typename> class Trait, typename T>
+// NOLINTNEXTLINE(modernize-type-traits)
 using with_trait = typename std::enable_if<Trait<T>::value>::type;
 
 // Traits of types that can be used as the ScalarType parameter of `expression`.
diff --git a/include/libcommute/utility.hpp b/include/libcommute/utility.hpp
index 143ef52..9ce428e 100644
--- a/include/libcommute/utility.hpp
+++ b/include/libcommute/utility.hpp
@@ -64,12 +64,16 @@ namespace detail {
 
 template <typename Base, typename T, typename... Tail>
 struct all_derived_from_impl {
+  // NOLINTNEXTLINE(modernize-type-traits)
   using T_ = typename std::remove_reference<T>::type;
+  // NOLINTNEXTLINE(modernize-type-traits)
   static constexpr bool value = std::is_base_of<Base, T_>::value &&
                                 all_derived_from_impl<Base, Tail...>::value;
 };
 template <typename Base, typename T> struct all_derived_from_impl<Base, T> {
+  // NOLINTNEXTLINE(modernize-type-traits)
   using T_ = typename std::remove_reference<T>::type;
+  // NOLINTNEXTLINE(modernize-type-traits)
   static constexpr bool value = std::is_base_of<Base, T_>::value;
 };
 
@@ -88,11 +92,13 @@ namespace detail {
 template <typename T, typename TypeListHead, typename... TypeListTail>
 struct not_in_type_list_impl {
   static constexpr bool value =
+      // NOLINTNEXTLINE(modernize-type-traits)
       (!std::is_same<T, TypeListHead>::value) &&
       not_in_type_list_impl<T, TypeListTail...>::value;
 };
 template <typename T, typename TypeListHead>
 struct not_in_type_list_impl<T, TypeListHead> {
+  // NOLINTNEXTLINE(modernize-type-traits)
   static constexpr bool value = !std::is_same<T, TypeListHead>::value;
 };
 
@@ -149,6 +155,7 @@ struct noncopyable {
 //
 
 template <typename T>
+// NOLINTNEXTLINE(modernize-type-traits)
 struct linear_function : std::conditional<std::is_copy_constructible<T>::value,
                                           copyable,
                                           noncopyable>::type {
diff --git a/test/check_ordering.hpp b/test/check_ordering.hpp
index cb83483..0af952c 100644
--- a/test/check_ordering.hpp
+++ b/test/check_ordering.hpp
@@ -47,6 +47,7 @@ void check_equality_impl(std::vector<T> const& v, std::false_type) {
 template <typename T> void check_equality(std::vector<T> const& v) {
   check_equality_impl(
       v,
+      // NOLINTNEXTLINE(modernize-type-traits)
       std::integral_constant<bool, std::is_pointer<T>::value>());
 }
 
@@ -77,6 +78,7 @@ void check_less_greater_impl(std::vector<T> const& v, std::false_type) {
 template <typename T> void check_less_greater(std::vector<T> const& v) {
   check_less_greater_impl(
       v,
+      // NOLINTNEXTLINE(modernize-type-traits)
       std::integral_constant<bool, std::is_pointer<T>::value>());
 }
 
diff --git a/test/factories_dyn.cpp b/test/factories_dyn.cpp
index 8cd7ed6..f6c5dce 100644
--- a/test/factories_dyn.cpp
+++ b/test/factories_dyn.cpp
@@ -41,6 +41,7 @@ TEST_CASE("Factory functions", "[factories]") {
 
   SECTION("make_complex()") {
     auto expr = make_complex(4.0 * c_dag(1, "up") * c(2, "dn") + 1.0);
+    // NOLINTNEXTLINE(modernize-type-traits)
     CHECK(std::is_same<decltype(expr),
                        expression<std::complex<double>, dyn_indices>>::value);
     CHECK(expr == std::complex<double>(4, 0) * c_dag(1, "up") * c(2, "dn") +