From 8dac17555880912672dea605167dbf1bb01ef4f5 Mon Sep 17 00:00:00 2001
From: Danny Robson <danny@nerdcruft.net>
Date: Fri, 19 May 2017 15:23:10 +1000
Subject: [PATCH] algo/sort: don't std::forward args we're going to reuse

---
 algo/sort.ipp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/algo/sort.ipp b/algo/sort.ipp
index cd31f815..f48a8e2f 100644
--- a/algo/sort.ipp
+++ b/algo/sort.ipp
@@ -54,7 +54,7 @@ namespace cruft::util::sort::detail {
     index_swap (IndexA a, IndexB b, RandomIt value, Tail ...tail)
     {
         index_swap (a, b, value);
-        index_swap (a, b, std::forward<Tail> (tail)...);
+        index_swap (a, b, tail...);
     };
 }
 
@@ -93,7 +93,7 @@ cruft::util::sort::reorder (IndexIt idx_first,
         while (i != (decltype(size))idx_first[i]) {
             auto j = idx_first[i];
 
-            detail::index_swap (i, j, value, std::forward<OtherIt> (tail)..., idx_first);
+            detail::index_swap (i, j, value, tail..., idx_first);
         }
     }
 };
@@ -151,5 +151,5 @@ cruft::util::sort::soa (RandomIt key_first,
     }
 
     // reorder all the arrays using the mapping we have discovered.
-    reorder (std::begin (dest), std::end (dest), key_first, std::forward<Args> (values)...);
+    reorder (std::begin (dest), std::end (dest), key_first, values...);
 };