03628: alpine: Red point on the title screen

ID	Category [?]	Severity [?]	Reproducibility	Date Submitted	Last Update
03628	Graphics	Trivial	Always	Jan 2, 2010, 01:34	May 9, 2010, 17:14

Tester	M.A.S.H.	View Status	Public	Platform	MAME (Self-compiled)
Assigned To	M.A.S.H.	Resolution	Fixed	OS	Windows XP (32-bit)
Status [?]	Resolved			Driver	taito/taitosj.cpp
Version	0.136	Fixed in Version	0.138	Build	Normal
		Fixed in Git Commit		Github Pull Request #

Summary	03628: alpine: Red point on the title screen
Description	There is a red point on the title screen since MAME 0.122u5 (see snapshot). Zsolt Vasvari made a general Taito SJ code clean-up in 0.122u5. Complete memory map from Nicola's ancient notes. So my guess is that a size typo was made (1 and not 0 => 1-256; 0-255) or some sort of decimal/hex.
Steps To Reproduce
Additional Information
Github Commit

Flags
Regression Version	0.122u5
Affected Sets / Systems	alpine

Attached Files	alpine.png (2,219 bytes) Jan 2, 2010, 01:34
Attached Files	taitosj-Driver-Compare.png (18,987 bytes) Mar 8, 2010, 12:34

No.05843 M.A.S.H. Senior Tester Mar 8, 2010, 12:34	I found the problem with the red point in mame\video\taitosj.c driver. For me it looks like a typo (see attached compare pic) and if you changed in MAME 0.136u4 mame\video\taitosj.c line 275 from: sx = taitosj_spriteram[SPRITE_RAM_PAGE_OFFSET + offs + 0]; back to sx = taitosj_spriteram[SPRITE_RAM_PAGE_OFFSET - 1]; the red point dissapear.
No.05845 Tafoid Administrator Mar 8, 2010, 22:17 edited on: Mar 8, 2010, 22:17	Please do not assign bugs to other devs. If you were going to work on it, assign yourself - otherwise leave it alone. If you were wondering, there is an empty line at the very TOP of the assign list - which can be used to withdraw from a bug.
No.06060 hap Developer May 9, 2010, 17:12	Agreed about the accidental error during cleanup. More so because it fixes Elevator Action.