Forgot to fix up TIA register writes in a few locations, so it was possible

that the register held values up to 255 (when it should only contain 0-159).
Thanks to Omegamatrix for the report and fix.

On another note, I hope to get back to Stella development soon; it's been a
rough time since the last release.


git-svn-id: svn://svn.code.sf.net/p/stella/code/trunk@3307 8b62c5a3-ac7e-4cc8-8f21-d9a121418aba

Changes.txt

@@ -12,6 +12,15 @@
                                Release History
 ===========================================================================
 
+4.7.2 to 4.8: (xxx. xx, 2016)
+
+  * Fixed HMOVE positioning bug that occurred under certain circumstances.
+    Thanks to Omegamatrix of AtariAge for the bug report and patch to fix
+    the issue.
+
+-Have fun!
+
+
 4.7.1 to 4.7.2: (Mar. 25, 2016)
 
   * Fixed bug when entering and exiting the debugger; sometimes the
@@ -27,8 +36,6 @@
     remove references to obsolete compiler versions that can no longer
     be used to compile Stella.
 
--Have fun!
-
 
 4.7 to 4.7.1: (Feb. 13, 2016)
 

src/emucore/TIA.cxx

@@ -1765,6 +1765,7 @@ bool TIA::poke(uInt16 addr, uInt8 value)
             break;
         }
         myPOSP0 = newx;
+        CLAMP_POS(myPOSP0);
       }
       break;
     }
@@ -1815,6 +1816,7 @@ bool TIA::poke(uInt16 addr, uInt8 value)
             break;
         }
         myPOSP1 = newx;
+        CLAMP_POS(myPOSP1);
       }
       break;
     }
@@ -1839,6 +1841,7 @@ bool TIA::poke(uInt16 addr, uInt8 value)
       if(newx != myPOSM0)
       {
         myPOSM0 = newx;
+        CLAMP_POS(myPOSM0);
       }
       break;
     }
@@ -1863,6 +1866,7 @@ bool TIA::poke(uInt16 addr, uInt8 value)
       if(newx != myPOSM1)
       {
         myPOSM1 = newx;
+        CLAMP_POS(myPOSM1);
       }
       break;
     }