From 258da28e79d99f75e0b626697bda2d459a37c0e6 Mon Sep 17 00:00:00 2001 From: Marco Costalba Date: Tue, 20 Mar 2012 07:09:58 +0100 Subject: [PATCH] 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 --- src/ucioption.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) 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 -- 2.39.2