Skip to content

battle2: make BattleSpawnFloatingIconAt and BattleSpawnFloatingIconAtPart non-static - #313

Closed
j3lee wants to merge 1 commit into
Xeeynamo:mainfrom
j3lee:battle-floating-icon
Closed

j3lee wants to merge 1 commit into
Xeeynamo:mainfrom
j3lee:battle-floating-icon

Conversation

@j3lee

@j3lee j3lee commented Oct 12, 2026

Copy link
Copy Markdown
Contributor

Two magic overlays (cho_kyu, kakusan) call these two functions, so they cannot be static. This drops static in battle2.c and declares both in battle.h. No code change.

BattleSpawnFloatingIconAt and BattleSpawnFloatingIconAtPart are called from the Cho-Kyu and Kakusan
magic overlays (by address); dropping static and declaring them in battle.h lets those overlays use
a header prototype instead of a local one. No code change.
@j3lee

j3lee commented Oct 13, 2026

Copy link
Copy Markdown
Contributor Author

Closing this and the spell import drafts (#314 to #325) for now, following the hold on mass-importing MAGIC overlays. They were written one overlay at a time and would need a rewrite once the shared code and asset handling are settled, so there is no point keeping them on the board. I'll come back with something along the lines of the SOTN src/weapon/shared.h approach when it is ready to look at.

@j3lee j3lee closed this Oct 13, 2026
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.

1 participant