[libsigcplusplus/c++17v3] C++17: limit_trackable_target: Use constexpr if to simplify code.



commit 8c417786504c4d0dc909b9ccbd6f457b71a083ef
Author: Murray Cumming <murrayc murrayc com>
Date:   Thu Jul 7 22:36:07 2016 +0200

    C++17: limit_trackable_target: Use constexpr if to simplify code.

 sigc++/visit_each.h |   27 +++------------------------
 1 files changed, 3 insertions(+), 24 deletions(-)
---
diff --git a/sigc++/visit_each.h b/sigc++/visit_each.h
index e28d385..51b53aa 100644
--- a/sigc++/visit_each.h
+++ b/sigc++/visit_each.h
@@ -42,10 +42,10 @@ struct limit_trackable_target
   template <typename T_type>
   void operator()(T_type&& type) const
   {
-    using T_self = limit_trackable_target<T_action>;
-
     //Only call action_() if T_Type derives from trackable.
-    with_type<T_type, T_self>::execute_(std::forward<T_type>(type), *this);
+    if constexpr(is_base_of_or_same_v<sigc::trackable, T_type>) {
+        action_(type);
+    }
   }
 
   explicit limit_trackable_target(const T_action& action) : action_(action) {}
@@ -56,27 +56,6 @@ struct limit_trackable_target
   limit_trackable_target& operator=(limit_trackable_target&& src) = delete;
 
   T_action action_;
-
-private:
-  template <typename T_type, typename T_limit, bool I_derived = is_base_of_or_same_v<sigc::trackable, 
T_type>>
-  struct with_type;
-
-  // Specialization for I_derived = false
-  template <typename T_type, typename T_limit>
-  struct with_type<T_type, T_limit, false>
-  {
-    static void execute_(const T_type&, const T_limit&) {}
-  };
-
-  // Specialization for I_derived = true
-  template <typename T_type, typename T_limit>
-  struct with_type<T_type, T_limit, true>
-  {
-    static void execute_(const T_type& type, const T_limit& action)
-    {
-      action.action_(type);
-    }
-  };
 };
 
 } /* namespace internal */


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]