Redux-Team / SM63Redux

Code base for Super Mario 63 Redux
https://sm63redux.com
Mozilla Public License 2.0
202 stars 27 forks source link

Refactor LD to use StringName item IDs instead of ints #301

Open Koopa1018 opened 3 months ago

Koopa1018 commented 3 months ago

Description of changes

Exactly what it says on the tin. Also, comment and cleanup (I had left some functions literally marked "does nothing" because I couldn't tell what they output--even though I knew commenting them out broke the code--so now that I've figured it out, I've gone back and clarified).

Issue(s)

Meant to facilitate using human-readable item IDs when we switch to JSON level serialization.

Future work

items.xml.tres still uses ints for its IDs. The ints are stored as strings, so it still works, but it's far from human-readable yet.