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

SCUMM: detection fixes #530

Merged
merged 5 commits into from Dec 5, 2014
Merged

SCUMM: detection fixes #530

merged 5 commits into from Dec 5, 2014

Conversation

@BenCastricum
Copy link
Contributor

BenCastricum commented Nov 10, 2014

This adds detection for all the Kelloggs Mini Games, and a few other fixes as well.

@BenCastricum BenCastricum force-pushed the BenCastricum:detection_fixes branch 2 times, most recently from 0e1cbf8 to c246f4a Nov 11, 2014
@BenCastricum BenCastricum force-pushed the BenCastricum:detection_fixes branch from c246f4a to 7dda44b Nov 18, 2014
@BenCastricum BenCastricum force-pushed the BenCastricum:detection_fixes branch from 7dda44b to ae630f3 Nov 25, 2014
@Kirben
Copy link
Member

Kirben commented Nov 28, 2014

It would be best to revert the English detection changes for Freddi Fish 1, Putt-Putt Goes to the Moon and Putt-Putt Saves the Zoo. Since they appear to be used by multiple language, and we can't be sure the file sizes are the same either.

@BenCastricum BenCastricum force-pushed the BenCastricum:detection_fixes branch from ae630f3 to 1aed15d Nov 29, 2014
@BenCastricum
Copy link
Contributor Author

BenCastricum commented Nov 29, 2014

ok, I removed those.

@bluegr
Copy link
Member

bluegr commented Dec 2, 2014

Looks good to be merged now.

@BenCastricum BenCastricum force-pushed the BenCastricum:detection_fixes branch from 1aed15d to 30038ef Dec 3, 2014
@bluegr
Copy link
Member

bluegr commented Dec 5, 2014

Thanks for the detection fixes - these look OK to be merged. Merging

bluegr added a commit that referenced this pull request Dec 5, 2014
@bluegr bluegr merged commit 665c813 into scummvm:master Dec 5, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.