Cannot disable lightgun crosshair

[sbugliarisi]

When the lightgun is enabled in the SMS core there's an option to turn on or off the crosshair but it always stays on

[birdybro]

https://github.com/MiSTer-devel/SMS_MiSTer/compare/master...birdybro:lightgun-fix

This should fix it. Please test for me. It looks like 2 of the conditions were not setup correctly from the beginning, you probably didn't get 3 different sizes of lightgun crosshair either ;)

SMS.zip

[sbugliarisi]

Hmm that didn't work. I copied the rbf to my MiSTer and named it "SMS-NEW", but when I run it there's no difference with the crosshair. It'll switch sizes but it won't turn off. Am I supposed to delete/rename my old SMS rbf?

On 10/3/22, birdybro @.***> wrote:

https://github.com/MiSTer-devel/SMS_MiSTer/compare/master...birdybro:lightgun-fix

This should fix it. Please test for me. It looks like 2 of the conditions were not setup correctly from the beginning, you probably didn't get 3 different sizes of lightgun crosshair either ;)

SMS.zip

-- Reply to this email directly or view it on GitHub: https://github.com/MiSTer-devel/SMS_MiSTer/issues/151#issuecomment-1266030284 You are receiving this because you authored the thread.

Message ID: @.***>

[birdybro]

Nope, you tested it right. It was kind of a guess on my part, I'll check with Shane as he did this change originally.

[birdybro]

Figured it out, didn't realize that the nand for 2'b11 of that status bit was missing :) Also didn't know how it was being drawn, but I see it now.

[sbugliarisi]

Awesome! Thanks birdybro! So I guess this will eventually roll out through an update to MiSTer?

[birdybro]

Yup, you can get it with update_all with the unstable nightlies builds in a bit, or wait for the next release.

[sbugliarisi]

You’re a prince among men, thanks birdybro!

[birdybro]

This can be closed, as it's resolved now.