diff options
author | Johannes Schickel | 2013-08-02 23:22:48 +0200 |
---|---|---|
committer | Johannes Schickel | 2013-08-03 02:52:32 +0200 |
commit | 2fdebe41b545a814b4eac83f24497ae194f255c6 (patch) | |
tree | ef3917ca39f031f20399ac9a85e7ee0b11c77293 /engines | |
parent | 1f0832b4f2ebb9abf798ea5c31ea16275e316d11 (diff) | |
download | scummvm-rg350-2fdebe41b545a814b4eac83f24497ae194f255c6.tar.gz scummvm-rg350-2fdebe41b545a814b4eac83f24497ae194f255c6.tar.bz2 scummvm-rg350-2fdebe41b545a814b4eac83f24497ae194f255c6.zip |
COMPOSER: Prefer getBasePtr over direct Surface::pixels access.
Diffstat (limited to 'engines')
-rw-r--r-- | engines/composer/graphics.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/engines/composer/graphics.cpp b/engines/composer/graphics.cpp index 2b68fac233..a751da20a3 100644 --- a/engines/composer/graphics.cpp +++ b/engines/composer/graphics.cpp @@ -39,7 +39,7 @@ bool Sprite::contains(const Common::Point &pos) const { return false; if (adjustedPos.y < 0 || adjustedPos.y >= _surface.h) return false; - byte *pixels = (byte *)_surface.pixels; + const byte *pixels = (const byte *)_surface.getBasePtr(0, 0); return (pixels[(_surface.h - adjustedPos.y - 1) * _surface.w + adjustedPos.x] != 0); } @@ -541,7 +541,7 @@ void ComposerEngine::redraw() { for (uint i = 0; i < _dirtyRects.size(); i++) { const Common::Rect &rect = _dirtyRects[i]; - byte *pixels = (byte *)_screen.pixels + (rect.top * _screen.pitch) + rect.left; + byte *pixels = (byte *)_screen.getBasePtr(rect.left, rect.top); _system->copyRectToScreen(pixels, _screen.pitch, rect.left, rect.top, rect.width(), rect.height()); } _system->updateScreen(); @@ -794,7 +794,7 @@ bool ComposerEngine::initSprite(Sprite &sprite) { if (width > 0 && height > 0) { sprite._surface.create(width, height, Graphics::PixelFormat::createFormatCLUT8()); - decompressBitmap(type, stream, (byte *)sprite._surface.pixels, size, width, height); + decompressBitmap(type, stream, (byte *)sprite._surface.getBasePtr(0, 0), size, width, height); } else { // there are some sprites (e.g. a -998x-998 one in Gregory's title screen) // which have an invalid size, but the original engine doesn't notice for @@ -814,13 +814,13 @@ void ComposerEngine::drawSprite(const Sprite &sprite) { int y = sprite._pos.y; // incoming data is BMP-style (bottom-up), so flip it - byte *pixels = (byte *)_screen.pixels; + byte *pixels = (byte *)_screen.getBasePtr(0, 0); for (int j = 0; j < sprite._surface.h; j++) { if (j + y < 0) continue; if (j + y >= _screen.h) break; - byte *in = (byte *)sprite._surface.pixels + (sprite._surface.h - j - 1) * sprite._surface.w; + const byte *in = (const byte *)sprite._surface.getBasePtr(0, sprite._surface.h - j - 1); byte *out = pixels + ((j + y) * _screen.w) + x; for (int i = 0; i < sprite._surface.w; i++) if ((x + i >= 0) && (x + i < _screen.w) && in[i]) |