From: Marco Costalba <mcostalba@gmail.com>
Date: Tue, 20 Mar 2012 06:09:58 +0000 (+0100)
Subject: Better on_change() argument name
X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=258da28e79d99f75e0b626697bda2d459a37c0e6;hp=df80232495f9500c7d6ed197e40cebbc77175f07

Better on_change() argument name

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>
---

diff --git a/src/ucioption.cpp b/src/ucioption.cpp
index b4ed59dc..03946e8b 100644
--- a/src/ucioption.cpp
+++ b/src/ucioption.cpp
@@ -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