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 version differences #7

Merged
merged 4 commits into from
Jul 16, 2023
Merged

Conversation

mbugert
Copy link
Collaborator

@mbugert mbugert commented Jul 11, 2023

Since we talked about potential changes in the History Edition, I got my hands on it. Comparing file hashes of haeuser.cod, figuren.cod and all BSH files for KE, GOG, History Edition, I found that they're all identical.
So whoever stated that BSH files were changed for the History Edition may have compared them to the original or NINA versions. 🤔

@Green-Sky
Copy link
Owner

that cant be, i remember them adding non color-indexed sprites. but i cant remember where i got that info...

@Green-Sky
Copy link
Owner

found it: siredmar/mdcii-engine#87 (comment)

@mbugert
Copy link
Collaborator Author

mbugert commented Jul 15, 2023

Whoopsie, looks like I didn't have the ToolGfx files on my radar at all. I looked into them for KE, GOG, HE, for each language I had available.

The way assets are reused across versions is more complex than I had suspected. Funnily enough, the Polish GOG version must be a version below KE, but I cannot tell right now whether it's NINA or the original (because I don't have those versions around).

I put all the new version-specific details into a separate file, because that seemed more manageable. Let me know what you think.

@mbugert mbugert changed the title Add sha256sum for COD and BSH files Add version differences Jul 15, 2023
Copy link
Owner

@Green-Sky Green-Sky left a comment

Choose a reason for hiding this comment

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

Very nice research.
I don't have the files at me to test right now, but I'm gonna trust you that they are correct 😃

version_differences.md Outdated Show resolved Hide resolved
version_differences.md Outdated Show resolved Hide resolved
version_differences.md Outdated Show resolved Hide resolved
@Green-Sky Green-Sky merged commit b9a767a into Green-Sky:master Jul 16, 2023
@mbugert mbugert deleted the version_differences branch July 16, 2023 21:03
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