From 2350d2aded7101cf8ba60af0de7cbf0df8ef964b Mon Sep 17 00:00:00 2001
From: John Spray <john.spray@redhat.com>
Date: Wed, 28 Jun 2017 09:27:07 -0400
Subject: [PATCH] encoding: remove encode_array_nohead

This was just a for loop.  No longer needed for
MonCommands, and the usage in memstore/PageSet
was just iterating over char* and should never have
been there to begin with.

Signed-off-by: John Spray <john.spray@redhat.com>
---
 src/include/encoding.h    | 16 ----------------
 src/os/memstore/PageSet.h |  2 +-
 2 files changed, 1 insertion(+), 17 deletions(-)

diff --git a/src/include/encoding.h b/src/include/encoding.h
index 0013728efed..f7ee5d11828 100644
--- a/src/include/encoding.h
+++ b/src/include/encoding.h
@@ -197,22 +197,6 @@ inline void encode(const char *s, bufferlist& bl)
 }
 
 
-// array
-template<class A>
-inline void encode_array_nohead(const A a[], int n, bufferlist &bl)
-{
-  for (int i=0; i<n; i++)
-    encode(a[i], bl);
-}
-template<class A>
-inline void decode_array_nohead(A a[], int n, bufferlist::iterator &p)
-{
-  for (int i=0; i<n; i++)
-    decode(a[i], p);
-}
-
-
-
 // -----------------------------
 // buffers
 
diff --git a/src/os/memstore/PageSet.h b/src/os/memstore/PageSet.h
index e989a361072..a40b86425ec 100644
--- a/src/os/memstore/PageSet.h
+++ b/src/os/memstore/PageSet.h
@@ -58,7 +58,7 @@ struct Page {
     ::encode(offset, bl);
   }
   void decode(bufferlist::iterator &p, size_t page_size) {
-    ::decode_array_nohead(data, page_size, p);
+    p.copy(page_size, data);
     ::decode(offset, p);
   }
 
-- 
2.47.3