00390: spaceftr: does not allow you to turn to the right.

ID	Category [?]	Severity [?]	Reproducibility	Date Submitted	Last Update
00390	DIP/Input	Major	Have not tried	Jan 29, 2008, 06:56	May 5, 2010, 19:25

Tester	qapo	View Status	Public	Platform
Assigned To	hap	Resolution	Fixed	OS
Status [?]	Resolved			Driver	cinematronics/cinemat.cpp
Version	0.95u6	Fixed in Version	0.138	Build
		Fixed in Git Commit		Github Pull Request #

Summary	00390: spaceftr: does not allow you to turn to the right.
Description	does not allow you to turn to the right. Whatever you push right or left you always turning left.
Steps To Reproduce
Additional Information	qapo
Github Commit

Flags
Regression Version	0.95u6
Affected Sets / Systems	spaceftr

Attached Files	DASM-Compare-1.png (9,857 bytes) Feb 28, 2010, 20:04
Attached Files	DASM-Compare-2.png (9,909 bytes) Feb 28, 2010, 20:04

No.05799 M.A.S.H. Senior Tester Feb 28, 2010, 20:04	The BAD_DUMP rom in spaceftr (Space Fortress (Zaccaria)) is fortresp7.7p. This rom has two different bytes as the rom starcas1, stellcas and starcase uses. They uses all the same rom (crc = d8f58d9a). If you use this rom for spaceftr, the spaceship can turn to the right direction again (!). If you compare the DASM from spaceftr with starcas1, stellcas or starcase you see that the two different bytes are the result of a bad dump. Space Fortress uses the same rom as starcas1, stellcas and starcase.
No.05800 Haze Senior Tester Feb 28, 2010, 20:11	yes, it looks like bitrot. 1 bit is different in each case.
No.06035 hap Developer May 2, 2010, 19:56	Indeed, only 2 bytes difference between fortresp7.7p (bad dump) and starcast.p7. Looking at the disassembly, these bytes clearly results in bogus code. Is anyone against it if we just assume fortresp7.7p rom is the same as its siblings? + include a /* note in source */ of course