diff options
| author | Bendegúz Nagy | 2016-09-12 15:49:25 +0200 |
|---|---|---|
| committer | Bendegúz Nagy | 2016-09-12 15:49:25 +0200 |
| commit | 06d6e118eb7d6bace91837f155429e55bd425e76 (patch) | |
| tree | f007ab57a153f3ce38706b28636299625d244b20 /engines | |
| parent | 3b3572a110dc1c4e6956043d37fa166d83cad34f (diff) | |
| download | scummvm-rg350-06d6e118eb7d6bace91837f155429e55bd425e76.tar.gz scummvm-rg350-06d6e118eb7d6bace91837f155429e55bd425e76.tar.bz2 scummvm-rg350-06d6e118eb7d6bace91837f155429e55bd425e76.zip | |
DM: Add partial fix to drawing explosions
Diffstat (limited to 'engines')
| -rw-r--r-- | engines/dm/TODOs/todo.txt | 3 | ||||
| -rw-r--r-- | engines/dm/gfx.cpp | 28 |
2 files changed, 26 insertions, 5 deletions
diff --git a/engines/dm/TODOs/todo.txt b/engines/dm/TODOs/todo.txt index 4309c984b5..def2956c8e 100644 --- a/engines/dm/TODOs/todo.txt +++ b/engines/dm/TODOs/todo.txt @@ -2,6 +2,7 @@ Bugs: Display: Spellcasting tabs are displayed inproperly, switching between them is possible tho Cursor icons are drawn twice + DisplayMan::blitBoxFilledWithMaskedBitmap does not produce the same effect as the original Logic: Items thrown on the right side end up on the left side @@ -15,7 +16,7 @@ Todo: I forgot to add a bunch of warning for show/hide mouse pointer and other mouse functions Code stuff todo: - Complete stub methods(blitShrink, blitmask) + Complete stub methods(blitShrink) Add proper save header, add error handling to it Add translations to f433_processCommand140_saveGame 'LOAD'
\ No newline at end of file diff --git a/engines/dm/gfx.cpp b/engines/dm/gfx.cpp index 8aa3aa5245..2ad9a6076f 100644 --- a/engines/dm/gfx.cpp +++ b/engines/dm/gfx.cpp @@ -866,10 +866,30 @@ void DisplayMan::fillBoxBitmap(byte *destBitmap, Box &box, Color color, int16 by } void DisplayMan::blitBoxFilledWithMaskedBitmap(byte *src, byte *dest, byte *mask, byte *tmp, Box& box, - int16 lastUnitIndex, int16 firstUnitIndex, int16 destByteWidth, Color transparent, - int16 xPos, int16 yPos, int16 destHeight, int16 height2) { - // make sure to take care of inclusive boundaries, color can have 0x8000 flag to not use mask - warning("STUB: blitBoxFilledWithMaskedBitmap"); + int16 lastUnitIndex, int16 firstUnitIndex, int16 destByteWidth, Color transparent, + int16 xPos, int16 yPos, int16 destHeight, int16 height2) { + + // FIXME: does not produce the same effect as the original + + byte nextUnitIndex = firstUnitIndex; + bool useMask = !(transparent & k0x0080_BlitDoNotUseMask); + transparent = (Color)(transparent & ~(k0x0080_BlitDoNotUseMask)); // clear flag 0x0080 + for (byte next_y = box._y1; next_y <= box._y2; next_y++) { // '<=' for inclusive boundaries + for (byte next_x = box._x1; next_x <= box._x2; next_x++) { // '<=' for inclusive boundaries + byte *nextDestPixel = dest + next_y * destByteWidth * 2 + next_x; + byte nextSrcPixel = src[nextUnitIndex]; + + if (nextSrcPixel != transparent) { + if (useMask && mask && *mask++) { + *nextDestPixel = *mask & nextSrcPixel; + } else + *nextDestPixel = nextSrcPixel; + } + + if (++nextUnitIndex >= lastUnitIndex) + nextUnitIndex = 0; // 0 is not an error + } + } } void DisplayMan::flipBitmapHorizontal(byte *bitmap, uint16 byteWidth, uint16 height) { |