From 18c5867d9ccfbaadc4bfbee406bb5aff2d51b194 Mon Sep 17 00:00:00 2001
From: tildearrow <tildearrow@protonmail.com>
Date: Sun, 20 Feb 2022 23:03:42 -0500
Subject: [PATCH] fix playSub strategy

properly handle nested playSubs
fixes #174
---
 src/engine/engine.cpp | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/engine/engine.cpp b/src/engine/engine.cpp
index 3069f9193..2f3d75e9a 100644
--- a/src/engine/engine.cpp
+++ b/src/engine/engine.cpp
@@ -743,6 +743,7 @@ void DivEngine::getCommandStream(std::vector<DivCommand>& where) {
 }
 
 void DivEngine::playSub(bool preserveDrift, int goalRow) {
+  for (int i=0; i<song.systemLen; i++) disCont[i].dispatch->setSkipRegisterWrites(false);
   reset();
   if (preserveDrift && curOrder==0) return;
   bool oldRepeatPattern=repeatPattern;
@@ -767,11 +768,11 @@ void DivEngine::playSub(bool preserveDrift, int goalRow) {
   playing=true;
   for (int i=0; i<song.systemLen; i++) disCont[i].dispatch->setSkipRegisterWrites(true);
   while (playing && curOrder<goal) {
-    if (nextTick(preserveDrift)) break;
+    if (nextTick(preserveDrift)) return;
   }
   int oldOrder=curOrder;
   while (playing && curRow<goalRow) {
-    if (nextTick(preserveDrift)) break;
+    if (nextTick(preserveDrift)) return;
     if (oldOrder!=curOrder) break;
   }
   for (int i=0; i<song.systemLen; i++) disCont[i].dispatch->setSkipRegisterWrites(false);