Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Remove blank default values for emergequeue_limit_* settings
Add checks for invalid user input for important settings
Rename Settings::tryGet* to Settings::get*NoEx for consistency
  • Loading branch information
kwolekr committed Feb 5, 2014
1 parent c6c5edd commit 7f74317
Show file tree
Hide file tree
Showing 6 changed files with 32 additions and 26 deletions.
2 changes: 0 additions & 2 deletions src/defaultsettings.cpp
Expand Up @@ -219,8 +219,6 @@ void set_default_settings(Settings *settings)
settings->setDefault("remote_media", "");
settings->setDefault("debug_log_level", "2");
settings->setDefault("emergequeue_limit_total", "256");
settings->setDefault("emergequeue_limit_diskonly", "");
settings->setDefault("emergequeue_limit_generate", "");
settings->setDefault("num_emerge_threads", "1");

// physics stuff
Expand Down
24 changes: 16 additions & 8 deletions src/emerge.cpp
Expand Up @@ -102,17 +102,25 @@ EmergeManager::EmergeManager(IGameDef *gamedef) {
// if unspecified, leave a proc for the main thread and one for
// some other misc thread
int nthreads = 0;
if (!g_settings->tryGetS16("num_emerge_threads", nthreads))
if (!g_settings->getS16NoEx("num_emerge_threads", nthreads))
nthreads = porting::getNumberOfProcessors() - 2;
if (nthreads < 1)
nthreads = 1;

qlimit_total = g_settings->getU16("emergequeue_limit_total");
if (!g_settings->tryGetU16("emergequeue_limit_diskonly", qlimit_diskonly))
if (!g_settings->getU16NoEx("emergequeue_limit_diskonly", qlimit_diskonly))
qlimit_diskonly = nthreads * 5 + 1;
if (!g_settings->tryGetU16("emergequeue_limit_generate", qlimit_generate))
if (!g_settings->getU16NoEx("emergequeue_limit_generate", qlimit_generate))
qlimit_generate = nthreads + 1;

// don't trust user input for something very important like this
if (qlimit_total < 1)
qlimit_total = 1;
if (qlimit_diskonly < 1)
qlimit_diskonly = 1;
if (qlimit_generate < 1)
qlimit_generate = 1;

for (int i = 0; i != nthreads; i++)
emergethread.push_back(new EmergeThread((Server *)gamedef, i));

Expand Down Expand Up @@ -352,13 +360,13 @@ void EmergeManager::loadParamsFromSettings(Settings *settings) {
std::string seed_str;
const char *setname = (settings == g_settings) ? "fixed_map_seed" : "seed";

if (settings->tryGet(setname, seed_str))
if (settings->getNoEx(setname, seed_str))
params.seed = read_seed(seed_str.c_str());

settings->tryGet("mg_name", params.mg_name);
settings->tryGetS16("water_level", params.water_level);
settings->tryGetS16("chunksize", params.chunksize);
settings->tryGetFlagStr("mg_flags", params.flags, flagdesc_mapgen);
settings->getNoEx("mg_name", params.mg_name);
settings->getS16NoEx("water_level", params.water_level);
settings->getS16NoEx("chunksize", params.chunksize);
settings->getFlagStrNoEx("mg_flags", params.flags, flagdesc_mapgen);

delete params.sparams;
params.sparams = createMapgenParams(params.mg_name);
Expand Down
2 changes: 1 addition & 1 deletion src/mapgen_indev.cpp
Expand Up @@ -164,7 +164,7 @@ MapgenIndevParams::MapgenIndevParams() {
void MapgenIndevParams::readParams(Settings *settings) {
MapgenV6Params::readParams(settings);

settings->tryGetS16("mgindev_float_islands", float_islands);
settings->getS16NoEx("mgindev_float_islands", float_islands);

settings->getNoiseIndevParams("mgindev_np_terrain_base", npindev_terrain_base);
settings->getNoiseIndevParams("mgindev_np_terrain_higher", npindev_terrain_higher);
Expand Down
6 changes: 3 additions & 3 deletions src/mapgen_v6.cpp
Expand Up @@ -112,9 +112,9 @@ MapgenV6Params::MapgenV6Params() {


void MapgenV6Params::readParams(Settings *settings) {
settings->tryGetFlagStr("mgv6_spflags", spflags, flagdesc_mapgen_v6);
settings->tryGetFloat("mgv6_freq_desert", freq_desert);
settings->tryGetFloat("mgv6_freq_beach", freq_beach);
settings->getFlagStrNoEx("mgv6_spflags", spflags, flagdesc_mapgen_v6);
settings->getFloatNoEx("mgv6_freq_desert", freq_desert);
settings->getFloatNoEx("mgv6_freq_beach", freq_beach);

settings->getNoiseParams("mgv6_np_terrain_base", np_terrain_base);
settings->getNoiseParams("mgv6_np_terrain_higher", np_terrain_higher);
Expand Down
2 changes: 1 addition & 1 deletion src/mapgen_v7.cpp
Expand Up @@ -126,7 +126,7 @@ MapgenV7Params::MapgenV7Params() {


void MapgenV7Params::readParams(Settings *settings) {
settings->tryGetFlagStr("mgv7_spflags", spflags, flagdesc_mapgen_v7);
settings->getFlagStrNoEx("mgv7_spflags", spflags, flagdesc_mapgen_v7);

settings->getNoiseParams("mgv7_np_terrain_base", np_terrain_base);
settings->getNoiseParams("mgv7_np_terrain_alt", np_terrain_alt);
Expand Down
22 changes: 11 additions & 11 deletions src/settings.h
Expand Up @@ -745,7 +745,7 @@ class Settings
}

//////////// Try to get value, no exception thrown
bool tryGet(std::string name, std::string &val)
bool getNoEx(std::string name, std::string &val)
{
try {
val = get(name);
Expand All @@ -755,7 +755,7 @@ class Settings
}
}

bool tryGetFlagStr(std::string name, u32 &val, FlagDesc *flagdesc)
bool getFlagStrNoEx(std::string name, u32 &val, FlagDesc *flagdesc)
{
try {
val = getFlagStr(name, flagdesc);
Expand All @@ -765,7 +765,7 @@ class Settings
}
}

bool tryGetFloat(std::string name, float &val)
bool getFloatNoEx(std::string name, float &val)
{
try {
val = getFloat(name);
Expand All @@ -775,7 +775,7 @@ class Settings
}
}

bool tryGetU16(std::string name, int &val)
bool getU16NoEx(std::string name, int &val)
{
try {
val = getU16(name);
Expand All @@ -785,7 +785,7 @@ class Settings
}
}

bool tryGetU16(std::string name, u16 &val)
bool getU16NoEx(std::string name, u16 &val)
{
try {
val = getU16(name);
Expand All @@ -795,7 +795,7 @@ class Settings
}
}

bool tryGetS16(std::string name, int &val)
bool getS16NoEx(std::string name, int &val)
{
try {
val = getU16(name);
Expand All @@ -805,7 +805,7 @@ class Settings
}
}

bool tryGetS16(std::string name, s16 &val)
bool getS16NoEx(std::string name, s16 &val)
{
try {
val = getS16(name);
Expand All @@ -815,7 +815,7 @@ class Settings
}
}

bool tryGetS32(std::string name, s32 &val)
bool getS32NoEx(std::string name, s32 &val)
{
try {
val = getS32(name);
Expand All @@ -825,7 +825,7 @@ class Settings
}
}

bool tryGetV3F(std::string name, v3f &val)
bool getV3FNoEx(std::string name, v3f &val)
{
try {
val = getV3F(name);
Expand All @@ -835,7 +835,7 @@ class Settings
}
}

bool tryGetV2F(std::string name, v2f &val)
bool getV2FNoEx(std::string name, v2f &val)
{
try {
val = getV2F(name);
Expand All @@ -845,7 +845,7 @@ class Settings
}
}

bool tryGetU64(std::string name, u64 &val)
bool getU64NoEx(std::string name, u64 &val)
{
try {
val = getU64(name);
Expand Down

0 comments on commit 7f74317

Please sign in to comment.