From: mstembera <MissingEmail@email>
Date: Mon, 8 May 2017 03:14:23 +0000 (-0700)
Subject: Avoid *begin always being included in the sorted list regardless of its value.
X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=321a27fbe3b5af9961b99585ddd365a9cc855e38

Avoid *begin always being included in the sorted list regardless of its value.

This was a minor criticism by @zamar in the original pull request
https://github.com/official-stockfish/Stockfish/pull/1065
necessitating a comment explanation.

No functional change.

Closes #1091
---

diff --git a/src/movepick.cpp b/src/movepick.cpp
index 992657e7..5c246664 100644
--- a/src/movepick.cpp
+++ b/src/movepick.cpp
@@ -36,18 +36,16 @@ namespace {
 
   // partial_insertion_sort() sorts moves in descending order up to and including
   // a given limit. The order of moves smaller than the limit is left unspecified.
-  // To keep the implementation simple, *begin is always included in the sorted moves.
   void partial_insertion_sort(ExtMove* begin, ExtMove* end, int limit) {
 
-    for (ExtMove *sortedEnd = begin + 1, *p = begin + 1; p < end; ++p)
+    for (ExtMove *sortedEnd = begin, *p = begin + 1; p < end; ++p)
         if (p->value >= limit)
         {
             ExtMove tmp = *p, *q;
-            *p = *sortedEnd;
-            for (q = sortedEnd; q != begin && *(q-1) < tmp; --q)
-                *q = *(q-1);
+            *p = *++sortedEnd;
+            for (q = sortedEnd; q != begin && *(q - 1) < tmp; --q)
+                *q = *(q - 1);
             *q = tmp;
-            ++sortedEnd;
         }
   }