From 0fc5ba914f7dab24046a631f889cf2c6db0a0cbe Mon Sep 17 00:00:00 2001
From: Brooke Vibber <bvibber@pobox.com>
Date: Sun, 29 Dec 2024 12:29:36 -0800
Subject: [PATCH] fix pan/zoom bug

was missing an rts on update_palette

this happened to fall through to keycheck
which if timing was wrong would dutifully process the viewport
change and return to update_palette's caller

which in turn was -not- expecting to reset the outer loop

fixed
---
 mandel.s | 2 ++
 todo.md  | 2 --
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/mandel.s b/mandel.s
index fe29001..d6ae4c6 100644
--- a/mandel.s
+++ b/mandel.s
@@ -1094,6 +1094,8 @@ done:
     and #$f0
     adc palette + 3
     sta COLOR2
+
+    rts
 .endproc
 
 .proc update_speed
diff --git a/todo.md b/todo.md
index ed4e628..aebaae3 100644
--- a/todo.md
+++ b/todo.md
@@ -1,7 +1,5 @@
 things to try:
 
-* fix the pan/zoom bug where it doesn't reset loop right :(
-
 * add some preset viewports that can be switched via number keys (1, 2, 3 etc)
 
 * patch the entire expanded-ram imul8xe on top of imul8 to avoid the 3-cycle thunk penalty :D