Skip to content

Cheater - #6012

Open
knifour wants to merge 4 commits into
joncampbell123:masterfrom
knifour:cheater
Open

Cheater#6012
knifour wants to merge 4 commits into
joncampbell123:masterfrom
knifour:cheater

Conversation

@knifour

@knifour knifour commented Dec 24, 2025

Copy link
Copy Markdown

Add a summary of the change(s) brought by this PR here.
Add Game Cheater utility integrated under debugger mode

This PR integrates a Game Cheater utility as an extension of the debugger. Once the debugger is activated, users can launch the cheater by entering the command:
CHEAT ProjectName

Full Tutor
https://knifour.gitbook.io/dosbox-x-game-cheater/

What issue(s) does this PR address?

If there are any related issues, reference them here. Ex. #⁠1234, Fixes #⁠1234, Closes #⁠1234, etc.

Does this PR introduce new feature(s)?

If yes, describe the feature(s) introduced.

Does this PR introduce any breaking change(s)?

If yes, describe the breaking change(s) in detail.

Additional information

Add any additional information here.

@joncampbell123

Copy link
Copy Markdown
Owner

There are some interesting errors from the CI builds. Can you address them?

Go to each CI build and use the search function to highlight every line that contains the text "error:" (without the quotes).

The Linux ones appear to be a simple one-line change to include the required header for a C standard library function.

@knifour

knifour commented Dec 25, 2025 via email

Copy link
Copy Markdown
Author

@knifour

knifour commented Dec 26, 2025 via email

Copy link
Copy Markdown
Author

@knifour

knifour commented Dec 26, 2025 via email

Copy link
Copy Markdown
Author

@joncampbell123

Copy link
Copy Markdown
Owner

The first and simplest one is to add #include <string.h> to the top of the source file where memset() is used. That would fix Linux builds.

@joncampbell123

Copy link
Copy Markdown
Owner

Visual Studio doesn't consider toupper part of the std:: namespace.

image

@joncampbell123

Copy link
Copy Markdown
Owner

There needs to be a definition of GetDOSBoxXPath in your cheat source file. Add this near the top:

std::string GetDOSBoxXPath(bool withexe=false);

image

@joncampbell123

Copy link
Copy Markdown
Owner

You should be able to make these corrections and push them to the branch you've submitted to merge, and it should reflect the changes here.

@joncampbell123

joncampbell123 commented Dec 28, 2025

Copy link
Copy Markdown
Owner

@knifour I've described how to fix this. I can't make the changes for you. They are fairly simple changes to make. If you push to your branch set up to merge here, the CI system here on GitHub should run the build with your changes and show that they no longer cause compile failures This looks like an interesting cheat system that may be useful for others.

Changes:
1. Remove platform-specific C_CHEAT compilation flags
2. Integrate functionality into existing C_DEBUG framework
3. Restore build-mingw-sdl2 and vs/config.h to upstream original versions

Benefits:
- Ensure Game Cheater works on all platforms supporting debug mode
- Reduce platform-specific maintenance burden
- Clearer feature logic (as a debug tool extension)
@Torinde

Torinde commented Feb 1, 2026

Copy link
Copy Markdown
Contributor

@joncampbell123

Copy link
Copy Markdown
Owner

Please look at the error messages. Many of them are one-line changes. I'm not going to merge something that breaks the code.

If this process takes too long, the code will drift to the point your patch will no longer apply.

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