SGB border visual glitch

[chiascuro]

SGB border visual glitch

[endrift]

I thought I fixed this one already...it keeps coming back.

[endrift]

Ah so I wasn't crazy! I didn't realize you had a GitHub account or I would have tagged you. I was discussing this with beware on the gbdev Discord and he said that the SGBC implementation in bgb is kind of ad hoc and not necessarily known to be well behaved, so we agreed to clean it up to make it more consistent. Hopefully it'll lead to it being caught before release if it regresses again.

[endrift]

I remember now. This is actually a bug in Crystal Clear. I'm pretty sure it was fixed at some point. I backed out the patch that fixed it (8a87eaa) because it was clearly erroneous to include. It's not real hardware so there's no "well defined" way to do this, but it was special casing for one ROM hack so I didn't want to keep it. I guess bgb decided to leave that special casing in.

Please use an up-to-date version of Crystal Clear to confirm.

[chiascuro]

I tried it with the latest patch (2.5.10), I didn't find the files for 2.5.9 so I couldn't try it with the older one. I tried both, the normal version and the color filter version. I tried it on my current mgba installation and on the latest development build. The bug still persists

[endrift]

Hmm, alright, I'll take a closer look.

[endrift]

Reverting the reversion fixes it. Ugh. Looks like Crystal Clear still has the bug. I need to figure out if I work around their bug or pressure them to fix it instead.

[ShockSlayer]

No need for any pressure. I agree that mGBA should not have any special handling just for one hack. I thought I had fixed this on my end as of CC 2.5.3 by disabling OBJs when updating the border, but I've just checked both that version and the latest against mGBA 0.10.3 and I'm able to replicate this. I guess either I didn't properly fix this, or if I did, it was only partially, since updating the border seems to work correctly in most other places.

I've made a note of this, and I'll see if I can work it out before the next version. I apologize for any confusion and appreciate your insight into figuring this out.

[endrift]

@ShockSlayer While looking into this last night, beware noticed that you don't set BGP to $e4. Since this is a GBC-only game and that's a GB-only register that makes sense, but the SGB requires that for VRAM transfers to work properly. I don't think we're going to enforce that in SGBC, but it would be appreciated if you set it once on startup just for good measure, as that's what licensed titles that happen to upload borders in GBC mode do.

[ShockSlayer]

Cool. I'll make a note of that as well.