Skip to content

Cheat: Child Link Can Hold Hylian Shield (Modern Codebase Port)#5990

Draft
nclok1405 wants to merge 10 commits intoHarbourMasters:developfrom
nclok1405:childholdshylianshield
Draft

Cheat: Child Link Can Hold Hylian Shield (Modern Codebase Port)#5990
nclok1405 wants to merge 10 commits intoHarbourMasters:developfrom
nclok1405:childholdshylianshield

Conversation

@nclok1405
Copy link
Copy Markdown
Contributor

@nclok1405 nclok1405 commented Nov 27, 2025

This is an experimental port of Patrick12115's "Child Link Can Hold Hylian Shield" (#3887) to the modern codebase.

I also moved the category of this option from Enhancements to Cheat, and modified Octorok's projectile to be reflectable with this option.

Build Artifacts

Comment thread soh/src/overlays/actors/ovl_En_Okuta/z_en_okuta.c Outdated
Comment thread soh/soh/Enhancements/Graphics/ChildHoldsHylianShield.cpp Outdated
Comment thread soh/src/code/z_player_lib.c Outdated
return false; // Skip vanilla check for making child Link have the Hylian Shield on his back, allowing for it to
// be used in hand
} else {
return gSaveContext.linkAge != 0 && (this->currentShield == PLAYER_SHIELD_HYLIAN);
Copy link
Copy Markdown
Member

@serprex serprex Jan 5, 2026

Choose a reason for hiding this comment

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

can be return GameInteractor_Should(...) with *should = false in handler instead of doing cvar check here

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Added a new hook and changed z_player_lib.c to use that.

I'm not sure what to do with remaining direct CVar usage (adding many more hooks?)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yes, long term having a mod_hooks.cpp file in Enhancements that hooks a bunch of VBs may be the way

@nclok1405 nclok1405 force-pushed the childholdshylianshield branch from 7a1fc15 to 9345ff7 Compare January 7, 2026 06:58
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