Support move-only key types in the Max/MinElementBy routines

It is also slightly faster for non-trivial key types.
commit_hash:4104cabfe8dc9a51174034c62aae25be16b57bf9

1 files changed, 4 insertions, 4 deletions

diff --git a/util/generic/algorithm.h b/util/generic/algorithm.h
index a770ab3033c..d1f48c479d9 100644
--- a/util/generic/algorithm.h
+++ b/util/generic/algorithm.h
@@ -21,11 +21,11 @@ namespace NPrivate {
         auto bestValue = func(*begin);
         auto bestPos = begin;
 
-        for (auto i = ++begin; i != end; ++i) {
-            auto curValue = func(*i);
+        for (++begin; begin != end; ++begin) {
+            auto curValue = func(*begin);
             if (pred(curValue, bestValue)) {
-                bestValue = curValue;
-                bestPos = i;
+                bestValue = std::move(curValue);
+                bestPos = begin;
             }
         }