unconditional branch wasn't always taken because stepptr doesn't always return with Z=0

2024-12-25 13:33:13 +00:00 · 2021-11-21 15:34:44 -05:00 · 2021-11-21 15:34:44 -05:00 · 9f5fd1e382
commit 9f5fd1e382
parent 5db033f8ef
1 changed files with 5 additions and 2 deletions
--- a/src/okvs.a
+++ b/src/okvs.a
@ -395,8 +395,11 @@ okvs_next_field
         +ST16 PTR
 okvs_next_field_PTR_is_already_set
         jsr   stepptr
-         bne   incptr
-
+         jmp   incptr                ; do NOT change this to BNE
+                                     ; because if the low byte of PTR is 0
+                                     ; and the value of (PTR) is also 0
+                                     ; then stepptr will return with Z=1 and
+                                     ; we will fall through here unexpectedly
 incptr4  jsr   incptr2
 incptr2  jsr   incptr
 incptr