mirror of
https://github.com/microsoft/mimalloc.git
synced 2025-05-05 06:59:32 +03:00
small rewrite of pr #506
This commit is contained in:
parent
6424cfab41
commit
1ca48210fb
1 changed files with 8 additions and 9 deletions
|
@ -545,19 +545,18 @@ static void mi_option_init(mi_option_desc_t* desc) {
|
||||||
desc->init = INITIALIZED;
|
desc->init = INITIALIZED;
|
||||||
}
|
}
|
||||||
else {
|
else {
|
||||||
/* _mi_warning_message() will itself call mi_option_get() for some options,
|
// set `init` first to avoid recursion through _mi_warning_message on mimalloc_verbose.
|
||||||
* so to avoid a possible infinite recursion it's important to mark the option as
|
|
||||||
* "initialized" first */
|
|
||||||
desc->init = DEFAULTED;
|
desc->init = DEFAULTED;
|
||||||
if (desc->option == mi_option_verbose) {
|
if (desc->option == mi_option_verbose && desc->value == 0) {
|
||||||
/* Special case: if the 'mimalloc_verbose' env var has a bogus value we'd never know
|
// if the 'mimalloc_verbose' env var has a bogus value we'd never know
|
||||||
* (since the value default to 'off') - so in that one case briefly set the option to 'on' */
|
// (since the value defaults to 'off') so in that case briefly enable verbose
|
||||||
desc->value = 1;
|
desc->value = 1;
|
||||||
}
|
_mi_warning_message("environment option mimalloc_%s has an invalid value: %s\n", desc->name, buf);
|
||||||
_mi_warning_message("environment option mimalloc_%s has an invalid value: %s\n", desc->name, buf);
|
|
||||||
if (desc->option == mi_option_verbose) {
|
|
||||||
desc->value = 0;
|
desc->value = 0;
|
||||||
}
|
}
|
||||||
|
else {
|
||||||
|
_mi_warning_message("environment option mimalloc_%s has an invalid value: %s\n", desc->name, buf);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
mi_assert_internal(desc->init != UNINIT);
|
mi_assert_internal(desc->init != UNINIT);
|
||||||
|
|
Loading…
Add table
Reference in a new issue