Skip to content

Rupee Dash Mode#2510

Closed
Caladius wants to merge 20 commits intoHarbourMasters:developfrom
Caladius:SuperSonic2
Closed

Rupee Dash Mode#2510
Caladius wants to merge 20 commits intoHarbourMasters:developfrom
Caladius:SuperSonic2

Conversation

@Caladius
Copy link
Contributor

@Caladius Caladius commented Feb 20, 2023

Copy link
Contributor

@garrettjoecox garrettjoecox left a comment

Choose a reason for hiding this comment

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

I get the "sonic" thing is a bit of a meme, but are we sure that's what we want to name this? It's not super intuitive, to me at least

@briaguya0
Copy link
Contributor

I know "Hole in Pocket" or "Burning Rupees"/"Burning Cash" were suggested as non-sonic names. We just need to make sure whatever name we settle on communicates rupees functioning as a timer/health.

@Caladius
Copy link
Contributor Author

Naming of this is definitely not something I settled on, was just what I referred to it as when fleshing the idea out. It can be anything that makes sense for sure!

@aMannus
Copy link
Contributor

aMannus commented Feb 24, 2023

A reference to the Magic Armor might keep it inside the Zelda universe but I'm not sure how to spin that into something that describes the mode very well.

@Caladius
Copy link
Contributor Author

A reference to the Magic Armor might keep it inside the Zelda universe but I'm not sure how to spin that into something that describes the mode very well.

I have thought about making another mode where Nayrus Love consumes rupees on hit and the player is locked at a quarter heart. So maybe not directly tied to Magic Armor for this one but something akin to Doom Counter from final fantasy?

@briaguya0
Copy link
Contributor

@dcvz suggested "Rupee Dash" as the name and I dig it

Copy link
Contributor

@briaguya0 briaguya0 left a comment

Choose a reason for hiding this comment

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

the suggested inlining

@Caladius Caladius changed the title Super Sonic Mode Rupee Dash Mode Feb 26, 2023
@Caladius
Copy link
Contributor Author

That should be everything pending, please let me know if there's anything additional or something new that needs adjusting.

Copy link
Contributor

@dcvz dcvz left a comment

Choose a reason for hiding this comment

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

Looks good overall! Just had a small doubt

Rupees_ChangeBy(-1);
} else {
if (gSaveContext.health > 0) {
gSaveContext.health = gSaveContext.health - 16;
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a danger that health is 10 or something and you subtract 16 from it? Would the game handle it?
Do we need to do something like:

gSaveContext.health = std::min(gSaveContext.health - 16, 0);

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@dcvz im not taking suggestions at this time... lmao jk.

Solid point, i can double check but im pretty sure i had a quarter heart less than full at i just died rather than crashing ot anything

This reverts commit d2848d0.
@dcvz
Copy link
Contributor

dcvz commented Feb 28, 2023

Merged as #2553 . Great work on this @Caladius !
Bria wasn't able to fix your branch directly so he PR'd with the rebase

@dcvz dcvz closed this Feb 28, 2023
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.

5 participants