Import Log doesn't display non-ASCII characters

Question

Import Log doesn't display non-ASCII characters

GrimPixel opened this issue 2 years ago · 7 comments

GrimPixel commented 2 years ago

Non-ASCII characters are not displayed in the import log if there are errors.

Answer 1 · 2023-05-11T06:50:15.000Z

Can you provide a screenshot of this happening?

Answer 2 · 2023-05-11T19:41:08.000Z

Here is an example of wrong filename:

The Cyrillic characters don't show up.

Answer 3 · 2023-05-12T00:35:00.000Z

Weird, cyrillic characters seem to work just fine for me:

What version of the game are you running, and where did you download it? What operating system is your system running?

Answer 4 · 2023-05-12T02:56:28.000Z

I meant the red message.

Answer 5 · 2023-05-12T03:52:55.000Z

I can confirm the text does not display properly in the red messages.

In future, may I ask that you use the bug report template when creating bug reports? A lot of the back-and-forth in this issue could have been mitigated if more information was given at the beginning, e.g. screenshots, reproduction steps.

Answer 6 · 2023-05-12T10:53:45.000Z

Ah, I clicked on “New Issue” and signed in, so the selection was skipped. I will use the template.

Answer 7 · 2023-05-14T06:33:06.000Z

I think I know what's going on here. I initially thought it was something to do with fonts not loading, but this is not the case. Godot displays the error just fine in the log, but when Tabletop Club receives the error, the non-ASCII characters are then missing:

Loaded: /Users/drwhut/Documents/TabletopClub/assets/Test/cards/config.cfg
ERROR: 'scale' element in entry cannot be negative!
   at: call (modules/gdscript/gdscript_functions.cpp:774)
ERROR: Cannot add entry to AssetDB, entry is invalid!
   at: call (modules/gdscript/gdscript_functions.cpp:774)
ERROR: Parent 'Téßt Привет' for object 'Test 2' does not exist!
   at: call (modules/gdscript/gdscript_functions.cpp:774)
E: 'scale' element in entry cannot be negative!
E: Cannot add entry to AssetDB, entry is invalid!
E: Parent 'Tet ' for object 'Test 2' does not exist!

When push_error is called in GDScript, the string that is passed in is displayed in the log window, and in stderr, as-is. However, after the error has been displayed, the string then gets passed into a series of "error handlers", one of which is in the custom module, and is used by the game in various locations. But these error handlers expect C-style strings, not Godot strings. This is where I think the text is being corrupted: in the hidden conversion from a Godot string to a C string. This should be a relatively easy fix, I just need to make sure the error or warning is stored separately instead of retrieving it from the error handler.