X-Git-Url: http://shamusworld.gotdns.org/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=src%2Fscreen.cpp;h=bc733601fa4eda7620ce1030ddcd4e65a6ee897e;hb=ad913876dd8313ff082819d8cd5a921418670657;hp=5f92f223afeffc00cabb3d0023c932e4b8032e6e;hpb=daddbc234577dec4d318337f7ee0cf32517f0540;p=thunder diff --git a/src/screen.cpp b/src/screen.cpp index 5f92f22..bc73360 100644 --- a/src/screen.cpp +++ b/src/screen.cpp @@ -22,18 +22,21 @@ #include #include // For memset() #include "gui.h" +#include "video.h" // Private function prototypes void DrawSprites(uint8_t priority); int FindPCXName(void); -static void DrawChar(uint8_t * chr, uint8_t * ram, uint8_t sx, uint8_t sy, uint16_t scp, uint32_t baseAddr, uint32_t scrollOffset, bool transparent = true); +//static void DrawChar(uint8_t * chr, uint8_t * ram, uint8_t sx, uint8_t sy, uint16_t scp, uint32_t baseAddr, uint32_t scrollOffset, bool transparent = true); +static void DrawChar(uint8_t * chr, uint8_t * ram, uint8_t sx, uint8_t sy, uint16_t scp, uint32_t baseAddr, uint32_t xScroll = 0, uint32_t yScroll = 0, bool transparent = true); void Sprite(uint32_t sprnum, uint16_t x, uint16_t y, uint8_t flip, uint8_t horiz_bl, uint8_t vert_bl); // Private global variables uint8_t my_scr[0x14000]; // Screen buffer... -uint8_t palette[768]; // Screen palette +//uint8_t palette[768]; // Screen palette +uint32_t palette[256]; // Screen palette uint8_t ccolor[256][8]; // Character colors uint8_t scolor[128][16]; // Sprite colors bool charbase; // Character base pointer... @@ -41,7 +44,10 @@ uint8_t hScrollOffset; // Horizontal scroll offset uint8_t vScrollOffset; // Vertical scroll offset uint8_t spr_color_index; // Sprite color index uint32_t hoffsets[8] = { 0, 1, 2, 3, 4, 5, 6, 7 };// Scroll offsets... -uint32_t voffsets[8] = { 0, 320, 640, 960, 1280, 1600, 1920, 2240 }; +//uint32_t voffsets[8] = { 0, 320, 640, 960, 1280, 1600, 1920, 2240 }; +uint32_t voffsets[8] = { 288*0, 288*1, 288*2, 288*3, 288*4, 288*5, 288*6, 288*7 }; +//n.b.: 320 = $140 +// $000, $140, $280, $3C0, $500, $640, $780, $8C0 extern bool show_text; // Whether or not to show text extern bool show_scr; // Whether or not to show screen @@ -53,18 +59,18 @@ extern bool show_scr; // Whether or not to show screen void BlitChar(SDL_Surface * scr, uint8_t * chr, uint8_t * ram) { // Screen structure: - // Screen is 288 x 224 pixels, with character and independent sprites. Tiles are 36 x 28. - // There are four tile planes, three of them affected by the h/vscroll values, the last - // one is a static display (also has highest priority). Screens are 128 bytes wide by - // 32 bytes high. + // Screen is 288 x 224 pixels, with character and independent sprites. + // Tiles are 36 x 28. There are four tile planes, three of them affected by + // the h/vscroll values, the last one is a static display (also has highest + // priority). Screens are 128 bytes wide by 32 bytes high. if (show_scr) { uint32_t sc_base = ((ram[0x9000] << 8) | ram[0x9001]) + 4; // Adjust hscroll val - hScrollOffset = sc_base & 0x07; // Horiz. fine scroll offset - sc_base = (sc_base & 0xFFF8) >> 2; // Skip odds.. + hScrollOffset = sc_base & 0x07; // Horiz. fine scroll offset + sc_base = (sc_base & 0xFFF8) >> 2; // Skip odds.. uint8_t vsc_base = ((ram[0x9002] + 1) & 0xF8) >> 3;// Vertical scroll addr adjust - vScrollOffset = ((ram[0x9002] + 1) & 0x07); // Vertical fine scroll amount + vScrollOffset = ((ram[0x9002] + 1) & 0x07); // Vertical fine scroll amount uint32_t scp0 = 0x0180 | ((sc_base + 0x04) & 0x7F); /*0x0188;*/ uint32_t scp1 = 0x1180 | ((sc_base + 0x04) & 0x7F); /*0x1188;*/ @@ -72,101 +78,117 @@ void BlitChar(SDL_Surface * scr, uint8_t * chr, uint8_t * ram) uint32_t scp3 = 0x3208; scp0 += vsc_base * 0x80; - scp0 &= 0x0FFF; // Set vertical scroll addr + scp0 &= 0x0FFF; // Set vertical scroll addr scp1 += vsc_base * 0x80; - scp1 = 0x1000 | (scp1 & 0x0FFF); // Set vertical scroll addr + scp1 = 0x1000 | (scp1 & 0x0FFF); // Set vertical scroll addr scp2 += vsc_base * 0x80; - scp2 = 0x2000 | (scp2 & 0x0FFF); // Set vertical scroll addr + scp2 = 0x2000 | (scp2 & 0x0FFF); // Set vertical scroll addr // Layer 0 (bottom layer) for(uint8_t sy=0; sy<29; sy++) { for(uint8_t sx=0; sx<37; sx++) +#if 0 DrawChar(chr, ram, sx, sy, scp0, (charbase ? 0x20000 : 0x00000), 0, false); +#else + DrawChar(chr, ram, sx, sy, scp0, (charbase ? 0x20000 : 0x00000), hScrollOffset, vScrollOffset, false); +#endif scp0 += 0x80; scp0 = 0x0000 | (scp0 & 0x0FFF); } - DrawSprites(0x40); // Draw sprites at lowest layer... + // Draw sprites at lowest layer... + DrawSprites(0x40); // Layer 1 for(uint8_t sy=0; sy<29; sy++) { for(uint8_t sx=0; sx<37; sx++) +#if 0 DrawChar(chr, ram, sx, sy, scp1, (charbase ? 0x30000 : 0x10000), 0); +#else + DrawChar(chr, ram, sx, sy, scp1, (charbase ? 0x30000 : 0x10000), hScrollOffset, vScrollOffset); +#endif scp1 += 0x80; scp1 = 0x1000 | (scp1 & 0x0FFF); } - DrawSprites(0x80); // Draw sprites under layer #2... + // Draw sprites under layer #2... + DrawSprites(0x80); // Layer 2 for(uint8_t sy=0; sy<29; sy++) { for(uint8_t sx=0; sx<37; sx++) +#if 0 DrawChar(chr, ram, sx, sy, scp2, 0x40000, 0); +#else + DrawChar(chr, ram, sx, sy, scp2, 0x40000, hScrollOffset, vScrollOffset); +#endif scp2 += 0x80; scp2 = 0x2000 | (scp2 & 0x0FFF); } - DrawSprites(0xC0); // Draw highest priority sprites... + // Draw highest priority sprites... + DrawSprites(0xC0); // Layer 3 (top layer) for(uint8_t sy=0; sy<28; sy++) { for(uint8_t sx=0; sx<36; sx++) +#if 0 DrawChar(chr, ram, sx, sy, scp3, 0x50000, hScrollOffset + voffsets[vScrollOffset]); +#else + DrawChar(chr, ram, sx, sy, scp3, 0x50000); +#endif scp3 += 0x80; } } + // Draw a msg if needed... if (show_text) - DrawText(); // Draw a msg if needed... + DrawText(); + // Show GUI if active... if (ShowGUI()) - DrawGUI(); // Show GUI if active... + DrawGUI(); +#if 0 if (SDL_LockSurface(scr) < 0) { // fprintf(stderr, "Couldn't lock the display surface: %s\n", SDL_GetError()); // exit(2); } +#endif // Rolling Thunder screen size is 288 x 224. Virtual is this, real may not be... - // Doubled pixel blit (should be faster now!) - uint8_t * pMem = (uint8_t *)scr->pixels; - uint32_t src = (uint32_t)(hoffsets[hScrollOffset] + voffsets[vScrollOffset]), - dst1 = 0, dst2 = scr->pitch; - uint32_t srcAdd = 320 - VIRTUAL_SCREEN_WIDTH, dstAdd = (scr->pitch * 2) - (VIRTUAL_SCREEN_WIDTH * 2); + uint32_t src = 0;//(uint32_t)(hoffsets[hScrollOffset] + voffsets[vScrollOffset]); +// uint32_t srcAdd = 320 - VIRTUAL_SCREEN_WIDTH; for(int i=0; i= 288) + || ((yStart + y) < 0) || ((yStart + y) >= 224)) + { + sc_addr++; + chind++; + continue; + } + if (transparent) { if (chr[chind] != 7) @@ -192,7 +229,7 @@ static inline void DrawChar(uint8_t * chr, uint8_t * ram, uint8_t sx, uint8_t sy chind++; } - sc_addr += 312; // Do next line of char... + sc_addr += (288 - 8); // Do next line of char... } } @@ -202,16 +239,19 @@ static inline void DrawChar(uint8_t * chr, uint8_t * ram, uint8_t sx, uint8_t sy // void DrawSprites(uint8_t priority) { - // Sprite blocks: - // - // Offset Note - // ------ ----------------------------------------------------------------------------------- - // 4 h.fb .nnn (f = horz. flip, h = horz. expand, b = sprite offset lo bit, nnn = upper bits of sprite #) - // 5 Lower 7 bits of sprite # - // 6 Sprite color index (top 7 bits only), bottom bit is bit 8 of X position - // 7 Sprite X position (bits 0-7) - // 8 Top two bits are sprite priority, bits 4 & 2 are sprite offset hi bit, vert. expand - // 9 Sprite Y position (192 - value) +// Sprite blocks: +// +// Offset Note +// ------ -------------------------------------------------------------------- +// 4 h.fb .nnn (f = horz. flip, h = horz. expand, b = sprite offset lo +// bit, nnn = upper bits of sprite #) +// 5 Lower 7 bits of sprite # +// 6 Sprite color index (top 7 bits only), bottom bit is bit 8 of X +// position +// 7 Sprite X position (bits 0-7) +// 8 Top two bits are sprite priority, bits 4 & 2 are sprite offset hi +// bit, vert. expand +// 9 Sprite Y position (192 - value) extern uint8_t gram1[]; // Game RAM space @@ -227,8 +267,10 @@ void DrawSprites(uint8_t priority) uint16_t y = 192 - gram1[i + 9]; uint8_t flip = gram1[i + 4] & 0x20; // Horizontal flip - uint32_t spr_num = ((gram1[i + 4] & 0x07) << 9) | ((gram1[i + 5] & 0x7F) << 2) - | ((gram1[i + 4] & 0x10) >> 4) | ((gram1[i + 8] & 0x10) >> 3); + uint32_t spr_num = ((gram1[i + 4] & 0x07) << 9) + | ((gram1[i + 5] & 0x7F) << 2) + | ((gram1[i + 4] & 0x10) >> 4) + | ((gram1[i + 8] & 0x10) >> 3); // Draw sprite... Sprite(spr_num, x, y, flip, gram1[i + 4] & 0x80, gram1[i + 8] & 0x04); @@ -237,24 +279,78 @@ void DrawSprites(uint8_t priority) } +static inline void DrawSpriteBlock(uint32_t & sprnum, uint16_t x, uint16_t y, uint16_t xStart, uint16_t xEnd, int16_t xInc) +{ + extern uint8_t spr_rom[]; +// uint32_t sc_addr; + + for(uint16_t sy=0; sy<16; sy++) + { +// for(uint16_t sx=0; sx<16; sx+=2) + for(uint16_t sx=xStart; sx> 4, b2 = spr_rom[sprnum++] & 0x0F; + uint16_t spy = y + sy, spx = x + sx; // Need to optimize this clipping! [eh?] + + // This handles negative values, by casting as unsigned + uint32_t sc_addr = ((spy >= 224) || (spx >= 288) ? 0x13FFE : spx + (spy * 288)); + + if (b1 != 15) + my_scr[sc_addr] = scolor[spr_color_index][b1]; // Store it + + sc_addr++; + + if (b2 != 15) + my_scr[sc_addr] = scolor[spr_color_index][b2]; // Store it + } + } +} + + +static inline void DrawSpriteBlock2(uint32_t & sprnum, uint16_t x, uint16_t y, uint16_t xStart, uint16_t xEnd, int16_t xInc) +{ + extern uint8_t spr_rom[]; +// uint32_t sc_addr; + + for(uint16_t sy=0; sy<16; sy++) + { +// for(uint16_t sx=0; sx<16; sx+=2) + for(uint16_t sx=xStart; sx!=xEnd; sx+=xInc) + { + uint8_t b1 = spr_rom[sprnum] >> 4, b2 = spr_rom[sprnum++] & 0x0F; + uint16_t spy = y + sy, spx = x + sx; // Need to optimize this clipping! [eh?] + + // This handles negative values, by casting as unsigned + uint32_t sc_addr = ((spy >= 224) || (spx >= 288) ? 0x13FFE : spx + (spy * 288)); + + if (b2 != 15) + my_scr[sc_addr] = scolor[spr_color_index][b2]; // Store it + + sc_addr++; + + if (b1 != 15) + my_scr[sc_addr] = scolor[spr_color_index][b1]; // Store it + } + } +} + + // // Sprite handler // -void Sprite(uint32_t sprnum, uint16_t x, uint16_t y, uint8_t flip, uint8_t horiz_bl, uint8_t vert_bl) +void Sprite(uint32_t sprnum, uint16_t x, uint16_t y, uint8_t flip, + uint8_t horiz_bl, uint8_t vert_bl) { extern uint8_t spr_rom[]; uint32_t sc_addr; - - x += hScrollOffset; // Adjust x-coord - y += vScrollOffset; // Adjust y-coord - - sprnum <<= 7; // 128 bytes per sprite + sprnum <<= 7; // 128 bytes per sprite if (!vert_bl) y += 16; if (!flip) { +#if 0 for(uint16_t sy=0; sy<16; sy++) { for(uint16_t sx=0; sx<16; sx+=2) @@ -262,11 +358,17 @@ void Sprite(uint32_t sprnum, uint16_t x, uint16_t y, uint8_t flip, uint8_t horiz uint8_t b1 = spr_rom[sprnum] >> 4, b2 = spr_rom[sprnum++] & 0x0F; uint16_t spy = y + sy, spx = x + sx; // Need to optimize this clipping! - if (spy > 223 || spx > 299) + // This handles negative values, by casting as unsigned +#if 0 +// if (spy > 223 || spx > 299) + if (spy >= 224 || spx >= 288) sc_addr = 0x13FFE; else - sc_addr = spx + spy * 320; - +// sc_addr = spx + spy * 320; + sc_addr = spx + (spy * 288); +#else + sc_addr = ((spy >= 224) || (spx >= 288) ? 0x13FFE : spx + (spy * 288)); +#endif if (b1 != 15) my_scr[sc_addr] = scolor[spr_color_index][b1]; // Store it @@ -276,9 +378,13 @@ void Sprite(uint32_t sprnum, uint16_t x, uint16_t y, uint8_t flip, uint8_t horiz my_scr[sc_addr] = scolor[spr_color_index][b2]; // Store it } } +#else + DrawSpriteBlock(sprnum, x, y, 0, 16, 2); +#endif if (horiz_bl) { +#if 0 for(uint16_t sy=0; sy<16; sy++) { for(uint16_t sx=16; sx<32; sx+=2) @@ -286,10 +392,12 @@ void Sprite(uint32_t sprnum, uint16_t x, uint16_t y, uint8_t flip, uint8_t horiz uint8_t b1 = spr_rom[sprnum] >> 4, b2 = spr_rom[sprnum++] & 0x0F; uint16_t spy = y + sy, spx = x + sx; - if (spy > 223 || spx > 299) +// if (spy > 223 || spx > 299) + if (spy > 223 || spx > 287) sc_addr = 0x13FFE; else - sc_addr = spx + spy * 320; +// sc_addr = spx + spy * 320; + sc_addr = spx + (spy * 288); if (b1 != 15) my_scr[sc_addr] = scolor[spr_color_index][b1]; // Store it @@ -300,6 +408,9 @@ void Sprite(uint32_t sprnum, uint16_t x, uint16_t y, uint8_t flip, uint8_t horiz my_scr[sc_addr] = scolor[spr_color_index][b2]; // Store it } } +#else + DrawSpriteBlock(sprnum, x, y, 16, 32, 2); +#endif } else sprnum += 128; // Advance to next... @@ -308,6 +419,7 @@ void Sprite(uint32_t sprnum, uint16_t x, uint16_t y, uint8_t flip, uint8_t horiz { y += 16; // Do next row... +#if 0 for(uint16_t sy=0; sy<16; sy++) { for(uint16_t sx=0; sx<16; sx+=2) @@ -315,10 +427,12 @@ void Sprite(uint32_t sprnum, uint16_t x, uint16_t y, uint8_t flip, uint8_t horiz uint8_t b1 = spr_rom[sprnum] >> 4, b2 = spr_rom[sprnum++] & 0x0F; uint16_t spy = y + sy, spx = x + sx; - if (spy > 223 || spx > 299) +// if (spy > 223 || spx > 299) + if (spy > 223 || spx > 287) sc_addr = 0x13FFE; else - sc_addr = spx + spy * 320; +// sc_addr = spx + spy * 320; + sc_addr = spx + (spy * 288); if (b1 != 15) my_scr[sc_addr] = scolor[spr_color_index][b1]; // Store it @@ -329,9 +443,13 @@ void Sprite(uint32_t sprnum, uint16_t x, uint16_t y, uint8_t flip, uint8_t horiz my_scr[sc_addr] = scolor[spr_color_index][b2]; // Store it } } +#else + DrawSpriteBlock(sprnum, x, y, 0, 16, 2); +#endif if (horiz_bl) { +#if 0 for(uint16_t sy=0; sy<16; sy++) { for(uint16_t sx=16; sx<32; sx+=2) @@ -339,10 +457,12 @@ void Sprite(uint32_t sprnum, uint16_t x, uint16_t y, uint8_t flip, uint8_t horiz uint8_t b1 = spr_rom[sprnum] >> 4, b2 = spr_rom[sprnum++] & 0x0F; uint16_t spy = y + sy, spx = x + sx; - if (spy > 223 || spx > 299) +// if (spy > 223 || spx > 299) + if (spy > 223 || spx > 287) sc_addr = 0x13FFE; else - sc_addr = spx + spy * 320; +// sc_addr = spx + spy * 320; + sc_addr = spx + (spy * 288); if (b1 != 15) my_scr[sc_addr] = scolor[spr_color_index][b1]; // Store it @@ -353,6 +473,9 @@ void Sprite(uint32_t sprnum, uint16_t x, uint16_t y, uint8_t flip, uint8_t horiz my_scr[sc_addr] = scolor[spr_color_index][b2]; // Store it } } +#else + DrawSpriteBlock(sprnum, x, y, 16, 32, 2); +#endif } } } @@ -360,6 +483,7 @@ void Sprite(uint32_t sprnum, uint16_t x, uint16_t y, uint8_t flip, uint8_t horiz { if (horiz_bl) { +#if 0 for(uint16_t sy=0; sy<16; sy++) { for(uint16_t sx=30; sx!=14; sx-=2) @@ -367,10 +491,12 @@ void Sprite(uint32_t sprnum, uint16_t x, uint16_t y, uint8_t flip, uint8_t horiz uint8_t b1 = spr_rom[sprnum] >> 4, b2 = spr_rom[sprnum++] & 0x0F; uint16_t spy = y + sy, spx = x + sx; - if ((spy > 223) || (spx > 299)) +// if ((spy > 223) || (spx > 299)) + if ((spy > 223) || (spx > 287)) sc_addr = 0x13FFE; else - sc_addr = spx + spy * 320; +// sc_addr = spx + spy * 320; + sc_addr = spx + (spy * 288); if (b2 != 15) my_scr[sc_addr] = scolor[spr_color_index][b2]; // Store it @@ -381,8 +507,12 @@ void Sprite(uint32_t sprnum, uint16_t x, uint16_t y, uint8_t flip, uint8_t horiz my_scr[sc_addr] = scolor[spr_color_index][b1]; // Store it } } +#else + DrawSpriteBlock2(sprnum, x, y, 30, 14, -2); +#endif } +#if 0 for(uint16_t sy=0; sy<16; sy++) { for(uint16_t sx=14; sx!=0xFFFE; sx-=2) @@ -390,10 +520,12 @@ void Sprite(uint32_t sprnum, uint16_t x, uint16_t y, uint8_t flip, uint8_t horiz uint8_t b1 = spr_rom[sprnum] >> 4, b2 = spr_rom[sprnum++] & 0x0F; uint16_t spy = y + sy, spx = x + sx; - if ((spy > 223) || (spx > 299)) +// if ((spy > 223) || (spx > 299)) + if ((spy > 223) || (spx > 287)) sc_addr = 0x13FFE; else - sc_addr = spx+spy*320; +// sc_addr = spx + spy * 320; + sc_addr = spx + (spy * 288); if (b2 != 15) my_scr[sc_addr] = scolor[spr_color_index][b2]; // Store it @@ -404,8 +536,12 @@ void Sprite(uint32_t sprnum, uint16_t x, uint16_t y, uint8_t flip, uint8_t horiz my_scr[sc_addr] = scolor[spr_color_index][b1]; // Store it } } +#else + DrawSpriteBlock2(sprnum, x, y, 14, 0xFFFE, -2); +#endif - if (!horiz_bl) sprnum += 128; // If single, skip sprite... + if (!horiz_bl) + sprnum += 128; // If single, skip sprite... if (vert_bl) { @@ -413,6 +549,7 @@ void Sprite(uint32_t sprnum, uint16_t x, uint16_t y, uint8_t flip, uint8_t horiz if (horiz_bl) { +#if 0 for(uint16_t sy=0; sy<16; sy++) { for(uint16_t sx=30; sx!=14; sx-=2) @@ -420,10 +557,12 @@ void Sprite(uint32_t sprnum, uint16_t x, uint16_t y, uint8_t flip, uint8_t horiz uint8_t b1 = spr_rom[sprnum] >> 4, b2 = spr_rom[sprnum++] & 0x0F; uint16_t spy = y + sy, spx = x + sx; - if ((spy > 223) || (spx > 299)) +// if ((spy > 223) || (spx > 299)) + if ((spy > 223) || (spx > 287)) sc_addr = 0x13FFE; else - sc_addr = spx + spy * 320; +// sc_addr = spx + spy * 320; + sc_addr = spx + (spy * 288); if (b2 != 15) my_scr[sc_addr] = scolor[spr_color_index][b2]; // Store it @@ -434,7 +573,12 @@ void Sprite(uint32_t sprnum, uint16_t x, uint16_t y, uint8_t flip, uint8_t horiz my_scr[sc_addr] = scolor[spr_color_index][b1]; // Store it } } +#else + DrawSpriteBlock2(sprnum, x, y, 30, 14, -2); +#endif } + +#if 0 for(uint16_t sy=0; sy<16; sy++) { for(uint16_t sx=14; sx!=0xFFFE; sx-=2) @@ -442,10 +586,12 @@ void Sprite(uint32_t sprnum, uint16_t x, uint16_t y, uint8_t flip, uint8_t horiz uint8_t b1 = spr_rom[sprnum] >> 4, b2 = spr_rom[sprnum++] & 0x0F; uint16_t spy = y + sy, spx = x + sx; - if ((spy > 223) || (spx > 299)) +// if ((spy > 223) || (spx > 299)) + if ((spy > 223) || (spx > 287)) sc_addr = 0x13FFE; else - sc_addr = spx + spy * 320; +// sc_addr = spx + spy * 320; + sc_addr = spx + (spy * 288); if (b2 != 15) my_scr[sc_addr] = scolor[spr_color_index][b2]; // Store it @@ -456,14 +602,17 @@ void Sprite(uint32_t sprnum, uint16_t x, uint16_t y, uint8_t flip, uint8_t horiz my_scr[sc_addr] = scolor[spr_color_index][b1]; // Store it } } +#else + DrawSpriteBlock2(sprnum, x, y, 14, 0xFFFE, -2); +#endif } } } -int FindPCXName() +int FindPCXName(void) { - static int pcxNum = -1; // This needs to go elsewhere... (or does it?) + static int pcxNum = -1; // This needs to go elsewhere... (or does it?) char filename[30]; FILE * fr; @@ -473,8 +622,9 @@ int FindPCXName() { sprintf(filename, "thnd%04i.pcx", pcxNum); + // file does not exist - we can create it if ((fr = fopen(filename, "r")) == NULL) - return pcxNum; // file does not exist - we can create it + return pcxNum; pcxNum++; } @@ -559,7 +709,9 @@ void SnapPCX(SDL_Surface * scr) while (pMem[mem] == last && xpos < bytesPerLine && count < 63) { - mem++; count++; xpos++; + mem++; + count++; + xpos++; } if (count > 1 || (last & 0xC0) == 0xC0) @@ -570,6 +722,7 @@ void SnapPCX(SDL_Surface * scr) else fputc(last & 0xFF, fw); } + mem += (scr->pitch - 320); // Handle non-standard line lengths... }