Stowage/godot
2
0
Fork 0
mirror of https://github.com/godotengine/godot.git synced 2025-10-19 16:03:29 +00:00
Code Issues Projects Releases Packages Wiki Activity

Fix empty lines being added for errors with no script backtrace

Browse source
This commit is contained in:
Mikael Hermansson 2025-05-05 13:27:21 +02:00
parent 1cf573f44d
commit 31b90246e7
12 changed files with 37 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

2
core/io/logger.cpp
View file

@ -88,8 +88,10 @@ void Logger::log_error(const char *p_function, const char *p_file, int p_line, c
logf_error(" at: %s (%s:%i)\n", p_function, p_file, p_line);
for (const Ref<ScriptBacktrace> &backtrace : p_script_backtraces) {
if (!backtrace->is_empty()) {
logf_error("%s\n", backtrace->format(3).utf8().get_data());
}
}
}
void Logger::logf(const char *p_format, ...) {

3
core/object/script_backtrace.cpp
View file

@ -52,6 +52,7 @@ void ScriptBacktrace::_store_variables(const List<String> &p_names, const List<V
void ScriptBacktrace::_bind_methods() {
ClassDB::bind_method(D_METHOD("get_language_name"), &ScriptBacktrace::get_language_name);
ClassDB::bind_method(D_METHOD("is_empty"), &ScriptBacktrace::is_empty);
ClassDB::bind_method(D_METHOD("get_frame_count"), &ScriptBacktrace::get_frame_count);
ClassDB::bind_method(D_METHOD("get_frame_function", "index"), &ScriptBacktrace::get_frame_function);
ClassDB::bind_method(D_METHOD("get_frame_file", "index"), &ScriptBacktrace::get_frame_file);
@ -173,7 +174,7 @@ Variant ScriptBacktrace::get_member_variable_value(int p_frame_index, int p_vari
}
String ScriptBacktrace::format(int p_indent_all, int p_indent_frames) const {
if (stack_frames.is_empty()) {
if (is_empty()) {
return String();
}

1
core/object/script_backtrace.h
View file

@ -65,6 +65,7 @@ public:
String get_language_name() const { return language_name; }
bool is_empty() const { return stack_frames.is_empty(); }
int get_frame_count() const { return stack_frames.size(); }
String get_frame_function(int p_index) const;
String get_frame_file(int p_index) const;

6
doc/classes/ScriptBacktrace.xml
View file

@ -124,5 +124,11 @@
[b]Warning:[/b] With GDScript backtraces, the returned [Variant] will be the variable's actual value, including any object references. This means that storing the returned [Variant] will prevent any such object from being deallocated, so it's generally recommended not to do so.
</description>
</method>
<method name="is_empty" qualifiers="const">
<return type="bool" />
<description>
Returns [code]true[/code] if the backtrace has no stack frames.
</description>
</method>
</methods>
</class>

2
drivers/unix/os_unix.cpp
View file

@ -1228,8 +1228,10 @@ void UnixTerminalLogger::log_error(const char *p_function, const char *p_file, i
logf_error("%s%sat: %s (%s:%i)%s\n", gray, indent, p_function, p_file, p_line, reset);
for (const Ref<ScriptBacktrace> &backtrace : p_script_backtraces) {
if (!backtrace->is_empty()) {
logf_error("%s%s%s\n", gray, backtrace->format(strlen(indent)).utf8().get_data(), reset);
}
}
}
UnixTerminalLogger::~UnixTerminalLogger() {}

2
platform/ios/ios_terminal_logger.mm
View file

@ -71,8 +71,10 @@ void IOSTerminalLogger::log_error(const char *p_function, const char *p_file, in
}
for (const Ref<ScriptBacktrace> &backtrace : p_script_backtraces) {
if (!backtrace->is_empty()) {
os_log_error(OS_LOG_DEFAULT, "%{public}s", backtrace->format().utf8().get_data());
}
}
}
#endif // IOS_ENABLED

2
platform/linuxbsd/crash_handler_linuxbsd.cpp
View file

@ -153,8 +153,10 @@ static void handle_crash(int sig) {
}
if (!script_backtraces.is_empty()) {
for (const Ref<ScriptBacktrace> &backtrace : script_backtraces) {
if (!backtrace->is_empty()) {
print_error(backtrace->format());
}
}
print_error("-- END OF SCRIPT BACKTRACE --");
print_error("================================================================");
}

2
platform/macos/crash_handler_macos.mm
View file

@ -183,8 +183,10 @@ static void handle_crash(int sig) {
}
if (!script_backtraces.is_empty()) {
for (const Ref<ScriptBacktrace> &backtrace : script_backtraces) {
if (!backtrace->is_empty()) {
print_error(backtrace->format());
}
}
print_error("-- END OF SCRIPT BACKTRACE --");
print_error("================================================================");
}

2
platform/macos/macos_terminal_logger.mm
View file

@ -84,9 +84,11 @@ void MacOSTerminalLogger::log_error(const char *p_function, const char *p_file,
}
for (const Ref<ScriptBacktrace> &backtrace : p_script_backtraces) {
if (!backtrace->is_empty()) {
os_log_error(OS_LOG_DEFAULT, "%{public}s", backtrace->format().utf8().get_data());
logf_error("\E[0;90m%s\E[0m\n", backtrace->format(strlen(indent)).utf8().get_data());
}
}
}
#endif // MACOS_ENABLED

2
platform/windows/crash_handler_windows_seh.cpp
View file

@ -237,8 +237,10 @@ DWORD CrashHandlerException(EXCEPTION_POINTERS *ep) {
}
if (!script_backtraces.is_empty()) {
for (const Ref<ScriptBacktrace> &backtrace : script_backtraces) {
if (!backtrace->is_empty()) {
print_error(backtrace->format());
}
}
print_error("-- END OF SCRIPT BACKTRACE --");
print_error("================================================================");
}

2
platform/windows/crash_handler_windows_signal.cpp
View file

@ -191,8 +191,10 @@ extern void CrashHandlerException(int signal) {
}
if (!script_backtraces.is_empty()) {
for (const Ref<ScriptBacktrace> &backtrace : script_backtraces) {
if (!backtrace->is_empty()) {
print_error(backtrace->format());
}
}
print_error("-- END OF SCRIPT BACKTRACE --");
print_error("================================================================");
}

2
platform/windows/windows_terminal_logger.cpp
View file

@ -143,8 +143,10 @@ void WindowsTerminalLogger::log_error(const char *p_function, const char *p_file
}
for (const Ref<ScriptBacktrace> &backtrace : p_script_backtraces) {
if (!backtrace->is_empty()) {
logf_error("%s\n", backtrace->format(strlen(indent)).utf8().get_data());
}
}
SetConsoleTextAttribute(hCon, sbi.wAttributes);
}