Skip to content

chore: replace true and false#1241

Merged
bates64 merged 3 commits intopmret:mainfrom
a2937:chore/true-false-enum
Jul 25, 2025
Merged

chore: replace true and false#1241
bates64 merged 3 commits intopmret:mainfrom
a2937:chore/true-false-enum

Conversation

@a2937
Copy link
Copy Markdown
Contributor

@a2937 a2937 commented Jul 24, 2025

I feel like I did something not quite right here in this PR given all the changed files.

Closes #1169

@ethteck
Copy link
Copy Markdown
Member

ethteck commented Jul 24, 2025

The changes here look good to me. This project just has an enormous number of source files. Unfortunately the build is failing for some reason due to "bool" being defined twice. I'm trying to figure out where the other place is.,,

@a2937
Copy link
Copy Markdown
Contributor Author

a2937 commented Jul 24, 2025

My Visual Studio Code installation was acting up while I was making it and complained about the item_enum. I did not intend to have a code error.

Comment thread include/macros.h Outdated
@ethteck
Copy link
Copy Markdown
Member

ethteck commented Jul 24, 2025

I fixed several issues that fell out of making this change. I hope you don't mind if I push them to this PR, since they're needed for the build to pass

@a2937
Copy link
Copy Markdown
Contributor Author

a2937 commented Jul 24, 2025

I don't mind at all. The only thing I care about is the code working.

Comment thread include/macros.h Outdated
@bates64 bates64 merged commit 3e5df3a into pmret:main Jul 25, 2025
3 checks passed
@a2937 a2937 deleted the chore/true-false-enum branch July 25, 2025 12:46
bates64 added a commit to bates64/papermario-dx that referenced this pull request Jan 4, 2026
bates64 added a commit to bates64/papermario-dx that referenced this pull request Jan 4, 2026
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.

Lowercase TRUE and FALSE, rename b32 to bool

3 participants