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

Assert using Hex value #12

Closed
mauvesea opened this issue Jul 13, 2023 · 5 comments
Closed

Assert using Hex value #12

mauvesea opened this issue Jul 13, 2023 · 5 comments
Labels
expand-mon-ID Branch PC16 Issue Issues related to PC16 itself.

Comments

@mauvesea
Copy link

gfx/footprints.asm uses $100 for assert, instead of using a constant like NUM_POKEMON.

@aaaaaa123456789
Copy link
Collaborator

This is not a simple issue, because the codebase actually expects $100 as written, even though NUM_POKEMON is less than $100. Maybe the solution is to fix the code so there's no unused padding.

@vulcandth vulcandth added PC16 Issue Issues related to PC16 itself. expand-mon-ID Branch labels Jul 13, 2023
@Rangi42
Copy link
Collaborator

Rangi42 commented Oct 24, 2023

Yeah, fixing the design flaw would be appropriate, like the many applied bugfixes which were relevant to the 16-bit features.

@vulcandth
Copy link
Owner

vulcandth commented Oct 24, 2023

Yeah, fixing the design flaw would be appropriate, like the many applied bugfixes which were relevant to the 16-bit features.

That design flaw was already fixed in @aaaaaa123456789 's commit 03414e5.

@Rangi42
Copy link
Collaborator

Rangi42 commented Oct 24, 2023

Great, then yes, the extra footprints are no longer necessary.

@vulcandth
Copy link
Owner

This has been resolved in the latest rebase.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
expand-mon-ID Branch PC16 Issue Issues related to PC16 itself.
Projects
None yet
Development

No branches or pull requests

4 participants