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

Another take on archive scanning splash screen #1798

Merged
merged 2 commits into from
Dec 4, 2024
Merged

Another take on archive scanning splash screen #1798

merged 2 commits into from
Dec 4, 2024

Conversation

lhog
Copy link
Collaborator

@lhog lhog commented Dec 4, 2024

Apparently (and obviosly in the hindsight) the previous attempt worked for local skirmishes only.
The unified approach will use CPreGame::Draw() to display what's going on both with the network connection and archives check-summing.
The changes look quite scary, but seem to work fine

Apparently (and obviosly in the hindsight) the previous code only worked for local skirmishes only.
The unified approach will use CPreGame::Draw() to display what's going on both with the network connection and archives checksumming.
The changes look quite scary, but seem to work fine
rts/Game/PreGame.cpp Outdated Show resolved Hide resolved
@lhog lhog changed the title Another take on https://github.com/beyond-all-reason/spring/pull/1792 Another take on archive scanning splash screen Dec 4, 2024
@lhog lhog merged commit 3d35645 into master Dec 4, 2024
2 checks passed
@lhog lhog deleted the async-pregame branch December 4, 2024 07:16
github-actions bot pushed a commit that referenced this pull request Dec 5, 2024
* Another take on #1792
Apparently (and obviously in the hindsight) the previous code worked for local skirmishes only.
The unified approach will use `PreGame::Draw()` to display what's going on both with the network connection and archives check-summing.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants