Skip to content
This repository was archived by the owner on Sep 16, 2025. It is now read-only.

feat: add hook helpers#44

Merged
ThirdEyeSqueegee merged 7 commits intomainfrom
feature/helpers
Sep 14, 2023
Merged

feat: add hook helpers#44
ThirdEyeSqueegee merged 7 commits intomainfrom
feature/helpers

Conversation

@ThirdEyeSqueegee
Copy link
Copy Markdown
Contributor

No description provided.

@ThirdEyeSqueegee ThirdEyeSqueegee changed the title feat: add hook helpers, update PCH.h (a few C++23 headers were missing) feat: add hook helpers Sep 13, 2023
Comment thread CommonLibSF/include/SFSE/API.h Outdated
Comment thread CommonLibSF/include/SFSE/API.h Outdated
Comment thread CommonLibSF/include/SFSE/API.h Outdated
Comment thread CommonLibSF/include/SFSE/Utilities.h
Comment thread CommonLibSF/include/SFSE/Utilities.h Outdated
Comment thread CommonLibSF/include/SFSE/Utilities.h Outdated
Comment thread CommonLibSF/include/SFSE/Utilities.h Outdated
@gottyduke
Copy link
Copy Markdown
Contributor

I'm okay with these additions, leaving a note here though because sometime in the future I'll look into the Starfield IPO problem, that apparently Bethesda has explicitly turned on for this game. Some function call hooks cannot be safely hooked now without preserving the volatile registers that are treated as non-volatile by the optimization.
When that happens, we'll also need to come back and test if these can work with new changes.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants