wRAM labels

[zelosos]

• give all used wRAM addresses a label
• replace wRAM addresses with known labels (need to be continued at 0xD150)

@kemenaran Have fun looking through the work of some days :P

[kemenaran]

Wow, that's quite an impressive refactoring work. This looks much better – and labeling the WRAM addresses will greatly help to reach full RAM shiftability (were we can move the WRAM labels without breaking the game).

I'm all for merging this. I left a few comments, but nothing too bad, I guess. Thanks!

[kemenaran]

I think the OAMBuffer area can be used for several purposes.

One is on the World gameplay mode, where indeed part of it is allocated for Link sprites, and part of it for entities.

But it is also used in other gameplay modes, like cutscenes, menus, etc to store a variety of arbitrary sprites.

So I think we need to keep both variables names (wOAMBuffer and wLinkOAMBuffer), even if they refer to the same address technically. In the code, sometimes the code purpose is clearly to update Links sprites (and in this case wLinkOAMBuffer is better used). But otherwise, when the code is simply trying to manipulate arbitrary OAM data, wOAMBuffer is a better fit.

[zelosos]

I tried not to change the comments. Just relocate them.

[kemenaran]

Thanks for your work!

Also I created a follow-up Pull Request to fix the wOAMBuffer naming.