More idiomatic signature for operator=()

[stockfish] / src / ucioption.cpp

diff --git a/src/ucioption.cpp b/src/ucioption.cpp
index 763db556654e4c87828854bc7290d056ba8038da..58f3dbc0c663a6cd0412f913fd556191048d1c2e 100644 (file)
--- a/src/ucioption.cpp
+++ b/src/ucioption.cpp
@@ -134,18 +134,20 @@ UCIOption::UCIOption(int v, int minv, int maxv, Fn* f) : type("spin"), min(minv)
 /// 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)))
-      return;
+      return *this;
 
   if (type != "button")
       currentValue = v;
 
   if (on_change)
       (*on_change)(*this);
+
+  return *this;
 }