Skip to content

Add initialization checks for location table - #845

Merged
HylianFreddy merged 5 commits into
gamestabled:mainfrom
HylianFreddy:location-checks
Jul 19, 2026
Merged

Add initialization checks for location table#845
HylianFreddy merged 5 commits into
gamestabled:mainfrom
HylianFreddy:location-checks

Conversation

@HylianFreddy

Copy link
Copy Markdown
Collaborator

This adds the following checks to help catch bugs during development:

  • the maximum possible location count must be less than the amount of item overrides allocated in the patch code;
  • each location name must be at most 51 characters long to fit in the in-game menu.

The second check revealed that some MQ locations already had a name too long, so I shortened those.

I've also renamed the DungeonInfo enum to DungeonImportance, because the app code defines a class named DungeonInfo that could cause namespace conflicts in some cases.

And one last change I've included is printing "Initializing..." on the top screen during application startup.

@HylianFreddy
HylianFreddy merged commit abb3721 into gamestabled:main Jul 19, 2026
2 checks passed
@HylianFreddy
HylianFreddy deleted the location-checks branch July 19, 2026 21:54
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