[openMSX-commits] SF.net SVN: openmsx:[12408] openmsx/trunk/src/sound/Y8950.cc

[<m97...@us...>]

Revision: 12408
          http://openmsx.svn.sourceforge.net/openmsx/?rev=12408&view=rev
Author:   m9710797
Date:     2012-02-18 09:19:02 +0000 (Sat, 18 Feb 2012)
Log Message:
-----------
Fixed Y8950::peekReg()

Manuel noticed the 'vu_meters' script was broken for MSX-AUDIO. After some
debugging it turned out the problem was that 'debug read "MSX-Audio regs" ...'
(almost) always returned the value 255. This change was introduced in rev12397,
where I 'fixed' this debuggable to not directly return the value of the
internal register file, but go via the peekReg() method which also takes
'behavior' into account (e.g. reading the adpcm data register returns a value
that is changing in time). Though it turned out that there was also a bug in
the peekReg() method itself, this method only worked for the registers with
extra behavior. Fixed now so that both ways work for both type of registers.

Revision Links:
--------------
    http://openmsx.svn.sourceforge.net/openmsx/?rev=12397&view=rev

Modified Paths:
--------------
    openmsx/trunk/src/sound/Y8950.cc

Modified: openmsx/trunk/src/sound/Y8950.cc
===================================================================
--- openmsx/trunk/src/sound/Y8950.cc	2012-02-12 19:34:45 UTC (rev 12407)
+++ openmsx/trunk/src/sound/Y8950.cc	2012-02-18 09:19:02 UTC (rev 12408)
@@ -1327,7 +1327,7 @@
 			return (output & enable) | (input & ~enable) | 0xF0;
 		}
 		default:
-			return 255;
+			return reg[rg];
 	}
 }
 

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.