Skip to content

Performed small graphx optimizations #611

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

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Conversation

ZERICO2005
Copy link
Contributor

@ZERICO2005 ZERICO2005 commented Jun 2, 2025

gfx_FloodFill was moved so it can leverage ld sp, ix \ pop ix from gfx_Polygon, allowing for jr to be used instead of jp. This also makes the git diff look weird so I will fix that in a bit.

Otherwise, small optimizations were performed across some functions. _GetSinCos is 2 bytes smaller, changed pop hl \ push hl to ex (sp), hl, and other functions have been reordered so jp can be changed to jr.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant