From: Marco Costalba <mcostalba@gmail.com>
Date: Thu, 29 Dec 2011 17:25:08 +0000 (+0100)
Subject: Fix an off-by-one bug in ucioption.cpp
X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=20a6f99cdb87d5877d490abceb795f7e68c5b4a6

Fix an off-by-one bug in ucioption.cpp

Harmless but anyhow wrong.

No functional change.

Signed-off-by: Marco Costalba <mcostalba@gmail.com>
---

diff --git a/src/ucioption.cpp b/src/ucioption.cpp
index 8ea4c5f6..6b066f22 100644
--- a/src/ucioption.cpp
+++ b/src/ucioption.cpp
@@ -81,9 +81,9 @@ OptionsMap::OptionsMap() {
 /// order (the idx field) and in the format defined by the UCI protocol.
 std::ostream& operator<<(std::ostream& os, const OptionsMap& om) {
 
-  for (size_t i = 0; i <= om.size(); i++)
+  for (size_t idx = 0; idx < om.size(); idx++)
       for (OptionsMap::const_iterator it = om.begin(); it != om.end(); ++it)
-          if (it->second.idx == i)
+          if (it->second.idx == idx)
           {
               const UCIOption& o = it->second;
               os << "\noption name " << it->first << " type " << o.type;