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

Implemented proper handling of game start-end events. #48

Merged
merged 3 commits into from
Dec 3, 2023

Conversation

MiranDMC
Copy link
Collaborator

Fixed multiple execution of scripts during new game start.

@MiranDMC MiranDMC requested a review from x87 November 18, 2023 06:34
@MiranDMC MiranDMC force-pushed the proper_game_start_end_handling branch 2 times, most recently from 379787f to ff6621f Compare November 18, 2023 06:53
Fixed multiple execution of scripts during new game start.
@MiranDMC MiranDMC force-pushed the proper_game_start_end_handling branch from ff6621f to 5446503 Compare November 18, 2023 15:15
@MiranDMC MiranDMC changed the title Implemented proper of game start-end events. Implemented proper handling of game start-end events. Nov 19, 2023
void(__cdecl* GameRestart3)() = nullptr;
static void __cdecl OnGameRestart1();
static void __cdecl OnGameRestart2();
static void __cdecl OnGameRestart3();
Copy link

Choose a reason for hiding this comment

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

@MiranDMC can you document in wiki what events and CLEO callbacks are executed when we

  • start a new game
  • start a game from a save file
  • loading another save
  • exiting game

Copy link
Collaborator Author

Choose a reason for hiding this comment

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


gangWeapons[7].weapon[0] = 22;
gangWeapons[7].weapon[1] = 28;
gangWeapons[7].weapon[2] = 0;
Copy link

Choose a reason for hiding this comment

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

I don't know why we make this for EU 1.01 specifically. CLEO does not work here anyway. Let's delete this, it's not CLEO business

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done

@x87 x87 merged commit 9fa9b50 into master Dec 3, 2023
@MiranDMC MiranDMC deleted the proper_game_start_end_handling branch December 3, 2023 22:13
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