Skip to content

Fixing -Wall warnings#2585

Open
djevangelia wants to merge 1 commit into
OoTRandomizer:Devfrom
djevangelia:wall-edit
Open

Fixing -Wall warnings#2585
djevangelia wants to merge 1 commit into
OoTRandomizer:Devfrom
djevangelia:wall-edit

Conversation

@djevangelia

Copy link
Copy Markdown

There are a few -Wall warnings, mostly because of some logic branches missing return. I added return values and default cases to switches as seemed appropriate. Sometimes the code really shouldn't reach those returns and a soft bug info screen would have been useful.

I would particularly want someone familiar with the get item override system to look at get_override_search_key and get_override_search_key_by_newflag which both lack a default return, if something else should be done there.

Testing

Ran around playing and getting items for 15 minutes in Ares. I've had same changes in my R&D build for a while with no issues in Ares and Project64.

@fenhl

fenhl commented May 26, 2026

Copy link
Copy Markdown
Collaborator

See #2492 for previous discussion on this. At the very least, we should test to make sure #2486 doesn't repro on this branch.

@fenhl fenhl added Component: ASM/C Changes some internals of the ASM/C libraries Status: Needs Review Someone should be looking at it Status: Needs Testing Probably should be tested Type: Maintenance Code style, infrastructure, updating dependencies labels May 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component: ASM/C Changes some internals of the ASM/C libraries Status: Needs Review Someone should be looking at it Status: Needs Testing Probably should be tested Type: Maintenance Code style, infrastructure, updating dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants