fixed bug with advancing tape counter if no input events have been recorded

[rocksndiamonds.git] / src / tape.c

diff --git a/src/tape.c b/src/tape.c
index 71f707ced8a8ed3e1ade522fbb0940379c54f387..0a7dffc3db840681520a6c5fdb371cb155009772 100644 (file)
--- a/src/tape.c
+++ b/src/tape.c
@@ -779,7 +779,9 @@ static void TapeAppendRecording(void)
 
 void TapeHaltRecording(void)
 {
-  tape.counter++;
+  // only advance tape counter if any input events have been recorded
+  if (tape.pos[tape.counter].delay > 0)
+    tape.counter++;
 
   // initialize delay for next tape entry (to be able to continue recording)
   if (tape.counter < MAX_TAPE_LEN)
@@ -834,6 +836,8 @@ boolean TapeAddAction(byte action[MAX_TAPE_ACTIONS])
     tape.pos[tape.counter].delay++;
   }
 
+  tape.changed = TRUE;
+
   return TRUE;
 }
 
@@ -1185,16 +1189,13 @@ static void TapeSingleStep(void)
 
 void TapeQuickSave(void)
 {
-  if (game_status == GAME_MODE_MAIN)
+  if (game_status != GAME_MODE_PLAYING)
   {
     Request("No game that could be saved!", REQ_CONFIRM);
 
     return;
   }
 
-  if (game_status != GAME_MODE_PLAYING)
-    return;
-
   if (!tape.recording)
   {
     Request("No recording that could be saved!", REQ_CONFIRM);
@@ -1202,8 +1203,7 @@ void TapeQuickSave(void)
     return;
   }
 
-  if (tape.recording)
-    TapeHaltRecording();       // prepare tape for saving on-the-fly
+  TapeHaltRecording();         // prepare tape for saving on-the-fly
 
   if (TAPE_IS_EMPTY(tape))
   {