-
Notifications
You must be signed in to change notification settings - Fork 470
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
Fix: Various Rando generation crashes #3108
Fix: Various Rando generation crashes #3108
Conversation
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.
while (!locations.empty() && (!endOnItemsEmpty || !items.empty())) { | ||
uint32_t loc = RandomElement(locations, true); | ||
Location(loc)->SetAsHintable(); | ||
PlaceItemInLocation(loc, RandomElement(items, true)); | ||
|
||
if (items.empty() && !endOnItemsEmpty) { | ||
items.push_back(GetJunkItem()); | ||
} | ||
|
||
uint32_t loc = RandomElement(locations, true); | ||
Location(loc)->SetAsHintable(); | ||
PlaceItemInLocation(loc, RandomElement(items, true)); | ||
} |
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.
is this a change that needs to be PR'd to 3drando or is it a fix from 3drando that we didn't have yet?
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.
This change is not in 3drando, and could be upstreamed
Re-adding merge ready label as criteria is now met (3 approvals & 3+ days since first) |
Fixes possible rando generation crashes by:
I also made it so the rando gen success sound (horse goal) only plays when the seed generates correctly. If it fails, it now plays an error tone (failed ocarina song).
Build Artifacts