}
bool CaseInsensitiveLess::operator() (const string& s1, const string& s2) const {
- return lexicographical_compare(s1.begin(), s1.end(), s2.begin(), s2.end(), ci_less);
+ return std::lexicographical_compare(s1.begin(), s1.end(), s2.begin(), s2.end(), ci_less);
}
/// check for option's limits, but we could receive the new value directly from
/// the user by console window, so let's check the bounds anyway.
-void UCIOption::operator=(const string& v) {
+UCIOption& UCIOption::operator=(const string& v) {
assert(!type.empty());
- if ( (type == "button" || !v.empty())
- && (type != "check" || (v == "true" || v == "false"))
- && (type != "spin" || (atoi(v.c_str()) >= min && atoi(v.c_str()) <= max)))
- {
- if (type != "button")
- currentValue = v;
+ if ( (type != "button" && v.empty())
+ || (type == "check" && v != "true" && v != "false")
+ || (type == "spin" && (atoi(v.c_str()) < min || atoi(v.c_str()) > max)))
+ return *this;
- if (on_change)
- (*on_change)(*this);
- }
+ if (type != "button")
+ currentValue = v;
+
+ if (on_change)
+ (*on_change)(*this);
+
+ return *this;
}