aboutsummaryrefslogtreecommitdiff
path: root/scumm
diff options
context:
space:
mode:
authorMax Horn2004-08-14 19:42:00 +0000
committerMax Horn2004-08-14 19:42:00 +0000
commit152a5e97fe5197a5494fdde8967301888f360d2c (patch)
treeed39818d7a6b2f3e1fce0f3b379bedb3652f0d63 /scumm
parentd5f20cacd7061b4cff302f5b4f50c61a14461425 (diff)
downloadscummvm-rg350-152a5e97fe5197a5494fdde8967301888f360d2c.tar.gz
scummvm-rg350-152a5e97fe5197a5494fdde8967301888f360d2c.tar.bz2
scummvm-rg350-152a5e97fe5197a5494fdde8967301888f360d2c.zip
Modified VirtScreen to inherit from Graphics::Surface, and added getPixels()/getBackPixels() accessors (these changes make it a bit easier to implement smooth scrolling); also replaced some uses of rtBuffer by proper access through the corresponding VirtScreen
svn-id: r14612
Diffstat (limited to 'scumm')
-rw-r--r--scumm/base-costume.cpp10
-rw-r--r--scumm/charset.cpp30
-rw-r--r--scumm/cursor.cpp18
-rw-r--r--scumm/debugger.cpp4
-rw-r--r--scumm/gfx.cpp221
-rw-r--r--scumm/gfx.h22
-rw-r--r--scumm/object.cpp16
-rw-r--r--scumm/saveload.cpp6
-rw-r--r--scumm/script_v5.cpp4
-rw-r--r--scumm/script_v6.cpp10
-rw-r--r--scumm/script_v6he.cpp15
-rw-r--r--scumm/scumm.cpp2
-rw-r--r--scumm/scumm.h2
-rw-r--r--scumm/smush/smush_player.cpp2
-rw-r--r--scumm/verbs.cpp12
15 files changed, 182 insertions, 192 deletions
diff --git a/scumm/base-costume.cpp b/scumm/base-costume.cpp
index 718678ec99..6d5298e3fe 100644
--- a/scumm/base-costume.cpp
+++ b/scumm/base-costume.cpp
@@ -30,13 +30,13 @@ byte BaseCostumeRenderer::drawCostume(const VirtScreen &vs, const CostumeData &c
byte result = 0;
if (drawVirScr)
- _outptr = _vm->getResourceAddress(rtBuffer, 5) + vs.xstart;
+ _outptr = _vm->virtscr[0].getPixels(0, 0);
else
- _outptr = vs.screenPtr + vs.xstart;
+ _outptr = vs.getPixels(0, 0);
- _outwidth = vs.width;
- _outheight = vs.height;
- _numStrips = vs.width / 8;
+ _outwidth = vs.w;
+ _outheight = vs.h;
+ _numStrips = _outwidth / 8;
if (_vm->_version == 1) {
_xmove = 0;
diff --git a/scumm/charset.cpp b/scumm/charset.cpp
index 61a42f0313..ac90b4bb5b 100644
--- a/scumm/charset.cpp
+++ b/scumm/charset.cpp
@@ -980,7 +980,7 @@ void CharsetRendererV3::printChar(int chr) {
_textScreenID = vs->number;
}
if (_ignoreCharsetMask || !vs->hasTwoBuffers) {
- dest_ptr = vs->screenPtr + vs->xstart + drawTop * vs->width + _left;
+ dest_ptr = vs->getPixels(_left, drawTop);
} else {
dest_ptr = (byte *)_vm->gdi._textSurface.pixels + _top * _vm->gdi._textSurface.pitch + _left;
}
@@ -1096,14 +1096,14 @@ void CharsetRendererClassic::printChar(int chr) {
_textScreenID = vs->number;
}
if (_ignoreCharsetMask || !vs->hasTwoBuffers) {
- dst = vs->screenPtr + vs->xstart + drawTop * vs->width + _left;
+ dst = vs->getPixels(_left, drawTop);
} else {
dst = (byte *)_vm->gdi._textSurface.pixels + (_top - _vm->_screenTop) * _vm->gdi._textSurface.pitch + _left;
}
back = dst;
if (_blitAlso && vs->hasTwoBuffers) {
- dst = vs->backBuf + vs->xstart + drawTop * vs->width + _left;
+ dst = vs->getBackPixels(_left, drawTop);
}
if (is2byte) {
@@ -1120,11 +1120,12 @@ void CharsetRendererClassic::printChar(int chr) {
// once to each of the two buffers. That should hypothetically yield
// identical results, though I didn't try it and right now I don't know
// any spots where I can test this...
+ warning("This is broken?!");
int h = height;
do {
memcpy(back, dst, width);
- back += vs->width;
- dst += vs->width;
+ back += vs->pitch;
+ dst += vs->pitch;
} while (--h);
}
@@ -1151,7 +1152,7 @@ void CharsetRendererClassic::drawBitsN(VirtScreen *vs, byte *dst, const byte *sr
bits = *src++;
numbits = 8;
- for (y = 0; y < height && y + drawTop < vs->height; y++) {
+ for (y = 0; y < height && y + drawTop < vs->h; y++) {
for (x = 0; x < width; x++) {
color = (bits >> (8 - bpp)) & 0xFF;
@@ -1166,7 +1167,7 @@ void CharsetRendererClassic::drawBitsN(VirtScreen *vs, byte *dst, const byte *sr
numbits = 8;
}
}
- dst += vs->width - width;
+ dst += vs->pitch - width;
}
}
@@ -1174,22 +1175,22 @@ void CharsetRendererCommon::drawBits1(VirtScreen *vs, byte *dst, const byte *src
int y, x;
byte bits = 0;
- for (y = 0; y < height && y + drawTop < vs->height; y++) {
+ for (y = 0; y < height && y + drawTop < vs->h; y++) {
for (x = 0; x < width; x++) {
if ((x % 8) == 0)
bits = *src++;
if ((bits & revBitMask[x % 8]) && y + drawTop >= 0) {
if (_dropShadow) {
*(dst + 1) = _shadowColor;
- *(dst + vs->width) = _shadowColor;
- *(dst + vs->width + 1) = _shadowColor;
+ *(dst + vs->pitch) = _shadowColor;
+ *(dst + vs->pitch + 1) = _shadowColor;
}
*dst = _color;
}
dst++;
}
- dst += vs->width - width;
+ dst += vs->pitch - width;
}
}
@@ -1274,11 +1275,8 @@ void CharsetRendererNut::printChar(int chr) {
}
if (_ignoreCharsetMask) {
VirtScreen *vs = &_vm->virtscr[kMainVirtScreen];
- s.pixels = vs->screenPtr + vs->xstart + _vm->_screenTop * vs->width;
- s.w = vs->width;
- s.h = vs->height;
- s.pitch = vs->width;
- s.bytesPerPixel = 1;
+ s = *vs;
+ s.pixels = vs->getPixels(0, _vm->_screenTop);
} else {
s = _vm->gdi._textSurface;
}
diff --git a/scumm/cursor.cpp b/scumm/cursor.cpp
index fd8d03bc85..5cf17bc66f 100644
--- a/scumm/cursor.cpp
+++ b/scumm/cursor.cpp
@@ -84,7 +84,7 @@ void ScummEngine::grabCursor(int x, int y, int w, int h) {
return;
}
- grabCursor(vs->screenPtr + (y - vs->topline) * vs->width + x, w, h);
+ grabCursor((byte *)vs->pixels + (y - vs->topline) * vs->pitch + x, w, h);
}
@@ -119,15 +119,17 @@ void ScummEngine::useIm01Cursor(const byte *im, int w, int h) {
w *= 8;
h *= 8;
- dst = buf = (byte *) malloc(w * h);
- src = vs->screenPtr + vs->xstart;
+ // Backup the screen content
+ dst = buf = (byte *)malloc(w * h);
+ src = vs->getPixels(0, 0);
for (i = 0; i < h; i++) {
memcpy(dst, src, w);
dst += w;
- src += vs->width;
+ src += vs->pitch;
}
+ // Do some drawing
drawBox(0, 0, w - 1, h - 1, 0xFF);
vs->hasTwoBuffers = false;
@@ -136,14 +138,16 @@ void ScummEngine::useIm01Cursor(const byte *im, int w, int h) {
vs->hasTwoBuffers = true;
gdi.enableZBuffer();
- grabCursor(vs->screenPtr + vs->xstart, w, h);
+ // Grab the data we just drew and setup the cursor with it
+ grabCursor(vs->getPixels(0, 0), w, h);
+ // Restore the screen content
src = buf;
- dst = vs->screenPtr + vs->xstart;
+ dst = vs->getPixels(0, 0);
for (i = 0; i < h; i++) {
memcpy(dst, src, w);
- dst += vs->width;
+ dst += vs->pitch;
src += w;
}
diff --git a/scumm/debugger.cpp b/scumm/debugger.cpp
index 0a537399ea..5ad7f4811d 100644
--- a/scumm/debugger.cpp
+++ b/scumm/debugger.cpp
@@ -705,7 +705,7 @@ static void hlineColor(ScummEngine *scumm, int x1, int x2, int y, byte color) {
x2 = right - 1;
- ptr = vs->screenPtr + x1 + y * vs->width;
+ ptr = (byte *)vs->pixels + x1 + y * vs->pitch;
while (x1++ <= x2) {
*ptr++ = color;
@@ -784,7 +784,7 @@ void ScummDebugger::drawBox(int box) {
VirtScreen *vs = _vm->findVirtScreen(coords.ul.y);
if (vs != NULL)
- _vm->markRectAsDirty(vs->number, 0, vs->width, 0, vs->height);
+ _vm->markRectAsDirty(vs->number, 0, vs->w, 0, vs->h);
_vm->drawDirtyScreenParts();
_vm->_system->updateScreen();
}
diff --git a/scumm/gfx.cpp b/scumm/gfx.cpp
index 3e92c597aa..e8482229bb 100644
--- a/scumm/gfx.cpp
+++ b/scumm/gfx.cpp
@@ -28,6 +28,8 @@
namespace Scumm {
+static void blit(byte *dst, int dstPitch, const byte *src, int srcPitch, int w, int h);
+
struct StripTable {
int offsets[160];
int run[160];
@@ -235,14 +237,16 @@ void ScummEngine::initVirtScreen(VirtScreenNumber slot, int number, int top, int
}
vs->number = slot;
- vs->width = width;
+ vs->w = width;
vs->topline = top;
- vs->height = height;
+ vs->h = height;
vs->hasTwoBuffers = twobufs;
vs->xstart = 0;
vs->backBuf = NULL;
+ vs->bytesPerPixel = 1;
+ vs->pitch = width;
- size = vs->width * vs->height;
+ size = vs->pitch * vs->h;
if (scrollable) {
// Allow enough spaces so that rooms can be up to 4 resp. 8 screens
// wide. To achieve (horizontal!) scrolling, we use a neat trick:
@@ -251,15 +255,15 @@ void ScummEngine::initVirtScreen(VirtScreenNumber slot, int number, int top, int
// memory overhead (namely for every pixel we want to scroll, we need
// one additional byte in the buffer).
if (_version >= 7) {
- size += width * 8;
+ size += vs->pitch * 8;
} else {
- size += width * 4;
+ size += vs->pitch * 4;
}
}
createResource(rtBuffer, slot + 1, size);
- vs->screenPtr = getResourceAddress(rtBuffer, slot + 1);
- memset(vs->screenPtr, 0, size); // reset background
+ vs->pixels = getResourceAddress(rtBuffer, slot + 1);
+ memset(vs->pixels, 0, size); // reset background
if (twobufs) {
vs->backBuf = createResource(rtBuffer, slot + 5, size);
@@ -275,7 +279,7 @@ VirtScreen *ScummEngine::findVirtScreen(int y) {
int i;
for (i = 0; i < 3; i++, vs++) {
- if (y >= vs->topline && y < vs->topline + vs->height) {
+ if (y >= vs->topline && y < vs->topline + vs->h) {
return vs;
}
}
@@ -288,13 +292,13 @@ void ScummEngine::markRectAsDirty(VirtScreenNumber virt, int left, int right, in
if (left > right || top > bottom)
return;
- if (top > vs->height || bottom < 0)
+ if (top > vs->h || bottom < 0)
return;
if (top < 0)
top = 0;
- if (bottom > vs->height)
- bottom = vs->height;
+ if (bottom > vs->h)
+ bottom = vs->h;
if (virt == kMainVirtScreen && dirtybit) {
lp = left / 8 + _screenStartStrip;
@@ -355,8 +359,8 @@ void ScummEngine::drawDirtyScreenParts() {
// Small side note: most of our GFX code relies on this identity:
// gdi._numStrips * 8 == _screenWidth == vs->width
VirtScreen *vs = &virtscr[kMainVirtScreen];
- gdi.drawStripToScreen(vs, 0, vs->width, 0, vs->height);
- vs->setDirtyRange(vs->height, 0);
+ gdi.drawStripToScreen(vs, 0, vs->w, 0, vs->h);
+ vs->setDirtyRange(vs->h, 0);
} else {
updateDirtyScreen(kMainVirtScreen);
}
@@ -381,7 +385,7 @@ void ScummEngine::updateDirtyScreen(VirtScreenNumber slot) {
*/
void Gdi::updateDirtyScreen(VirtScreen *vs) {
// Do nothing for unused virtual screens
- if (vs->height == 0)
+ if (vs->h == 0)
return;
int i;
@@ -392,7 +396,7 @@ void Gdi::updateDirtyScreen(VirtScreen *vs) {
if (vs->bdirty[i]) {
const int top = vs->tdirty[i];
const int bottom = vs->bdirty[i];
- vs->tdirty[i] = vs->height;
+ vs->tdirty[i] = vs->h;
vs->bdirty[i] = 0;
if (i != (_numStrips - 1) && vs->bdirty[i + 1] == bottom && vs->tdirty[i + 1] == top) {
// Simple optimizations: if two or more neighbouring strips
@@ -419,11 +423,11 @@ void Gdi::drawStripToScreen(VirtScreen *vs, int x, int width, int top, int botto
if (bottom <= top)
return;
- if (top >= vs->height)
+ if (top >= vs->h)
return;
- assert(top >= 0 && bottom <= vs->height); // Paranoia checks
- assert(x >= 0 && width <= vs->width);
+ assert(top >= 0 && bottom <= vs->h); // Paranoia checks
+ assert(x >= 0 && width <= vs->w);
assert(_textSurface.pixels);
assert(_compositeBuf);
@@ -438,7 +442,7 @@ void Gdi::drawStripToScreen(VirtScreen *vs, int x, int width, int top, int botto
const int height = bottom - top;
// Compute screen etc. buffer pointers
- const byte *src = vs->screenPtr + (x + vs->xstart) + top * vs->width;
+ const byte *src = vs->getPixels(x, top);
byte *dst = _compositeBuf + x + y * _vm->_screenWidth;
const byte *text = (byte *)_textSurface.pixels + x + y * _textSurface.pitch;
@@ -450,7 +454,7 @@ void Gdi::drawStripToScreen(VirtScreen *vs, int x, int width, int top, int botto
else
dst[w] = text[w];
}
- src += vs->width;
+ src += vs->pitch;
dst += _vm->_screenWidth;
text += _textSurface.pitch;
}
@@ -601,7 +605,7 @@ void ScummEngine::redrawBGStrip(int start, int num) {
room = getResourceAddress(rtRoom, _roomResource);
gdi.drawBitmap(room + _IM00_offs,
- &virtscr[0], s, 0, _roomWidth, virtscr[0].height, s, num, 0, _roomStrips);
+ &virtscr[0], s, 0, _roomWidth, virtscr[0].h, s, num, 0, _roomStrips);
}
void ScummEngine::restoreBG(Common::Rect rect, byte backColor) {
@@ -616,19 +620,18 @@ void ScummEngine::restoreBG(Common::Rect rect, byte backColor) {
if ((vs = findVirtScreen(rect.top)) == NULL)
return;
- if (rect.left > vs->width)
+ if (rect.left > vs->w)
return;
// Convert 'rect' to local (virtual screen) coordinates
rect.top -= vs->topline;
rect.bottom -= vs->topline;
- rect.clip(vs->width, vs->height);
+ rect.clip(vs->w, vs->h);
markRectAsDirty(vs->number, rect, USAGE_BIT_RESTORED);
- const int offset = rect.top * vs->width + vs->xstart + rect.left;
- screenBuf = vs->screenPtr + offset;
+ screenBuf = vs->getPixels(rect.left, rect.top);
int height = rect.height();
int width = rect.width();
@@ -637,7 +640,7 @@ void ScummEngine::restoreBG(Common::Rect rect, byte backColor) {
return;
if (vs->hasTwoBuffers && _currentRoom != 0 && isLightOn()) {
- blit(screenBuf, vs->backBuf + offset, width, height);
+ blit(screenBuf, vs->pitch, vs->getBackPixels(rect.left, rect.top), vs->pitch, width, height);
if (vs->number == kMainVirtScreen && _charset->_hasMask) {
// Note: At first sight it may look as if this could
// be optimized to (rect.right - rect.left) / 8 and
@@ -653,7 +656,7 @@ void ScummEngine::restoreBG(Common::Rect rect, byte backColor) {
} else {
while (height--) {
memset(screenBuf, backColor, width);
- screenBuf += vs->width;
+ screenBuf += vs->pitch;
}
}
}
@@ -672,26 +675,26 @@ void CharsetRenderer::restoreCharsetBg() {
// currently covered by the charset mask.
VirtScreen *vs = &_vm->virtscr[_textScreenID];
- if (!vs->height)
+ if (!vs->h)
return;
- _vm->markRectAsDirty(vs->number, Common::Rect(vs->width, vs->height), USAGE_BIT_RESTORED);
+ _vm->markRectAsDirty(vs->number, Common::Rect(vs->w, vs->h), USAGE_BIT_RESTORED);
- byte *screenBuf = vs->screenPtr + vs->xstart;
+ byte *screenBuf = vs->getPixels(0, 0);
if (vs->hasTwoBuffers && _vm->_currentRoom != 0 && _vm->isLightOn()) {
- const byte *backBuf = vs->backBuf + vs->xstart;
+ const byte *backBuf = vs->getBackPixels(0, 0);
if (vs->number == kMainVirtScreen) {
// Clean out the charset mask
memset(_vm->gdi._textSurface.pixels, CHARSET_MASK_TRANSPARENCY, _vm->gdi._textSurface.pitch * _vm->gdi._textSurface.h);
} else {
// Restore from back buffer
- _vm->blit(screenBuf, backBuf, vs->width, vs->height);
+ blit(screenBuf, vs->pitch, backBuf, vs->pitch, vs->w, vs->h);
}
} else {
// Clear area
- memset(screenBuf, 0, vs->height * vs->width);
+ memset(screenBuf, 0, vs->h * vs->pitch);
}
}
}
@@ -714,22 +717,22 @@ byte *Gdi::getMaskBuffer(int x, int y, int z) {
#pragma mark --- Misc ---
#pragma mark -
-void ScummEngine::blit(byte *dst, const byte *src, int w, int h) {
+static void blit(byte *dst, int dstPitch, const byte *src, int srcPitch, int w, int h) {
assert(h > 0);
assert(src != NULL);
assert(dst != NULL);
// TODO: This function currently always assumes that srcPitch == dstPitch
// and furthermore that both equal _screenWidth.
+ //if (w==_screenWidth)
- if (w==_screenWidth)
- memcpy (dst, src, w*h);
- else
- {
+ if (w == srcPitch && w == dstPitch) {
+ memcpy(dst, src, w*h);
+ } else {
do {
memcpy(dst, src, w);
- dst += _screenWidth;
- src += _screenWidth;
+ dst += dstPitch;
+ src += srcPitch;
} while (--h);
}
}
@@ -758,45 +761,45 @@ void ScummEngine::drawBox(int x, int y, int x2, int y2, int color) {
// Clip the coordinates
if (x < 0)
x = 0;
- else if (x >= vs->width)
+ else if (x >= vs->w)
return;
if (x2 < 0)
return;
- else if (x2 > vs->width)
- x2 = vs->width;
+ else if (x2 > vs->w)
+ x2 = vs->w;
if (y < 0)
y = 0;
- else if (y > vs->height)
+ else if (y > vs->h)
return;
if (y2 < 0)
return;
- else if (y2 > vs->height)
- y2 = vs->height;
+ else if (y2 > vs->h)
+ y2 = vs->h;
markRectAsDirty(vs->number, x, x2, y, y2);
- backbuff = vs->screenPtr + vs->xstart + y * vs->width + x;
+ backbuff = vs->getPixels(x, y);
width = x2 - x;
height = y2 - y;
if (color == -1) {
if (vs->number != kMainVirtScreen)
error("can only copy bg to main window");
- bgbuff = vs->backBuf + vs->xstart + y * vs->width + x;
- blit(backbuff, bgbuff, width, height);
+ bgbuff = vs->getBackPixels(x, y);
+ blit(backbuff, vs->pitch, bgbuff, vs->pitch, width, height);
} else {
while (height--) {
memset(backbuff, color, width);
- backbuff += vs->width;
+ backbuff += vs->pitch;
}
}
}
void ScummEngine::drawFlashlight() {
- int i, j, offset, x, y;
+ int i, j, x, y;
VirtScreen *vs = &virtscr[kMainVirtScreen];
// Remove the flash light first if it was previously drawn
@@ -808,7 +811,7 @@ void ScummEngine::drawFlashlight() {
i = _flashlight.h;
do {
memset(_flashlight.buffer, 0, _flashlight.w);
- _flashlight.buffer += vs->width;
+ _flashlight.buffer += vs->pitch;
} while (--i);
}
_flashlight.isDrawn = false;
@@ -841,32 +844,31 @@ void ScummEngine::drawFlashlight() {
_flashlight.x = gdi._numStrips * 8 - _flashlight.w;
if (_flashlight.y < 0)
_flashlight.y = 0;
- else if (_flashlight.y + _flashlight.h> vs->height)
- _flashlight.y = vs->height - _flashlight.h;
+ else if (_flashlight.y + _flashlight.h> vs->h)
+ _flashlight.y = vs->h - _flashlight.h;
// Redraw any actors "under" the flashlight
for (i = _flashlight.x / 8; i < (_flashlight.x + _flashlight.w) / 8; i++) {
assert(0 <= i && i < gdi._numStrips);
setGfxUsageBit(_screenStartStrip + i, USAGE_BIT_DIRTY);
vs->tdirty[i] = 0;
- vs->bdirty[i] = vs->height;
+ vs->bdirty[i] = vs->h;
}
byte *bgbak;
- offset = _flashlight.y * vs->width + vs->xstart + _flashlight.x;
- _flashlight.buffer = vs->screenPtr + offset;
- bgbak = vs->backBuf + offset;
+ _flashlight.buffer = vs->getPixels(_flashlight.x, _flashlight.y);
+ bgbak = vs->getBackPixels(_flashlight.x, _flashlight.y);
- blit(_flashlight.buffer, bgbak, _flashlight.w, _flashlight.h);
+ blit(_flashlight.buffer, vs->pitch, bgbak, vs->pitch, _flashlight.w, _flashlight.h);
// Round the corners. To do so, we simply hard-code a set of nicely
// rounded corners.
static const int corner_data[] = { 8, 6, 4, 3, 2, 2, 1, 1 };
int minrow = 0;
int maxcol = _flashlight.w - 1;
- int maxrow = (_flashlight.h - 1) * vs->width;
+ int maxrow = (_flashlight.h - 1) * vs->pitch;
- for (i = 0; i < 8; i++, minrow += vs->width, maxrow -= vs->width) {
+ for (i = 0; i < 8; i++, minrow += vs->pitch, maxrow -= vs->pitch) {
int d = corner_data[i];
for (j = 0; j < d; j++) {
@@ -906,9 +908,9 @@ void Gdi::drawBitmapV2Helper(const byte *ptr, VirtScreen *vs, int x, int y, cons
memset(dither_table, 0, sizeof(dither_table));
if (vs->hasTwoBuffers)
- dst = vs->backBuf + (y * _numStrips + x) * 8;
+ dst = vs->backBuf + y * vs->pitch + x * 8;
else
- dst = vs->screenPtr + (y * _numStrips + x) * 8;
+ dst = (byte *)vs->pixels + y * vs->pitch + x * 8;
mask_ptr = getMaskBuffer(x, y, 1);
@@ -951,7 +953,7 @@ void Gdi::drawBitmapV2Helper(const byte *ptr, VirtScreen *vs, int x, int y, cons
}
if (left <= theX && theX < right) {
*dst = *ptr_dither_table++;
- dst += vs->width;
+ dst += vs->pitch;
}
}
if (left <= theX && theX < right) {
@@ -1099,11 +1101,11 @@ void Gdi::drawBitmap(const byte *ptr, VirtScreen *vs, int x, int y, const int wi
}
bottom = y + height;
- if (bottom > vs->height) {
- warning("Gdi::drawBitmap, strip drawn to %d below window bottom %d", bottom, vs->height);
+ if (bottom > vs->h) {
+ warning("Gdi::drawBitmap, strip drawn to %d below window bottom %d", bottom, vs->h);
}
- _vertStripNextInc = height * vs->width - 1;
+ _vertStripNextInc = height * vs->pitch - 1;
sx = x - vs->xstart / 8;
@@ -1132,7 +1134,7 @@ void Gdi::drawBitmap(const byte *ptr, VirtScreen *vs, int x, int y, const int wi
if (bottom > vs->bdirty[sx])
vs->bdirty[sx] = bottom;
- backbuff_ptr = vs->screenPtr + (y * _numStrips + x) * 8;
+ backbuff_ptr = (byte *)vs->pixels + (y * _numStrips + x) * 8;
if (vs->hasTwoBuffers)
bgbak_ptr = vs->backBuf + (y * _numStrips + x) * 8;
else
@@ -1267,7 +1269,7 @@ next_iter:
void Gdi::resetBackground(int top, int bottom, int strip) {
VirtScreen *vs = &_vm->virtscr[0];
byte *backbuff_ptr, *bgbak_ptr;
- int offs, numLinesToProcess;
+ int numLinesToProcess;
assert(0 <= strip && strip < _numStrips);
@@ -1277,9 +1279,8 @@ void Gdi::resetBackground(int top, int bottom, int strip) {
if (bottom > vs->bdirty[strip])
vs->bdirty[strip] = bottom;
- offs = top * vs->width + vs->xstart + strip * 8;
- bgbak_ptr = vs->backBuf + offs;
- backbuff_ptr = vs->screenPtr + offs;
+ bgbak_ptr = vs->getBackPixels(strip * 8, top);
+ backbuff_ptr = vs->getPixels(strip * 8, top);
numLinesToProcess = bottom - top;
if (numLinesToProcess) {
@@ -2297,7 +2298,7 @@ void ScummEngine::fadeOut(int effect) {
// Fill screen 0 with black
- memset(vs->screenPtr + vs->xstart, 0, vs->width * vs->height);
+ memset(vs->getPixels(0, 0), 0, vs->pitch * vs->h);
// Fade to black with the specified effect, if any.
switch (effect) {
@@ -2313,7 +2314,7 @@ void ScummEngine::fadeOut(int effect) {
break;
case 129:
// Just blit screen 0 to the display (i.e. display will be black)
- vs->setDirtyRange(0, vs->height);
+ vs->setDirtyRange(0, vs->h);
updateDirtyScreen(kMainVirtScreen);
break;
case 134:
@@ -2350,7 +2351,7 @@ void ScummEngine::transitionEffect(int a) {
int i, j;
int bottom;
int l, t, r, b;
- const int height = MIN((int)virtscr[0].height, _screenHeight);
+ const int height = MIN((int)virtscr[0].h, _screenHeight);
for (i = 0; i < 16; i++) {
delta[i] = transitionEffects[a].deltaTable[i];
@@ -2418,16 +2419,16 @@ void ScummEngine::dissolveEffect(int width, int height) {
// since we're only dealing with relatively small images, it shouldn't
// be too bad.
- w = vs->width / width;
- h = vs->height / height;
+ w = vs->w / width;
+ h = vs->h / height;
// When used correctly, vs->width % width and vs->height % height
// should both be zero, but just to be safe...
- if (vs->width % width)
+ if (vs->w % width)
w++;
- if (vs->height % height)
+ if (vs->h % height)
h++;
offsets = (int *) malloc(w * h * sizeof(int));
@@ -2441,7 +2442,7 @@ void ScummEngine::dissolveEffect(int width, int height) {
if (width == 1 && height == 1) {
// Optimized case for pixel-by-pixel dissolve
- for (i = 0; i < vs->width * vs->height; i++)
+ for (i = 0; i < vs->w * vs->h; i++)
offsets[i] = i;
for (i = 1; i < w * h; i++) {
@@ -2454,9 +2455,9 @@ void ScummEngine::dissolveEffect(int width, int height) {
} else {
int *offsets2;
- for (i = 0, x = 0; x < vs->width; x += width)
- for (y = 0; y < vs->height; y += height)
- offsets[i++] = y * vs->width + x;
+ for (i = 0, x = 0; x < vs->w; x += width)
+ for (y = 0; y < vs->h; y += height)
+ offsets[i++] = y * vs->pitch + x;
offsets2 = (int *) malloc(w * h * sizeof(int));
if (offsets2 == NULL) {
@@ -2493,9 +2494,9 @@ void ScummEngine::dissolveEffect(int width, int height) {
blits_before_refresh *= 2;
for (i = 0; i < w * h; i++) {
- x = offsets[i] % vs->width;
- y = offsets[i] / vs->width;
- _system->copyRectToScreen(vs->screenPtr + vs->xstart + y * vs->width + x, vs->width, x, y + vs->topline, width, height);
+ x = offsets[i] % vs->pitch;
+ y = offsets[i] / vs->pitch;
+ _system->copyRectToScreen(vs->getPixels(x, y), vs->pitch, x, y + vs->topline, width, height);
if (++blits >= blits_before_refresh) {
blits = 0;
@@ -2519,9 +2520,9 @@ void ScummEngine::scrollEffect(int dir) {
int step;
if ((dir == 0) || (dir == 1))
- step = vs->height;
+ step = vs->h;
else
- step = vs->width;
+ step = vs->w;
step = (step * kPictureDelay) / kScrolltime;
@@ -2529,12 +2530,12 @@ void ScummEngine::scrollEffect(int dir) {
case 0:
//up
y = 1 + step;
- while (y < vs->height) {
- _system->move_screen(0, -step, vs->height);
- _system->copyRectToScreen(vs->screenPtr + vs->xstart + (y - step) * vs->width,
- vs->width,
- 0, vs->height - step,
- vs->width, step);
+ while (y < vs->h) {
+ _system->move_screen(0, -step, vs->h);
+ _system->copyRectToScreen(vs->getPixels(0, y - step),
+ vs->pitch,
+ 0, vs->h - step,
+ vs->w, step);
_system->updateScreen();
waitForTimer(kPictureDelay);
@@ -2544,12 +2545,12 @@ void ScummEngine::scrollEffect(int dir) {
case 1:
// down
y = 1 + step;
- while (y < vs->height) {
- _system->move_screen(0, step, vs->height);
- _system->copyRectToScreen(vs->screenPtr + vs->xstart + vs->width * (vs->height-y),
- vs->width,
+ while (y < vs->h) {
+ _system->move_screen(0, step, vs->h);
+ _system->copyRectToScreen(vs->getPixels(0, vs->h - y),
+ vs->pitch,
0, 0,
- vs->width, step);
+ vs->w, step);
_system->updateScreen();
waitForTimer(kPictureDelay);
@@ -2559,12 +2560,12 @@ void ScummEngine::scrollEffect(int dir) {
case 2:
// left
x = 1 + step;
- while (x < vs->width) {
- _system->move_screen(-step, 0, vs->height);
- _system->copyRectToScreen(vs->screenPtr + vs->xstart + x - step,
- vs->width,
- vs->width - step, 0,
- step, vs->height);
+ while (x < vs->w) {
+ _system->move_screen(-step, 0, vs->h);
+ _system->copyRectToScreen(vs->getPixels(x - step, 0),
+ vs->pitch,
+ vs->w - step, 0,
+ step, vs->h);
_system->updateScreen();
waitForTimer(kPictureDelay);
@@ -2574,12 +2575,12 @@ void ScummEngine::scrollEffect(int dir) {
case 3:
// right
x = 1 + step;
- while (x < vs->width) {
- _system->move_screen(step, 0, vs->height);
- _system->copyRectToScreen(vs->screenPtr + vs->xstart + vs->width - x,
- vs->width,
+ while (x < vs->w) {
+ _system->move_screen(step, 0, vs->h);
+ _system->copyRectToScreen(vs->getPixels(vs->w - x, 0),
+ vs->pitch,
0, 0,
- step, vs->height);
+ step, vs->h);
_system->updateScreen();
waitForTimer(kPictureDelay);
diff --git a/scumm/gfx.h b/scumm/gfx.h
index 0bee946897..6f6b17be76 100644
--- a/scumm/gfx.h
+++ b/scumm/gfx.h
@@ -79,7 +79,7 @@ enum VirtScreenNumber {
* Each of these virtual screens has a fixed number or id (see also
* \ref VirtScreenNumber).
*/
-struct VirtScreen {
+struct VirtScreen : Graphics::Surface {
/**
* The unique id of this screen (corresponds to its position in the
* ScummEngine:virtscr array).
@@ -96,12 +96,6 @@ struct VirtScreen {
*/
uint16 topline;
- /** Width of the virtual screen (currently always identical to _screenWidth). */
- uint16 width;
-
- /** Height of the virtual screen. */
- uint16 height;
-
/**
* Horizontal scroll offset, tells how far the screen is scrolled to the
* right. Only used for the main screen. After all, verbs and the
@@ -120,12 +114,6 @@ struct VirtScreen {
bool hasTwoBuffers;
/**
- * Pointer to the screen's data buffer. This is where the content of
- * the screen is stored. Just as one would expect :-).
- */
- byte *screenPtr;
-
- /**
* Pointer to the screen's back buffer, if it has one (see also
* the hasTwoBuffers member).
* The backBuf is used by drawBitmap to store the background graphics of
@@ -167,6 +155,14 @@ struct VirtScreen {
bdirty[i] = bottom;
}
}
+
+ byte *getPixels(int x, int y) const {
+ return (byte *)pixels + xstart + y * pitch + x;
+ }
+
+ byte *getBackPixels(int x, int y) const {
+ return (byte *)backBuf + xstart + y * pitch + x;
+ }
};
/** Palette cycles */
diff --git a/scumm/object.cpp b/scumm/object.cpp
index 1f59bdb41c..8b0d6e65a5 100644
--- a/scumm/object.cpp
+++ b/scumm/object.cpp
@@ -1515,9 +1515,9 @@ void ScummEngine::drawBlastObject(BlastObject *eo) {
bdd.srcheight = READ_LE_UINT16(&((const BompHeader *)bomp)->old.height);
}
- bdd.out = vs->screenPtr + vs->xstart;
- bdd.outwidth = vs->width;
- bdd.outheight = vs->height;
+ bdd.out = vs->getPixels(0, 0);
+ bdd.outwidth = vs->w;
+ bdd.outheight = vs->h;
if (_version == 8) {
bdd.dataptr = bomp + 8; // Why this? See also useBompCursor
} else {
@@ -1570,17 +1570,17 @@ void ScummEngine::removeBlastObject(BlastObject *eo) {
r = eo->rect;
- if (r.bottom < 0 || r.right < 0 || r.top > vs->height || r.left > vs->width)
+ if (r.bottom < 0 || r.right < 0 || r.top > vs->h || r.left > vs->w)
return;
if (r.top < 0)
r.top = 0;
- if (r.bottom > vs->height)
- r.bottom = vs->height;
+ if (r.bottom > vs->h)
+ r.bottom = vs->h;
if (r.left < 0)
r.left = 0;
- if (r.right > vs->width)
- r.right = vs->width;
+ if (r.right > vs->w)
+ r.right = vs->w;
left_strip = r.left / 8;
right_strip = (r.right - 1) / 8;
diff --git a/scumm/saveload.cpp b/scumm/saveload.cpp
index 5130591437..c369032213 100644
--- a/scumm/saveload.cpp
+++ b/scumm/saveload.cpp
@@ -286,9 +286,9 @@ bool ScummEngine::loadState(int slot, bool compat) {
// Restore the virtual screens and force a fade to black.
initScreens(kMainVirtScreen, _screenHeight);
- VirtScreen *vs = &virtscr[0];
- memset(vs->screenPtr + vs->xstart, 0, vs->width * vs->height);
- vs->setDirtyRange(0, vs->height);
+ VirtScreen *vs = &virtscr[kMainVirtScreen];
+ memset(vs->getPixels(0, 0), 0, vs->pitch * vs->h);
+ vs->setDirtyRange(0, vs->h);
updateDirtyScreen(kMainVirtScreen);
updatePalette();
initScreens(sb, sh);
diff --git a/scumm/script_v5.cpp b/scumm/script_v5.cpp
index 7430d8ae87..7cae2d1930 100644
--- a/scumm/script_v5.cpp
+++ b/scumm/script_v5.cpp
@@ -2762,8 +2762,8 @@ void ScummEngine_v5::o5_oldRoomEffect() {
// For now, we force a redraw of the screen background. This
// Makes the Zak end credits work more or less correctly.
VirtScreen *vs = &virtscr[0];
- restoreBG(Common::Rect(0,vs->topline, vs->width, vs->topline + vs->height));
- virtscr[0].setDirtyRange(0, virtscr[0].height);
+ restoreBG(Common::Rect(0,vs->topline, vs->w, vs->topline + vs->h));
+ vs->setDirtyRange(0, vs->h);
updateDirtyScreen(kMainVirtScreen);
if (byte_2FCCF) {
diff --git a/scumm/script_v6.cpp b/scumm/script_v6.cpp
index d20ef2a95d..8c69af856f 100644
--- a/scumm/script_v6.cpp
+++ b/scumm/script_v6.cpp
@@ -2739,12 +2739,12 @@ void ScummEngine_v6::o6_kernelGetFunctions() {
{
VirtScreen *vs = &virtscr[0];
- if (args[1] < 0 || args[1] >= vs->width || args[2] < 0 || args[2] >= vs->height) {
+ if (args[1] < 0 || args[1] >= vs->w || args[2] < 0 || args[2] >= vs->h) {
// FIXME: Until we know what to do in this case...
- warning("o6_kernelGetFunctions:113: asking for pixel (%d, %d) outside of %dx%d screen", args[1], args[2], vs->width, vs->height);
+ warning("o6_kernelGetFunctions:113: asking for pixel (%d, %d) outside of %dx%d screen", args[1], args[2], vs->w, vs->h);
push(0);
} else
- push(vs->screenPtr[args[1] + args[2] * vs->width]);
+ push(*((byte *)vs->pixels + args[1] + args[2] * vs->pitch));
}
break;
case 115:
@@ -3123,9 +3123,7 @@ void ScummEngine_v6::o6_getPixel() {
return;
}
- int offset = (y - vs->topline) * vs->width + x + vs->xstart;
-
- byte area = *(vs->screenPtr + offset);
+ byte area = *vs->getPixels(x, y - vs->topline);
push(area);
}
diff --git a/scumm/script_v6he.cpp b/scumm/script_v6he.cpp
index 634fe1ea73..293162124b 100644
--- a/scumm/script_v6he.cpp
+++ b/scumm/script_v6he.cpp
@@ -771,16 +771,13 @@ void ScummEngine_v6he::virtScreenLoad(int resIdx, int x1, int y1, int x2, int y2
debug(1, "ScummEngine_v6he::virtScreenLoad(%d, %d, %d, %d, %d)", resIdx, x1, y1, x2, y2);
vsUnpackCtx ctx;
memset(&ctx, 0, sizeof(ctx));
- int vs = 0; // XXX gdi_virtScreen = 0;
+ VirtScreen &vs = virtscr[kMainVirtScreen]; // XXX gdi_virtScreen = 0;
ArrayHeader *ah = (ArrayHeader *)getResourceAddress(rtString, resIdx);
virtScreenLoadUnpack(&ctx, ah->data);
for (int j = y1; j <= y2; ++j) {
- uint32 yoff = (j - virtscr[kMainVirtScreen].topline) * 320;
- uint8 *p1 = getResourceAddress(rtBuffer, vs + 1);
- p1 += yoff + virtscr[kMainVirtScreen].xstart + x1;
- uint8 *p2 = getResourceAddress(rtBuffer, vs + 5);
- p2 += yoff + virtscr[kMainVirtScreen].xstart + x1;
+ uint8 *p1 = vs.getPixels(x1, j - vs.topline);
+ uint8 *p2 = vs.getBackPixels(x1, j - vs.topline);
if (x2 >= x1) {
uint32 w = x2 - x1 + 1;
while (w--) {
@@ -852,12 +849,10 @@ void ScummEngine_v6he::o6_kernelGetFunctions() {
int ScummEngine_v6he::virtScreenSave(byte *dst, int x1, int y1, int x2, int y2) {
debug(1, "ScummEngine_v6he::virtScreenSave(%d, %d, %d, %d)", x1, y1, x2, y2);
int packedSize = 0;
- int vs = 0; // XXX = gdi_virtScreen;
+ VirtScreen &vs = virtscr[kMainVirtScreen]; // XXX gdi_virtScreen = 0;
for (int j = y1; j <= y2; ++j) {
- uint8 *p = getResourceAddress(rtBuffer, vs + 5);
- p += virtscr[kMainVirtScreen].xstart;
- p += (j - virtscr[kMainVirtScreen].topline) * 320 + x1;
+ uint8 *p = vs.getBackPixels(x1, j - vs.topline);
int size = virtScreenSavePack(dst, p, x2 - x1 + 1, 0);
if (dst != 0) {
diff --git a/scumm/scumm.cpp b/scumm/scumm.cpp
index 965f0c72c5..ee623c10c4 100644
--- a/scumm/scumm.cpp
+++ b/scumm/scumm.cpp
@@ -1940,7 +1940,7 @@ void ScummEngine::processKbd(bool smushMode) {
if (_virtualMouse.y < 0)
_virtualMouse.y = -1;
- if (_virtualMouse.y >= virtscr[0].height)
+ if (_virtualMouse.y >= virtscr[0].h)
_virtualMouse.y = -1;
//
diff --git a/scumm/scumm.h b/scumm/scumm.h
index e23c1121e8..3b091558fe 100644
--- a/scumm/scumm.h
+++ b/scumm/scumm.h
@@ -972,8 +972,6 @@ protected:
void dissolveEffect(int width, int height);
void scrollEffect(int dir);
- void blit(byte *dst, const byte *src, int w, int h);
-
// bomp
public:
byte *_bompActorPalettePtr;
diff --git a/scumm/smush/smush_player.cpp b/scumm/smush/smush_player.cpp
index 8583151774..19d9833cbd 100644
--- a/scumm/smush/smush_player.cpp
+++ b/scumm/smush/smush_player.cpp
@@ -253,7 +253,7 @@ void SmushPlayer::init() {
_alreadyInit = false;
_vm->_smushVideoShouldFinish = false;
_vm->setDirtyColors(0, 255);
- _dst = _vm->virtscr[0].screenPtr + _vm->virtscr[0].xstart;
+ _dst = _vm->virtscr[0].getPixels(0, 0);
_smixer = new SmushMixer(_vm->_mixer);
g_timer->installTimerProc(&timerCallback, _speed, this);
}
diff --git a/scumm/verbs.cpp b/scumm/verbs.cpp
index 18654ebdc8..b210de6617 100644
--- a/scumm/verbs.cpp
+++ b/scumm/verbs.cpp
@@ -126,7 +126,7 @@ void ScummEngine::checkV2MouseOver(Common::Point pos) {
if (v2_mouseover_box != -1) {
rect = v2_mouseover_boxes[v2_mouseover_box].rect;
- dst = ptr = vs->screenPtr + vs->xstart + rect.top * vs->width + rect.left;
+ dst = ptr = vs->getPixels(rect.left, rect.top);
// Remove highlight.
for (y = rect.height() - 1; y >= 0; y--) {
@@ -134,7 +134,7 @@ void ScummEngine::checkV2MouseOver(Common::Point pos) {
if (dst[x] == v2_mouseover_boxes[v2_mouseover_box].hicolor)
dst[x] = v2_mouseover_boxes[v2_mouseover_box].color;
}
- dst += vs->width;
+ dst += vs->pitch;
}
markRectAsDirty(kVerbVirtScreen, rect);
@@ -143,7 +143,7 @@ void ScummEngine::checkV2MouseOver(Common::Point pos) {
if (new_box != -1) {
rect = v2_mouseover_boxes[new_box].rect;
- dst = ptr = vs->screenPtr + vs->xstart + rect.top * vs->width + rect.left;
+ dst = ptr = vs->getPixels(rect.left, rect.top);
// Apply highlight
for (y = rect.height() - 1; y >= 0; y--) {
@@ -151,7 +151,7 @@ void ScummEngine::checkV2MouseOver(Common::Point pos) {
if (dst[x] == v2_mouseover_boxes[new_box].color)
dst[x] = v2_mouseover_boxes[new_box].hicolor;
}
- dst += vs->width;
+ dst += vs->pitch;
}
markRectAsDirty(kVerbVirtScreen, rect);
@@ -209,9 +209,9 @@ void ScummEngine::redrawV2Inventory() {
// Clear on all invocations
inventoryBox.top = vs->topline + 32;
- inventoryBox.bottom = vs->topline + virtscr[2].height;
+ inventoryBox.bottom = vs->topline + virtscr[2].h;
inventoryBox.left = 0;
- inventoryBox.right = vs->width;
+ inventoryBox.right = vs->w;
restoreBG(inventoryBox);
_string[1].charset = 1;