Skip to content

Refactor BillsHouse and Pikachu movement#158

Merged
Rangi42 merged 1 commit into
pret:masterfrom
kqesar:refactor-bills-house
Jun 24, 2026
Merged

Refactor BillsHouse and Pikachu movement#158
Rangi42 merged 1 commit into
pret:masterfrom
kqesar:refactor-bills-house

Conversation

@kqesar

@kqesar kqesar commented Jun 19, 2026

Copy link
Copy Markdown
Contributor
  • Rename wd471 to wPikachuSpawnStateFlags globally
  • Rename wd492 to wPikachuMapScriptFlags globally
  • Introduce pikachu_movement_constants.asm
  • Refactor BillsHouse scripts and Pikachu/Bill movement data

@kqesar kqesar force-pushed the refactor-bills-house branch from 0dbe838 to 4e0644e Compare June 19, 2026 17:27
Comment thread constants/pikachu_movement_constants.asm Outdated
Comment thread tools/unnamed.py
Comment thread engine/events/poison.asm Outdated
Comment thread engine/minigame/surfing_pikachu.asm Outdated
Comment thread engine/pikachu/pikachu_follow.asm Outdated
Comment thread home/overworld.asm Outdated
@kqesar kqesar force-pushed the refactor-bills-house branch 2 times, most recently from 5fa8be5 to 8deede9 Compare June 23, 2026 15:25
@kqesar kqesar requested a review from Rangi42 June 23, 2026 20:10

@Rangi42 Rangi42 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for identifying the bit flags! I've commented a few more places they could be used.

Comment thread scripts/Route25.asm Outdated
Comment thread scripts/Route25.asm Outdated
Comment thread scripts/Route25.asm Outdated
Comment thread home/map_objects.asm Outdated
- Rename wd471 to wPikachuSpawnStateFlags globally
- Rename wd492 to wPikachuMapScriptFlags globally
- Introduce pikachu_movement_constants.asm
- Refactor BillsHouse scripts and Pikachu/Bill movement data
@kqesar kqesar force-pushed the refactor-bills-house branch from eb5c6cf to 36811a8 Compare June 24, 2026 15:11
@kqesar kqesar requested a review from Rangi42 June 24, 2026 15:13

@Rangi42 Rangi42 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@Rangi42 Rangi42 merged commit d150f7f into pret:master Jun 24, 2026
2 checks passed
github-actions Bot pushed a commit that referenced this pull request Jun 24, 2026
… and `PIKAMOVEMENT_*` constants (#158)

- Rename `wd471` to` wPikachuSpawnStateFlags`, with `BIT_PIKACHU_SPAWN_*` bits
- Rename `wd492` to `wPikachuMapScriptFlags`, with `BIT_PIKACHU_MAP_*` bits
- Introduce `PIKAMOVEMENT_*` constants for `ApplyPikachuMovementData`
- Refactor `BillsHouse` scripts and Pikachu/Bill movement data
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