diff options
author | HybridDog <3192173+HybridDog@users.noreply.github.com> | 2020-04-22 00:07:12 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-04-22 00:07:12 +0200 |
commit | 4361bfcb4da0b6f9de74c7de9f2d08084877713e (patch) | |
tree | bf2dd19f35560aecdc8c63457b885b18a73a9aeb /src | |
parent | 8ef239b448c52485cf94d334c1d8b1c6de37d976 (diff) | |
download | minetest-4361bfcb4da0b6f9de74c7de9f2d08084877713e.tar.gz minetest-4361bfcb4da0b6f9de74c7de9f2d08084877713e.tar.bz2 minetest-4361bfcb4da0b6f9de74c7de9f2d08084877713e.zip |
Fix configuration caching in log_deprecated (#9697)
* Fix configuration caching in log_deprecated
The configured variable was never set to true.
I've set the variables to thread_local because the configuration should be reloaded after reentering the world from mainmenu.
Diffstat (limited to 'src')
-rw-r--r-- | src/script/common/c_internal.cpp | 9 | ||||
-rw-r--r-- | src/script/lua_api/l_util.cpp | 2 | ||||
-rw-r--r-- | src/script/lua_api/l_util.h | 2 |
3 files changed, 7 insertions, 6 deletions
diff --git a/src/script/common/c_internal.cpp b/src/script/common/c_internal.cpp index b19af9f82..6df1f8b7b 100644 --- a/src/script/common/c_internal.cpp +++ b/src/script/common/c_internal.cpp @@ -157,9 +157,9 @@ static void script_log(lua_State *L, const std::string &message, void log_deprecated(lua_State *L, const std::string &message, int stack_depth) { - static bool configured = false; - static bool do_log = false; - static bool do_error = false; + static thread_local bool configured = false; + static thread_local bool do_log = false; + static thread_local bool do_error = false; // Only read settings on first call if (!configured) { @@ -167,9 +167,10 @@ void log_deprecated(lua_State *L, const std::string &message, int stack_depth) if (value == "log") { do_log = true; } else if (value == "error") { - do_log = true; + do_log = true; do_error = true; } + configured = true; } if (do_log) diff --git a/src/script/lua_api/l_util.cpp b/src/script/lua_api/l_util.cpp index ae3e5df3d..28ee39fc8 100644 --- a/src/script/lua_api/l_util.cpp +++ b/src/script/lua_api/l_util.cpp @@ -44,7 +44,7 @@ with this program; if not, write to the Free Software Foundation, Inc., // log([level,] text) // Writes a line to the logger. -// The one-argument version logs to infostream. +// The one-argument version logs to LL_NONE. // The two-argument version accepts a log level. // Either the special case "deprecated" for deprecation notices, or any specified in // Logger::stringToLevel(name). diff --git a/src/script/lua_api/l_util.h b/src/script/lua_api/l_util.h index 5697aab15..9ff91bb53 100644 --- a/src/script/lua_api/l_util.h +++ b/src/script/lua_api/l_util.h @@ -37,7 +37,7 @@ private: // log([level,] text) // Writes a line to the logger. - // The one-argument version logs to infostream. + // The one-argument version logs to LL_NONE. // The two-argument version accepts a log level. static int l_log(lua_State *L); |