From: Adam C. Emerson <aemerson@redhat.com>
Date: Wed, 10 Jan 2018 04:29:06 +0000 (-0500)
Subject: denc: Clean up tuples even more
X-Git-Tag: wip-pdonnell-testing-20180317.202121~568^2
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=98572fba64141287f116e21b2db913f6ff6a7168;p=ceph-ci.git

denc: Clean up tuples even more

Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
---

diff --git a/src/include/denc.h b/src/include/denc.h
index 9b291736899..325bada52bf 100644
--- a/src/include/denc.h
+++ b/src/include/denc.h
@@ -46,6 +46,8 @@
 #include "buffer.h"
 #include "byteorder.h"
 
+#include "common/convenience.h"
+
 template<typename T, typename=void>
 struct denc_traits {
   static constexpr bool supported = false;
@@ -1120,46 +1122,6 @@ private:
 		"Zero-length tuples are not supported.");
   using container = std::tuple<Ts...>;
 
-  template<typename U = container, size_t... Is>
-  static std::enable_if_t<denc_traits<U>::featured>
-  bound_encode_helper(const U& s, size_t& p,
-		      std::uint64_t f,
-		      std::index_sequence<Is...>) {
-    (denc(std::get<Is>(s), p, f), ..., void());
-  }
-  template<typename U = container, size_t... Is>
-  static std::enable_if_t<!denc_traits<U>::featured>
-  bound_encode_helper(const U& s, size_t& p,
-		      std::index_sequence<Is...>) {
-    (denc(std::get<Is>(s), p), ..., void());
-  }
-
-  template<typename U = container, size_t... Is>
-  static std::enable_if_t<denc_traits<U>::featured>
-  encode_helper(const U& s, buffer::list::contiguous_appender& p,
-		      std::uint64_t f,
-		      std::index_sequence<Is...>) {
-    (denc(std::get<Is>(s), p, f), ..., void());
-  }
-  template<typename U = container, size_t... Is>
-  static std::enable_if_t<!denc_traits<U>::featured>
-  encode_helper(const U& s, buffer::list::contiguous_appender& p,
-		std::index_sequence<Is...>) {
-    (denc(std::get<Is>(s), p), ..., void());
-  }
-
-  template<size_t... Is>
-  static void decode_helper(container& s, buffer::ptr::iterator& p,
-			    std::index_sequence<Is...>) {
-    (denc(std::get<Is>(s), p), ..., void());
-  }
-
-  template<typename U = container, size_t... Is>
-  static std::enable_if_t<!denc_traits<U>::need_contiguous>
-  decode_helper(U& s, buffer::list::iterator& p, std::index_sequence<Is...>) {
-    (denc(std::get<Is>(s), p), ..., void());
-  }
-
 public:
 
   static constexpr bool supported = true;
@@ -1171,33 +1133,53 @@ public:
   template<typename U = container>
   static std::enable_if_t<denc_traits<U>::featured>
   bound_encode(const container& s, size_t& p, uint64_t f) {
-    bound_encode_helper(s, p, f, std::index_sequence_for<Ts...>{});
+    ceph::for_each(s, [&p, f] (const auto& e) {
+	if constexpr (denc_traits<std::decay_t<decltype(p)>>::featured) {
+	  denc(e, p, f);
+	} else {
+	  denc(e, p);
+	}
+      });
   }
   template<typename U = container>
   static std::enable_if_t<!denc_traits<U>::featured>
   bound_encode(const container& s, size_t& p) {
-    bound_encode_helper(s, p, std::index_sequence_for<Ts...>{});
+    ceph::for_each(s, [&p] (const auto& e) {
+	denc(e, p);
+      });
   }
 
   template<typename U = container>
   static std::enable_if_t<denc_traits<U>::featured>
   encode(const container& s, buffer::list::contiguous_appender& p, uint64_t f) {
-    encode_helper(s, p, f, std::index_sequence_for<Ts...>{});
+    ceph::for_each(s, [&p, f] (const auto& e) {
+	if constexpr (denc_traits<std::decay_t<decltype(p)>>::featured) {
+	  denc(e, p, f);
+	} else {
+	  denc(e, p);
+	}
+      });
   }
   template<typename U = container>
   static std::enable_if_t<!denc_traits<U>::featured>
   encode(const container& s, buffer::list::contiguous_appender& p) {
-    encode_helper(s, p, std::index_sequence_for<Ts...>{});
+    ceph::for_each(s, [&p] (const auto& e) {
+	denc(e, p);
+      });
   }
 
   static void decode(container& s, buffer::ptr::iterator& p, uint64_t f = 0) {
-    decode_helper(s, p, std::index_sequence_for<Ts...>{});
+    ceph::for_each(s, [&p] (auto& e) {
+	denc(e, p);
+      });
   }
 
   template<typename U = container>
   static std::enable_if_t<!denc_traits<U>::need_contiguous>
   decode(container& s, buffer::list::iterator& p, uint64_t f = 0) {
-    decode_helper(s, p, std::index_sequence_for<Ts...>{});
+    ceph::for_each(s, [&p] (auto& e) {
+	denc(e, p);
+      });
   }
 };