Patch: Restored Ability Names (Update)

[DrakeyC]

If I'm understanding you correctly, your preference would be to have just one list, with a special-case redirect for the alternate Magitek Missile entry?
That would be less redundant, which would be nice.

Exactly. Though there wouldn't be a need for a special case redirect, as you've done before, could use custom characters to condense it a bit. Or could fall back on "Tek Missile" or even "Magic Missile".

It's doable but would require intrusive ad-hoc code which, IMO, is not worth it.
Also, an advantage of the current implementation is extensibility. Hackers can now customize the other Magitek abilities using oversized names without any further code change.

Also fair and true on both counts. For someone to use this patch then try longer Magitek names and discover they don't work, that would suck.

That said, the documentation definitely has room for improvement. There is little indication that the vanilla list is still being used at all.

Yeah, that confused me a bit. And specifying that there's a reason for two listings of their names would be important for hackers too, else they wonder why the abilities have different names between the menu and the attack window.

---

EDIT - In something helpful, iirc RAN has the word "Gil" pressed up against the amount in the "Got X Gil" text when winning a battle, yes? There's an unused text string that can be repurposed for this line and allow the space. At D1/F239 there's an unused string that reads "<variable> was crushed", unsure what it was gonna be for but it is unused. You can type out the gil string there and then redirect the message pointer, in vanilla it's at C2/5F95, but in my mod WIP it was at C2/5F8E, likely due to RAN and/or the Bugfix Comp moving it. Anywho, it looks like this; 

Code:

C2/5F95: A9 26        LDA #$26
C2/5F97: 20 D4 5F     JSR $5FD4      (buffer and display "Got [amount] GP" message)

Change the pointer to A9 1E and it'll load the "was crushed" string instead.