kate/BadFish
1
0
Fork 0
mirror of https://github.com/sockspls/badfish synced 2025-04-30 16:53:09 +00:00
Code Issues Releases Wiki Activity

Check bounds in set_option_value()

Browse source 
Normally it's up to the GUI to check for option's limits,
but we could receive the new value directly from the user
by teminal window. So let's check the bounds anyway.

No functional change.

Signed-off-by: Marco Costalba <mcostalba@gmail.com>
This commit is contained in:
Marco Costalba 2010-02-01 14:06:59 +01:00
parent 3941e4bdb3
commit 8008f78415
1 changed files with 21 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

24
src/ucioption.cpp
View file

@ -293,10 +293,28 @@ void set_option_value(const string& name, const string& value) {
else if (v == "false") else if (v == "false")
v = "0"; v = "0";
if (options.find(name) != options.end()) if (options.find(name) == options.end())
options[name].currentValue = v; {
else
std::cout << "No such option: " << name << std::endl; std::cout << "No such option: " << name << std::endl;
return;
}
// Normally it's up to the GUI to check for option's limits,
// but we could receive the new value directly from the user
// by teminal window. So let's check the bounds anyway.
Option& opt = options[name];
if (opt.type == CHECK && v != "0" && v != "1")
return;
else if (opt.type == SPIN)
{
int val = atoi(v.c_str());
if (val < opt.minValue || val > opt.maxValue)
return;
}
opt.currentValue = v;
} }