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

Add "CustomMissionID" for saved games #19

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

chaserli
Copy link
Member

@chaserli chaserli commented Feb 19, 2024

Implemented the updated requirements of issue #18

@SadPencil
Copy link
Member

Sorry, but I'm a bit confused. Are there any unclear passages in issue #18?

@chaserli
Copy link
Member Author

Sorry, but I'm a bit confused. Are there any unclear passages in issue #18?

I just noticed the additional message on the issue page, sorry.
So far I introduced 4 consecutive int32, indicating the CampaignID, total House count present, total Techno count present, total Anim count present, just for testing.

I'm not sure if the ScenarioNumber is eventually unused when the spawner is present, and since we can actually append more extra info on the file, why not recording more.

So far if you can manage the previous 4 bytes (int32) of the member Spawner MetaInfo, then you can set [Settings]CampaignID in spawn.ini to modify it.

@SadPencil
Copy link
Member

SadPencil commented Feb 21, 2024

@chaserli
When loading a saved game and saving the game again, the CampaignID is set to -1 instead of previous value in the .sav file if it is not specified in spawn.ini. Could you make the spawner read this value from .sav file first? (although the client is able to specify this value again, I think it is better to handle this in the spawner; for example users might load another mission inside the game, where the client is unable to interrupt)

Also, could be better if this value is an uint64. I use hash algorithm to generate such an ID so using an 8-byte unsigned integer is better than a signed number.


Besides, all the 4 values you introduced for testing work as intended. For the sake of ABI compatibility, could you just save the campaign ID (uint64) in an separate OLE key "Campaign ID"?

@chaserli
Copy link
Member Author

yeah I'm aware of such issue. There's another problem: when finishing the current mission and proceeding to the next one, such ID should change according to your requirement, but I have no idea how to make that happen yet. I'll try to investigate when I have time later

@chaserli chaserli marked this pull request as ready for review March 5, 2024 06:10
@SadPencil
Copy link
Member

@chaserli You are right. pINI->ReadInteger does not support reading uint64 so I guess I have to use int32

@chaserli
Copy link
Member Author

chaserli commented Mar 7, 2024

@chaserli You are right. pINI->ReadInteger does not support reading uint64 so I guess I have to use int32

Alright, I reverted that to int. Now Campaign ID appears in sav file only when its value is not 0

Copy link

github-actions bot commented Mar 9, 2024

Nightly build for this pull request:

This comment is automatic and is meant to allow guests to get latest nightly builds for this pull request without registering. It is updated on every successful build.

@SadPencil SadPencil changed the title Implement issue #18 Add "CustomMissionID" for saved games Sep 22, 2024
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