Skip to content

Conversation

@pipe01
Copy link
Contributor

@pipe01 pipe01 commented Dec 29, 2025

The SetTime method is indirectly called by the global variable constructor in main.cpp, which runs before the logger is initialized and thus crashes when trying to log a message. This doesn't affect release builds since logging is disabled there, but it does affect debug builds

@github-actions
Copy link

github-actions bot commented Dec 29, 2025

Build size and comparison to main:

Section Size Difference
text 382876B 0B
data 944B 0B
bss 22632B 0B

Run in InfiniEmu

@mark9064 mark9064 added the bug Something isn't working label Dec 30, 2025
@mark9064
Copy link
Member

Hmm, I don't really like adding an extra argument to solve this. Could we defer the set time call to after logging is initialised?

@pipe01
Copy link
Contributor Author

pipe01 commented Dec 30, 2025

You're right, I think it's better this way

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants