From 20a6f99cdb87d5877d490abceb795f7e68c5b4a6 Mon Sep 17 00:00:00 2001 From: Marco Costalba Date: Thu, 29 Dec 2011 18:25:08 +0100 Subject: [PATCH] Fix an off-by-one bug in ucioption.cpp Harmless but anyhow wrong. No functional change. Signed-off-by: Marco Costalba --- src/ucioption.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) 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; -- 2.39.2