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

GOB: improve magicstones detection #6400

Merged
merged 1 commit into from
Jan 10, 2025
Merged

Conversation

BJNFNE
Copy link
Contributor

@BJNFNE BJNFNE commented Jan 10, 2025

No description provided.

@bluegr
Copy link
Member

bluegr commented Jan 10, 2025

Thanks!

@bluegr bluegr merged commit a1fd47b into scummvm:master Jan 10, 2025
8 checks passed
Copy link
Member

@lephilousophe lephilousophe left a comment

Choose a reason for hiding this comment

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

I have a couple of comments on this PR.

AD_ENTRY2s("ed4.stk", "98721a7cfdc5a358d7ac56b7c6d3ba3d", 541882,
"ed4cd.itk", "0627a91d9a6f4772c33747ce752024c2", 606993908),
"ed4_cd.itk", "0627a91d9a6f4772c33747ce752024c2", 606993908),
Copy link
Member

Choose a reason for hiding this comment

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

This changes the filename.
@Strangerke could you check?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I own an physical copy of the french version and there is it ed4_cd.itk and those hashes are identical.

@@ -37,9 +37,9 @@
{ // Added by Strangerke in 2009

"magicstones",
MetaEngineDetection::GAME_NOT_IMPLEMENTED, // Le pays des Pierres Magiques (Engine: DEV7 version 1.2.0.0)
MetaEngineDetection::GAME_NOT_IMPLEMENTED, // A la poursuite du dragon de feu (Engine: DEV7 version 1.2.0.0)
Copy link
Member

Choose a reason for hiding this comment

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

Why this change?
A la poursuite du dragon de feu is only the subtitle of the game.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This what my physical copy is called, I added the subtitle as an reference.

@BJNFNE BJNFNE deleted the pierresmagiques branch January 10, 2025 20:15
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.

3 participants