Checkpoint.

src/emucore/tia/FrameManager.cxx

@@ -65,8 +65,6 @@ void FrameManager::reset()
   myState = State::waitForVsyncStart;
   myCurrentFrameTotalLines = myCurrentFrameFinalLines = 0;
   myLineInState = 0;
-  myLinesWithoutVsync = 0;
-  myWaitForVsync = true;
   myVsync = false;
   myVblank = false;
   myTotalFrames = 0;
@@ -84,21 +82,11 @@ void FrameManager::nextLine()
   {
     case State::waitForVsyncStart:
     case State::waitForVsyncEnd:
-      if (myLinesWithoutVsync > myMaxLinesWithoutVsync) {
-        myWaitForVsync = false;
-        setState(State::waitForFrameStart);
-      }
       break;
 
     case State::waitForFrameStart:
-      if (myWaitForVsync) {
+      if (myLineInState >= (myVblank ? myVblankLines : myVblankLines - Metrics::maxUnderscan))
-        if (myLineInState >= (myVblank ? myVblankLines : myVblankLines - Metrics::maxUnderscan))
+        setState(State::frame);
-          setState(State::frame);
-      } else {
-        if (!myVblank) {
-          setState(State::frame);
-        }
-      }
       break;
 
     case State::frame:
@@ -107,17 +95,9 @@ void FrameManager::nextLine()
       }
       break;
 
-    case State::overscan:
-      if (myLineInState >= myOverscanLines - Metrics::visibleOverscan) {
-        setState(myWaitForVsync ? State::waitForVsyncStart : State::waitForFrameStart);
-      }
-      break;
-
     default:
       throw runtime_error("frame manager: invalid state");
   }
-
-  if (myWaitForVsync) myLinesWithoutVsync++;
 }
 
 // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
@@ -129,8 +109,6 @@ void FrameManager::setVblank(bool vblank)
 // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
 void FrameManager::setVsync(bool vsync)
 {
-  if (!myWaitForVsync || vsync == myVsync) return;
-
 #ifdef TIA_FRAMEMANAGER_DEBUG_LOG
   (cout << "vsync " << myVsync << " -> " << vsync << ": state " << int(myState) << " @ " << myLineInState << "\n").flush();
 #endif
@@ -141,14 +119,12 @@ void FrameManager::setVsync(bool vsync)
   {
     case State::waitForVsyncStart:
     case State::waitForFrameStart:
-    case State::overscan:
       if (myVsync) setState(State::waitForVsyncEnd);
       break;
 
     case State::waitForVsyncEnd:
       if (!myVsync) {
         setState(State::waitForFrameStart);
-        myLinesWithoutVsync = 0;
       }
       break;
 
@@ -220,7 +196,6 @@ void FrameManager::setTvMode(TvMode mode)
   }
 
   myFrameLines = Metrics::vsync + myVblankLines + myKernelLines + myOverscanLines;
-  myMaxLinesWithoutVsync = myFrameLines * Metrics::maxFramesWithoutVsync;
 }
 
 // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -

src/emucore/tia/FrameManager.hxx

@@ -76,8 +76,7 @@ class FrameManager : public Serializable
       waitForVsyncStart,
       waitForVsyncEnd,
       waitForFrameStart,
-      frame,
+      frame
-      overscan
     };
 
   private:
@@ -86,7 +85,7 @@ class FrameManager : public Serializable
 
     void setState(State state);
 
-    void finalizeFrame(State state = State::overscan);
+    void finalizeFrame(State state = State::waitForVsyncStart);
 
   private:
 
@@ -95,9 +94,7 @@ class FrameManager : public Serializable
 
     TvMode myMode;
     State myState;
-    bool myWaitForVsync;
     uInt32 myLineInState;
-    uInt32 myLinesWithoutVsync;
     uInt32 myCurrentFrameTotalLines;
     uInt32 myCurrentFrameFinalLines;
 
@@ -112,7 +109,6 @@ class FrameManager : public Serializable
     uInt32 myKernelLines;
     uInt32 myOverscanLines;
     uInt32 myFrameLines;
-    uInt32 myMaxLinesWithoutVsync;
 
   private:
     FrameManager(const FrameManager&) = delete;