Better on_change() argument name
authorMarco Costalba <mcostalba@gmail.com>
Tue, 20 Mar 2012 06:09:58 +0000 (07:09 +0100)
committerMarco Costalba <mcostalba@gmail.com>
Tue, 20 Mar 2012 06:09:58 +0000 (07:09 +0100)
Using "o" as a parameter with the on_xxx(const UICOption& o)
functions is a bit dangerous because of confusion with "0".

Suggested by Rein Halbersma.

No functional change.

Signed-off-by: Marco Costalba <mcostalba@gmail.com>
src/ucioption.cpp

index b4ed59dc2f33bdfe6582376aa0bfb9ec04216c7d..03946e8bd5a0027a7d3a9b37ae4ff17d42925d60 100644 (file)
@@ -33,10 +33,10 @@ OptionsMap Options; // Global object
 namespace {
 
 /// 'On change' actions, triggered by an option's value change
-void on_logger(const UCIOption& o) { logger_set(o); }
+void on_logger(const UCIOption& opt) { logger_set(opt); }
 void on_eval(const UCIOption&) { Eval::init(); }
 void on_threads(const UCIOption&) { Threads.read_uci_options(); }
-void on_hash_size(const UCIOption& o) { TT.set_size(o); }
+void on_hash_size(const UCIOption& opt) { TT.set_size(opt); }
 void on_clear_hash(const UCIOption&) { TT.clear(); }
 
 /// Our case insensitive less() function as required by UCI protocol