-
Notifications
You must be signed in to change notification settings - Fork 467
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
make greg bridge more greg #2532
make greg bridge more greg #2532
Conversation
@@ -1035,6 +1037,15 @@ void ApplyOrResetCustomGfxPatches(bool manualChange = true) { | |||
PATCH_GFX(gNintendo64LogoDL, "Title_N64LogoYellow2", n64LogoYellow.changedCvar, 82, gsDPSetEnvColor(color.r, color.g, color.b, 128)); | |||
} | |||
|
|||
// Greg Bridge | |||
if (Randomizer_GetSettingValue(RSK_RAINBOW_BRIDGE) == RO_BRIDGE_GREG) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Main drawback of putting it here is this will run either the unpatch or the patch every frame which isn't super great. Not sure I can think of a better way to handle it right now though
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, i spent a while trying to think of a good spot for it, i kind of figured here works because it's where we're doing a ton of other patching
Should this match the color Greg is set to in the cosmetics editor? |
@RaelCappra not a bad idea, i'll try that out real quick |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Build Artifacts