Skip to content

[Modding] Fierce Deity Sheaths#1606

Open
Jameriquiah wants to merge 1 commit intoHarbourMasters:developfrom
Jameriquiah:fd-sheaths
Open

[Modding] Fierce Deity Sheaths#1606
Jameriquiah wants to merge 1 commit intoHarbourMasters:developfrom
Jameriquiah:fd-sheaths

Conversation

@Jameriquiah
Copy link
Contributor

@Jameriquiah Jameriquiah commented Mar 20, 2026

Allows modders to make sheaths for the FD sword. In objects/object_link_boy/, gLinkFierceDeitySwordInSheathDL and gLinkFierceDeitySwordSheathDL. If those DL's are detected the game will just load them. So no options or toggles were necessary.

Custom Fierce Deity Sheath model ported to this by @nickel246 for testing purposes, posted with permission.
Fierce Deity Sheath.zip

Build Artifacts

Comment on lines +17 to +30
static bool CustomFDSheath_LoadAssets() {
if (sFierceDeitySheathState == 0) {
sFierceDeitySheathState = 1;

if (ResourceMgr_FileExists(sFierceDeitySwordInSheathDLPath) &&
ResourceMgr_FileExists(sFierceDeitySwordSheathDLPath)) {
sFierceDeitySwordInSheathDL = ResourceMgr_LoadGfxByName(sFierceDeitySwordInSheathDLPath);
sFierceDeitySwordSheathDL = ResourceMgr_LoadGfxByName(sFierceDeitySwordSheathDLPath);
sFierceDeitySheathState = 1;
}
}

return sFierceDeitySheathState == 1;
}
Copy link
Contributor

Choose a reason for hiding this comment

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

What does state 1 mean here? The way its written, both outcomes end up at 1.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

just returns and stops it from running every frame, at 0 it checks for if the models exist then stops after, this implementation is kinda broken tho rn i was working on a different way but im at lunch with the wife now, will clean all this up in a bit lol

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.

2 participants