Skip to content

Conversation

@briaguya0
Copy link
Contributor

@briaguya0 briaguya0 commented Feb 27, 2023

  • make it so greg hint doesn't show up when shopkeepers say "Thanks a lot!" (buying refills after already having the item)
  • fix RSK parsing so we don't try to parse all off/on options as keyrings
  • remove a little copypasta in RSK parsing

Build Artifacts

Copy link
Contributor

@Archez Archez left a comment

Choose a reason for hiding this comment

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

Oh gosh.. I messed that up big time. Thanks for catching 😅

Copy link
Contributor

@aMannus aMannus left a comment

Choose a reason for hiding this comment

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

Couple of questions about the text ID's that are used here.

Comment on lines +109 to +114
// greg hint, we set it to 0x6E instead
if (gSaveContext.n64ddFlag && Randomizer_GetSettingValue(RSK_GREG_HINT)) {
this->actor.textId = 0x6E;
} else {
this->actor.textId = 0x84; //Thanks a lot! (Lost)
}
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 the '0x6E' text ID correspond to? Is it an unused text ID?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

0x6E is the text ID for talking to the treasure shop guy after winning the game, it's used on line 100

0x704C is the text ID for talking to the treasure shop guy after buying a key but before going into the game

Copy link
Contributor

Choose a reason for hiding this comment

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

I think I would've preferred a check for the treasure chest room ID inside OTRGlobals.cpp so all the checks for it would be in one file, but it's not a big deal overall. Thanks for the explanation.

} else if (Randomizer_GetSettingValue(RSK_DAMPES_DIARY_HINT) && textId == TEXT_DAMPES_DIARY) {
messageEntry = CustomMessageManager::Instance->RetrieveMessage(Randomizer::randoMiscHintsTableID, TEXT_DAMPES_DIARY);
} else if (Randomizer_GetSettingValue(RSK_GREG_HINT) && (textId == 0x704C || textId == 0x6E || textId == 0x84)) {
} else if (Randomizer_GetSettingValue(RSK_GREG_HINT) && (textId == 0x704C || textId == 0x6E)) {
Copy link
Contributor

Choose a reason for hiding this comment

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

I see there's 2 text ID's here, what's the difference between the 2?

Copy link
Member

@PurpleHato PurpleHato Feb 27, 2023

Choose a reason for hiding this comment

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

I see there's 2 text ID's here, what's the difference between the 2?

They're both for the treasure minigame NPC
704C is the "with that key you can xxxx"
6E is the you're a good gambler

@briaguya0 briaguya0 merged commit 061e232 into HarbourMasters:develop Feb 28, 2023
@briaguya0 briaguya0 deleted the fix-greg-hint branch March 1, 2023 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants