Refactor DPC+ code wrt 'jitter'. The code and commenting now properly indicates that it is

a difference in mask values, and not actually jitter (TV 'jitter' effect is actually something else,
and is not causing the screen shaking here).

Changes.txt

@@ -63,12 +63,12 @@
 
   * Fixed 'Dancing Plate (Unknown) (PAL)' to use joystick.
 
-  * Allow the DPC+ scheme to not enable 'jitter' effect for certain older
+  * Allow the DPC+ scheme to not enable playfield 'jitter' effect for
-    DPC+ driver versions; this allows 'Epic Adventure' ROM to finally
+    certain older DPC+ driver versions; this allows 'Epic Adventure' ROM
-    work in Stella.
+    to finally work in Stella.
 
-  * PNG image support is now conditionally compiled into Stella.  All
+  * PNG/ZIP image support is now conditionally compiled into Stella.
-    major ports (Linux/macOS/Windows) have it enabled by default.
+    All major ports (Linux/macOS/Windows) have it enabled by default.
 
 -Have fun!
 

src/common/StateManager.cxx

@@ -27,7 +27,7 @@
 
 #include "StateManager.hxx"
 
-#define STATE_HEADER "06000003state"
+#define STATE_HEADER "06000002state"
 // #define MOVIE_HEADER "03030000movie"
 
 // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -

src/emucore/CartDPCPlus.cxx

@@ -37,7 +37,7 @@ CartridgeDPCPlus::CartridgeDPCPlus(const BytePtr& image, uInt32 size,
     myARMCycles(0),
     myFractionalClocks(0.0),
     myBankOffset(0),
-    myJitterEnabled(true)
+    myFractionalLowMask(0x0F00FF)
 {
   // Image is always 32K, but in the case of ROM > 29K, the image is
   // copied to the end of the buffer
@@ -65,10 +65,10 @@ CartridgeDPCPlus::CartridgeDPCPlus(const BytePtr& image, uInt32 size,
        Thumbulator::ConfigureFor::DPCplus,
        this);
 
-  // Disable jitter on certain DPC+ driver versions
+  // Currently only one known DPC+ ARM driver exhibits a problem
-  string driverMD5;
+  // with the default mask to use for DFxFRACLOW
-  driverMD5 = MD5::hash(image, 3*1024);
+  if(MD5::hash(image, 3*1024) == "8dd73b44fd11c488326ce507cbeb19d1")
-  myJitterEnabled = driverMD5 != "8dd73b44fd11c488326ce507cbeb19d1";
+    myFractionalLowMask = 0x0F0000;
 
   setInitialState();
 }
@@ -406,7 +406,7 @@ bool CartridgeDPCPlus::poke(uInt16 address, uInt8 value)
       // DFxFRACLOW - fractional data pointer low byte
       case 0x00:
         myFractionalCounters[index] =
-          (myFractionalCounters[index] & (myJitterEnabled ? 0x0F00FF : 0x0F0000)) | (uInt16(value) << 8);
+          (myFractionalCounters[index] & myFractionalLowMask) | (uInt16(value) << 8);
         break;
 
       // DFxFRACHI - fractional data pointer high byte
@@ -685,9 +685,6 @@ bool CartridgeDPCPlus::save(Serializer& out) const
 
     // Clock info for Thumbulator
     out.putLong(myARMCycles);
-
-    // Jitter effect
-    out.putBool(myJitterEnabled);
   }
   catch(...)
   {
@@ -749,9 +746,6 @@ bool CartridgeDPCPlus::load(Serializer& in)
 
     // Clock info for Thumbulator
     myARMCycles = in.getLong();
-
-    // Jitter effect
-    myJitterEnabled = in.getBool();
   }
   catch(...)
   {

src/emucore/CartDPCPlus.hxx

@@ -270,11 +270,12 @@ class CartridgeDPCPlus : public Cartridge
     // Indicates the offset into the ROM image (aligns to current bank)
     uInt16 myBankOffset;
 
-    // Indicates whether 'jitter' should be emulated
+    // Older DPC+ driver code had different behaviour wrt the mask used
-    // This is the default option, and should always be enabled
+    // to retrieve 'DFxFRACLOW' (fractional data pointer low byte)
-    // However, certain ROMs were released with older DPC+ ARM drivers,
+    // ROMs built with an old DPC+ driver and using the newer mask can
-    // before the scheme was finalized; this allows those ROMs to run
+    // result in 'jittering' in the playfield display
-    bool myJitterEnabled;
+    // For current versions, this is 0x0F00FF; older versions need 0x0F0000
+    uInt32 myFractionalLowMask;
 
   private:
     // Following constructors and assignment operators not supported