Viewing Issue Advanced Details Jump to Notes ] [ View Source: cps2.c ]
ID Category [?] Severity [?] Reproducibility Date Submitted Last Update
04552 Graphics Minor Always 2011-12-07 02:21 2011-12-09 20:19
Tester MetalliC View Status Public Platform MAME (Self-compiled)
Assigned To hap Resolution Fixed OS Windows Vista/7 (64-bit)
Status [?] Resolved   Driver cps2.c
Version 0.144u2 Fixed in Version 0.144u3 Build 64-bit
Summary 04552: ssf2t and clones: flashing garbage during intro
Description there can be seen some trash onscreen during "flashing" parts of intro, especially at the end.

144u1 is OK. probably that deprecat.h removing in u2 caused regression.
Steps To Reproduce
Additional Information
Regression Version 0.144u1
Affected Sets / Systems ssf2t and clones
Attached Files
png file icon 0001.png (6,065 bytes)  2011-12-07 09:38 by MetalliC

-  Notes
User avatar (07948)
Layne (Tester)
2011-12-07 08:53

Are you referring to this?

If yes it's not a bug.
User avatar (07949)
MetalliC (Developer)
2011-12-07 09:44

No, I am talking about another issue.
And as was said - it happens every time, and very clearly seen, just need to wait or throttle until Ryu do a fireball in intro.
User avatar (07950)
Fujix (Administrator)
2011-12-07 10:59

I tested with 0.130 for regression test, it doesn't show the full screen garbage screen.
User avatar (07953)
AWJ (Developer)
2011-12-07 17:24
edited on: 2011-12-07 18:07

The intro isn't the only regression; there are graphic errors during gameplay too. They don't happen all the time (or even all the time on specific stages) but seem to be random... so far I've confirmed them on Cammy's, Chun Li's and Guile's stages. In Cammy's stages, the aurora is sometimes in the wrong place and the wrong priority (covering part of the castle walls). In the other stages, the linescrolling region of the floor jitters when a character jumps.

User avatar (07954)
hap (Developer)
2011-12-07 20:13
edited on: 2011-12-07 20:18

Yes, timing is off now. This means cps2_interrupt will have to be updated (no use for scancount either anymore when scanline is in param).

Old timing really starts at scanline 240 with vblank_int_hack, this makes scancount 0 when it's not. In the current implementation, I think changing driver_init to state->m_scancount = 19; will bring the timing back to how it was.

*edit*.. And no, simply doing +19 is not a good fix. :P I added that info for us to realize how bad the implementation of irq timing was/is in cps2.c

User avatar (07956)
MetalliC (Developer)
2011-12-09 20:19

issue was resolved, thanks hap. close it pls