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

Sync math operator/utility functions from pokeheartgold #50

Merged
merged 1 commit into from
Dec 27, 2023

Conversation

UsernameFodder
Copy link
Contributor

Compared/synced by hand from the pokeheartgold asm.

Some of the functions had labels corresponding to libgcc counterparts; rename them all for consistency.

Note that some of the functions have multiple names in pokeheartgold (of which at most one is actually used). Instead of polluting the xMAP files with multiple symbols (some with zero length), which makes it really hard for scripts to parse, just add these alternate names as comments. This makes them searchable without having an effect on build artifacts.

Compared/synced by hand from the pokeheartgold asm.

Some of the functions had labels corresponding to libgcc counterparts;
rename them all for consistency.

Note that some of the functions have multiple names in pokeheartgold
(of which at most one is actually used). Instead of polluting the xMAP
files with multiple symbols (some with zero length), which makes it
really hard for scripts to parse, just add these alternate names as
comments. This makes them searchable without having an effect on build
artifacts.
@AnonymousRandomPerson AnonymousRandomPerson merged commit 1dd0f32 into pret:main Dec 27, 2023
1 check passed
github-actions bot pushed a commit that referenced this pull request Dec 27, 2023
@UsernameFodder UsernameFodder deleted the sync branch December 28, 2023 02:53
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