diff options
author | Max Horn | 2004-08-10 23:51:52 +0000 |
---|---|---|
committer | Max Horn | 2004-08-10 23:51:52 +0000 |
commit | dfc04a7d237218c48ecc7eb6ed94b67c3d4c471d (patch) | |
tree | 84e3f13665c89ea7e80b6e1fff6f06865fe22973 | |
parent | f50f7e143836a41cd9a90e55e5b614791f7cdb38 (diff) | |
download | scummvm-rg350-dfc04a7d237218c48ecc7eb6ed94b67c3d4c471d.tar.gz scummvm-rg350-dfc04a7d237218c48ecc7eb6ed94b67c3d4c471d.tar.bz2 scummvm-rg350-dfc04a7d237218c48ecc7eb6ed94b67c3d4c471d.zip |
Perform proper clipping (this fixes some graphic regressions in The Dig)
svn-id: r14547
-rw-r--r-- | scumm/gfx.cpp | 29 |
1 files changed, 17 insertions, 12 deletions
diff --git a/scumm/gfx.cpp b/scumm/gfx.cpp index 2717796fba..3e92c597aa 100644 --- a/scumm/gfx.cpp +++ b/scumm/gfx.cpp @@ -415,7 +415,6 @@ void Gdi::updateDirtyScreen(VirtScreen *vs) { * specified by top/bottom coordinate in the virtual screen. */ void Gdi::drawStripToScreen(VirtScreen *vs, int x, int width, int top, int bottom) { - const int height = bottom - top; if (bottom <= top) return; @@ -424,23 +423,28 @@ void Gdi::drawStripToScreen(VirtScreen *vs, int x, int width, int top, int botto return; assert(top >= 0 && bottom <= vs->height); // Paranoia checks - - // We don't clip height and width here, rather we rely on the backend to - // perform any needed clipping. - const int y = vs->topline + top - _vm->_screenTop; - const byte *src = vs->screenPtr + (x + vs->xstart) + top * vs->width; - + assert(x >= 0 && width <= vs->width); assert(_textSurface.pixels); assert(_compositeBuf); - Common::Rect r(x, y, x+width, y+height); - r.clip(Common::Rect(_textSurface.w, _textSurface.h)); - // TODO: is this enough clipping? + // Clip to the visible part of the scene + if (top < _vm->_screenTop) + top = _vm->_screenTop; + if (bottom > _vm->_screenTop + _vm->_screenHeight) + bottom = _vm->_screenTop + _vm->_screenHeight; + + // Convert the vertical coordinates to real screen coords + const int y = vs->topline + top - _vm->_screenTop; + const int height = bottom - top; + + // Compute screen etc. buffer pointers + const byte *src = vs->screenPtr + (x + vs->xstart) + top * vs->width; byte *dst = _compositeBuf + x + y * _vm->_screenWidth; const byte *text = (byte *)_textSurface.pixels + x + y * _textSurface.pitch; - for (int h = 0; h < r.height(); ++h) { - for (int w = 0; w < r.width(); ++w) { + // Compose the text over the game graphics + for (int h = 0; h < height; ++h) { + for (int w = 0; w < width; ++w) { if (text[w] == CHARSET_MASK_TRANSPARENCY) dst[w] = src[w]; else @@ -451,6 +455,7 @@ void Gdi::drawStripToScreen(VirtScreen *vs, int x, int width, int top, int botto text += _textSurface.pitch; } + // Finally blit the whole thing to the screen _vm->_system->copyRectToScreen(_compositeBuf + x + y * _vm->_screenWidth, _vm->_screenWidth, x, y, width, height); } |