Skip to content
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

Add bit macros from SA3 #172

Merged
merged 2 commits into from
Nov 24, 2024
Merged

Add bit macros from SA3 #172

merged 2 commits into from
Nov 24, 2024

Conversation

JaceCear
Copy link
Collaborator

No description provided.

@@ -37,8 +37,8 @@
#define OAM_ENTRY_COUNT 128
#if PORTABLE
// NOTE: Used in gba/types.h, so they have to be defined before the #include
#define DISPLAY_WIDTH 240
#define DISPLAY_HEIGHT 160
#define DISPLAY_WIDTH 426
Copy link
Collaborator

Choose a reason for hiding this comment

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

Are we doing widescreen by default now? ;)

#define CheckBit(x, y) ((x) & (BitValue(y)))
#define GetBit(x, y) (((x) >> (y)) & 1)
#define SetBit(x, y) (x) |= BitValue(y)
#define SetSoleBit(x, y) (x) = BitValue(y)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do we need this one?

@@ -786,7 +786,7 @@ static void Task_IntroColorAnimation(void)
}
}

gUnknown_03002A80 = 2;
gUnknown_03002A80 = 2 * sizeof(int_vcount);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Unrelated change, but obviously makes sense to keep

@freshollie freshollie enabled auto-merge (squash) November 24, 2024 23:39
@freshollie freshollie disabled auto-merge November 24, 2024 23:40
@freshollie freshollie merged commit 3aca622 into main Nov 24, 2024
8 checks passed
github-actions bot pushed a commit that referenced this pull request Nov 24, 2024
@JaceCear JaceCear deleted the bit_macros_new branch November 30, 2024 18:48
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