Hi folks. Thanks C.V. Reynolds for giving my patch a whirl, thanks to Leet Sketcher for finding a solution, and thanks to seibaby at ff6hacking for notifying me about this.
Ah, it would seem that all those LDA index-Y commands need the new Y, not whatever garbage was in there before.
I must not have realized that $0803 (from the instruction I delayed) holds "the most up-to-date offset for where to find the current direction". I do want to be a bit more confident that this is always the case, though I don't know if I'll have time right now to look very deep. So unless Leet Sketcher already did that digging to arrive at his solution, the confidence may just have to come over time through testing.
In the meantime, this weekend I will be posting updates to the patch (here and across the pond) to include Leet Sketcher's fix.
also, i think the new function could benefit from a data table. it'd save 20 bytes if we use the starts of the tables in the patch Readme, or 18 if we make our own 4-byte table.
I agree, and yes, the table in the Readme (every fourth byte starting with C0/581D) can be used. I will make that a part of the update.
wow, walking toward him? even this one is due to "stuck in a pose" patch?
Ouch, have I been making headaches all over the place with this thing? Sorry about that.
I'll post again soon.
