Refactor Script API's log_deprecated

This commit is contained in:
sfan5 2020-02-12 19:17:04 +01:00
parent 4da057cc49
commit c657fb343f
6 changed files with 27 additions and 31 deletions

@ -2364,7 +2364,7 @@ bool GUIFormSpecMenu::parseSizeDirect(parserData* data, const std::string &eleme
return false; return false;
if (type == "invsize") if (type == "invsize")
log_deprecated("Deprecated formspec element \"invsize\" is used"); warningstream << "Deprecated formspec element \"invsize\" is used" << std::endl;
parseSize(data, description); parseSize(data, description);

@ -135,7 +135,27 @@ void script_run_callbacks_f(lua_State *L, int nargs,
lua_remove(L, error_handler); lua_remove(L, error_handler);
} }
void log_deprecated(lua_State *L, const std::string &message) static void script_log(lua_State *L, const std::string &message,
std::ostream &log_to, bool do_error, int stack_depth)
{
lua_Debug ar;
log_to << message << " ";
if (lua_getstack(L, stack_depth, &ar)) {
FATAL_ERROR_IF(!lua_getinfo(L, "Sl", &ar), "lua_getinfo() failed");
log_to << "(at " << ar.short_src << ":" << ar.currentline << ")";
} else {
log_to << "(at ?:?)";
}
log_to << std::endl;
if (do_error)
script_error(L, LUA_ERRRUN, NULL, NULL);
else
infostream << script_get_backtrace(L) << std::endl;
}
void log_deprecated(lua_State *L, const std::string &message, int stack_depth)
{ {
static bool configured = false; static bool configured = false;
static bool do_log = false; static bool do_log = false;
@ -152,24 +172,6 @@ void log_deprecated(lua_State *L, const std::string &message)
} }
} }
if (do_log) { if (do_log)
warningstream << message; script_log(L, message, warningstream, do_error, stack_depth);
if (L) { // L can be NULL if we get called from scripting_game.cpp
lua_Debug ar;
if (!lua_getstack(L, 2, &ar))
FATAL_ERROR_IF(!lua_getstack(L, 1, &ar), "lua_getstack() failed");
FATAL_ERROR_IF(!lua_getinfo(L, "Sl", &ar), "lua_getinfo() failed");
warningstream << " (at " << ar.short_src << ":" << ar.currentline << ")";
}
warningstream << std::endl;
if (L) {
if (do_error)
script_error(L, LUA_ERRRUN, NULL, NULL);
else
infostream << script_get_backtrace(L) << std::endl;
}
}
} }

@ -103,4 +103,5 @@ int script_exception_wrapper(lua_State *L, lua_CFunction f);
void script_error(lua_State *L, int pcall_result, const char *mod, const char *fxn); void script_error(lua_State *L, int pcall_result, const char *mod, const char *fxn);
void script_run_callbacks_f(lua_State *L, int nargs, void script_run_callbacks_f(lua_State *L, int nargs,
RunCallbacksMode mode, const char *fxn); RunCallbacksMode mode, const char *fxn);
void log_deprecated(lua_State *L, const std::string &message); void log_deprecated(lua_State *L, const std::string &message,
int stack_depth=1);

@ -59,7 +59,7 @@ int ModApiUtil::l_log(lua_State *L)
std::string name = luaL_checkstring(L, 1); std::string name = luaL_checkstring(L, 1);
text = luaL_checkstring(L, 2); text = luaL_checkstring(L, 2);
if (name == "deprecated") { if (name == "deprecated") {
log_deprecated(L, text); log_deprecated(L, text, 2);
return 0; return 0;
} }
level = Logger::stringToLevel(name); level = Logger::stringToLevel(name);

@ -121,8 +121,3 @@ void ServerScripting::InitializeModApi(lua_State *L, int top)
ModApiStorage::Initialize(L, top); ModApiStorage::Initialize(L, top);
ModApiChannels::Initialize(L, top); ModApiChannels::Initialize(L, top);
} }
void log_deprecated(const std::string &message)
{
log_deprecated(NULL, message);
}

@ -51,5 +51,3 @@ public:
private: private:
void InitializeModApi(lua_State *L, int top); void InitializeModApi(lua_State *L, int top);
}; };
void log_deprecated(const std::string &message);