Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build soh with LUS 1.0.0 #2881

Merged

Conversation

briaguya-ai
Copy link
Contributor

@briaguya-ai briaguya-ai commented May 11, 2023

kinda lost with the current errors, maybe the CI will help me figure out what's broken?

Build Artifacts

@briaguya-ai briaguya-ai added do not merge Not ready or not valid changes libultraship labels May 11, 2023
extern "C" void DeinitOTR() {
SaveManager_ThreadPoolWait();
Copy link
Contributor

@Malkierian Malkierian May 22, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume, having this moved to here instead of using the now removed ExitGame hook, means that DeinitOTR is being called when Alt+F4 or the title bar X are input?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It should be, yes. This is called after the main game loop which is only hit when the window is closed.

soh/soh/SohGui.cpp Outdated Show resolved Hide resolved
soh/soh/SohMenuBar.cpp Outdated Show resolved Hide resolved
@Kenix3 Kenix3 changed the title [wip] build soh LUS 1.0.0 release candidate build soh LUS 1.0.0 May 29, 2023
@Kenix3 Kenix3 changed the title build soh LUS 1.0.0 build soh with LUS 1.0.0 May 29, 2023
@briaguya-ai briaguya-ai removed the do not merge Not ready or not valid changes label Jun 3, 2023
@leggettc18 leggettc18 merged commit 2308ab8 into HarbourMasters:develop Jun 3, 2023
@briaguya-ai briaguya-ai deleted the build-soh-with-gui-changes branch June 3, 2023 19:39
@garrettjoecox garrettjoecox added this to the Sulu (7.1.x) milestone Jun 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants