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: add opcode for specfic Adibou 2 German version #5331

Closed
wants to merge 1 commit into from

Conversation

BJNFNE
Copy link
Contributor

@BJNFNE BJNFNE commented Sep 12, 2023

This PR is for adding the opcode for following reason: if you leave in one specfic German version of Adibou 2 (ADI Junior 2) the Application and want to return to the Garden of Adibou, Then the Console was printing an unimplemented opcode, so we can get rid of this message,

@bluegr
Copy link
Member

bluegr commented Sep 13, 2023

So, you're stubbing that opcode. What does it actually do? Is there any functionality missing when stubbing it out?

@BJNFNE
Copy link
Contributor Author

BJNFNE commented Sep 13, 2023

So, you're stubbing that opcode. What does it actually do? Is there any functionality missing when stubbing it out?

Its just continues

@sdelamarre
Copy link
Contributor

Does the game crash because of the missing opcode? Or is it just a warning displayed in the log?

@BJNFNE
Copy link
Contributor Author

BJNFNE commented Sep 14, 2023

Does the game crash because of the missing opcode? Or is it just a warning displayed in the log?

Just Prints an Warning message

@sdelamarre
Copy link
Contributor

I specifically advised you to avoid warnings-related PRs two weeks ago:
scummvm/scummvm-tools#75 (comment)

@sdelamarre sdelamarre closed this Sep 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants