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

Implement InitBinoc #104

Merged
merged 2 commits into from
Oct 25, 2024
Merged

Implement InitBinoc #104

merged 2 commits into from
Oct 25, 2024

Conversation

t895
Copy link
Contributor

@t895 t895 commented Oct 24, 2024

Had to untangle a couple of includes to get this compiled so please let me know if I made the right decisions while refactoring

@TheOnlyZac
Copy link
Owner

TheOnlyZac commented Oct 24, 2024

Could you put gs.h back how it was? It's the header for P2/gs.c.

t895 added 2 commits October 24, 2024 00:40
Needed to pull RGBA out into common to fix up some errors during compilation if you tried to include screen.h in binoc.h
I also had to change the internally defined name for sce/gs.h because there would be a conflict otherwise
@t895
Copy link
Contributor Author

t895 commented Oct 24, 2024

Sorry about that. I cleaned up the mistakes I made and found a more proper solution to the problem by changing #define GS_H to #define SCE_GS_H in sce/gs.h. It fixes an issue where the compiler has a hard time reconciling between the two identical define statements and results in an error about implicit function declarations where ClearFrameBuffers() is used.

@TheOnlyZac TheOnlyZac merged commit 7059b62 into TheOnlyZac:main Oct 25, 2024
1 check failed
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