07-05-2021, 03:31 PM
Well, you don't need to worry about the SEP #$20 in this case. Serity's been teaching me about the stack variables and the accumulator and because you used PHP to push it, all that accumulator stuff will be reset with the PLP to pull it back.
I'm inclined to suggest that maybe you should explicitly set $32F4,X with the tool identity in the no_break branch. That would involve preserving that first A, though, before you even multiply it by 30... either in a scratch variable or by loading its value from the stack. The later requires an intimate knowledge of what's on the stack and how tall it is, though, so I'm not sure what value you'd use.
It uses the first in last out method, though... so perhaps it's as simply as LDA $02,S, because A was pushed onto the stack in the subroutine and then P was.
So... you could maybe try this for no_break:
I'm inclined to suggest that maybe you should explicitly set $32F4,X with the tool identity in the no_break branch. That would involve preserving that first A, though, before you even multiply it by 30... either in a scratch variable or by loading its value from the stack. The later requires an intimate knowledge of what's on the stack and how tall it is, though, so I'm not sure what value you'd use.
It uses the first in last out method, though... so perhaps it's as simply as LDA $02,S, because A was pushed onto the stack in the subroutine and then P was.
So... you could maybe try this for no_break:
Code:
no_break:
LDA #$10
TSB $B1 ; set flag to re-add item to inventory at the end of the character's turn
SEP #$10 ; Set 8-bit X and Y
TYX
LDA $02,S ; Load A's original value from the stack?
STA $32F4,X ; Save as tool to add back to inventory, just to be safe.
LDA $3018,X