Skip to content

Cleanup of the XGA mapping to be less messed up.#4403

Merged
OBattler merged 1 commit into
masterfrom
tc1995
Apr 26, 2024
Merged

Cleanup of the XGA mapping to be less messed up.#4403
OBattler merged 1 commit into
masterfrom
tc1995

Conversation

@TC1995

@TC1995 TC1995 commented Apr 26, 2024

Copy link
Copy Markdown
Contributor

Summary

As well as the cursor/sprite being finally fixed when the xoff goes from 0x20 onwards. This makes Win3.x' XGA cursor look normal and everything else as well. (as in, intact).

Checklist

References

XGA 1-2 manual (for the mapping)

As well as the cursor/sprite being finally fixed when the xoff goes from 0x20 onwards. This makes Win3.x' XGA cursor look normal and everything else as well. (as in, intact).
@OBattler
OBattler merged commit dd58fbf into master Apr 26, 2024
Comment thread src/video/vid_xga.c
if (x >= idx) {
if (!(x & 0x03))
dat = xga->sprite_data[xga->hwcursor_latch.addr & 0x3ff];
for (uint8_t x = 0; x < xga->hwcursor_latch.cur_xsize; x++) {

Check failure

Code scanning / CodeQL

Comparison of narrow type with wide type in loop condition

Comparison between [x](1) of type uint8_t and [cur_xsize](2) of wider type int.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants