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

Port over ShipInit from 2Ship #4756

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

aMannus
Copy link
Contributor

@aMannus aMannus commented Dec 23, 2024

As title says, ports over the ShipInit changes from Proxy over from 2Ship to SoH. Only includes a converted Moon Jump for now to confirm it works. Eventually we'd want to transfer everything from hook_handlers.cpp etc to separate files like this and use this new pattern, but that'll be a while.

I want this in now because I want to start creating a couple of example PRs how enhancements etc should look using hooks, and I really want to start doing that with the new macros. That way we can direct people with existing PRs to the new examples as well instead of having to say "please use hooks instead" with no or little direction.

Build Artifacts

@aMannus
Copy link
Contributor Author

aMannus commented Dec 28, 2024

CVAR defines updated and also passed along a function inside the COND_ instead so Intellisense is actually useful (ty proxy :))

Copy link
Contributor

@Pepe20129 Pepe20129 left a comment

Choose a reason for hiding this comment

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

Unfortunate that intellisense doesn't work in some IDEs without separating the function but it is what it is.

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