[Scummvm-cvs-logs] SF.net SVN: scummvm:[41743] scummvm/trunk/engines/kyra
lordhoto at users.sourceforge.net
lordhoto at users.sourceforge.net
Mon Jun 22 04:36:54 CEST 2009
Revision: 41743
http://scummvm.svn.sourceforge.net/scummvm/?rev=41743&view=rev
Author: lordhoto
Date: 2009-06-22 02:36:54 +0000 (Mon, 22 Jun 2009)
Log Message:
-----------
- Changed the following Screen functions to take a reference to a Palette object:
-> setScreenPalette
-> fadePalette
-> getFadeParams
-> fadePalStep
- Fixed initialization of 256 color palettes
Modified Paths:
--------------
scummvm/trunk/engines/kyra/debugger.cpp
scummvm/trunk/engines/kyra/gui_hof.cpp
scummvm/trunk/engines/kyra/gui_lok.cpp
scummvm/trunk/engines/kyra/gui_mr.cpp
scummvm/trunk/engines/kyra/kyra_hof.cpp
scummvm/trunk/engines/kyra/kyra_mr.cpp
scummvm/trunk/engines/kyra/lol.cpp
scummvm/trunk/engines/kyra/scene_hof.cpp
scummvm/trunk/engines/kyra/scene_lok.cpp
scummvm/trunk/engines/kyra/scene_lol.cpp
scummvm/trunk/engines/kyra/scene_mr.cpp
scummvm/trunk/engines/kyra/screen.cpp
scummvm/trunk/engines/kyra/screen.h
scummvm/trunk/engines/kyra/screen_lok.cpp
scummvm/trunk/engines/kyra/screen_lol.cpp
scummvm/trunk/engines/kyra/screen_v2.cpp
scummvm/trunk/engines/kyra/screen_v2.h
scummvm/trunk/engines/kyra/script_hof.cpp
scummvm/trunk/engines/kyra/script_lok.cpp
scummvm/trunk/engines/kyra/script_lol.cpp
scummvm/trunk/engines/kyra/script_tim.cpp
scummvm/trunk/engines/kyra/seqplayer.cpp
scummvm/trunk/engines/kyra/sequences_hof.cpp
scummvm/trunk/engines/kyra/sequences_lok.cpp
scummvm/trunk/engines/kyra/sequences_lol.cpp
scummvm/trunk/engines/kyra/text_lok.cpp
scummvm/trunk/engines/kyra/vqa.cpp
Modified: scummvm/trunk/engines/kyra/debugger.cpp
===================================================================
--- scummvm/trunk/engines/kyra/debugger.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/debugger.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -69,7 +69,7 @@
}
bool Debugger::cmd_loadPalette(int argc, const char **argv) {
- uint8 palette[768];
+ Palette palette(_vm->screen()->getPalette(0).getNumColors());
if (argc <= 1) {
DebugPrintf("Use load_palette <file> [start_col] [end_col]\n");
@@ -80,24 +80,24 @@
uint8 buffer[320*200];
_vm->screen()->copyRegionToBuffer(5, 0, 0, 320, 200, buffer);
_vm->screen()->loadBitmap(argv[1], 5, 5, 0);
- memcpy(palette, _vm->screen()->getCPagePtr(5), 768);
+ palette.copy(_vm->screen()->getCPagePtr(5), 0, 256);
_vm->screen()->copyBlockToPage(5, 0, 0, 320, 200, buffer);
- } else if (!_vm->screen()->loadPalette(argv[1], palette)) {
+ } else if (!_vm->screen()->loadPalette(argv[1], palette.getData())) {
DebugPrintf("ERROR: Palette '%s' not found!\n", argv[1]);
return true;
}
int startCol = 0;
- int endCol = 255;
+ int endCol = palette.getNumColors();
if (argc > 2)
- startCol = MIN(255, MAX(0, atoi(argv[2])));
+ startCol = MIN(palette.getNumColors(), MAX(0, atoi(argv[2])));
if (argc > 3)
- endCol = MIN(255, MAX(0, atoi(argv[3])));
+ endCol = MIN(palette.getNumColors(), MAX(0, atoi(argv[3])));
if (startCol > 0)
- memcpy(palette, _vm->screen()->getScreenPalette(), startCol*3);
- if (endCol < 255)
- memcpy(palette + endCol * 3, _vm->screen()->getScreenPalette() + endCol * 3, (255-endCol)*3);
+ palette.copy(_vm->screen()->getPalette(0), 0, startCol);
+ if (endCol < palette.getNumColors())
+ palette.copy(_vm->screen()->getPalette(0), endCol);
_vm->screen()->setScreenPalette(palette);
_vm->screen()->updateScreen();
Modified: scummvm/trunk/engines/kyra/gui_hof.cpp
===================================================================
--- scummvm/trunk/engines/kyra/gui_hof.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/gui_hof.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -375,7 +375,7 @@
int oldItemInHand = _itemInHand;
removeHandItem();
- _screen->fadePalette(_screen->getPalette(0).getData(), 7);
+ _screen->fadePalette(_screen->getPalette(0), 7);
_screen->showMouse();
bookLoop();
@@ -394,7 +394,7 @@
setHandItem(_itemInHand);
_screen->copyPalette(0, 2);
- _screen->fadePalette(_screen->getPalette(0).getData(), 7, &_updateFunctor);
+ _screen->fadePalette(_screen->getPalette(0), 7, &_updateFunctor);
_screen->showMouse();
if (!queryGameFlag(4) && !queryGameFlag(0xB8)) {
@@ -812,14 +812,14 @@
_screen->getPalette(0).copy(_screen->getPalette(1), guiPal[i], 1);
if (_isDeathMenu)
- _screen->fadePalette(_screen->getPalette(0).getData(), 0x64);
+ _screen->fadePalette(_screen->getPalette(0), 0x64);
else
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
}
void GUI_HoF::restorePalette() {
_screen->copyPalette(0, 1);
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
}
void GUI_HoF::resetState(int item) {
Modified: scummvm/trunk/engines/kyra/gui_lok.cpp
===================================================================
--- scummvm/trunk/engines/kyra/gui_lok.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/gui_lok.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -1034,7 +1034,7 @@
++index;
}
- _screen->fadePalette(_screen->getPalette(0).getData(), 2);
+ _screen->fadePalette(_screen->getPalette(0), 2);
}
void GUI_LoK::restorePalette() {
@@ -1042,7 +1042,7 @@
return;
_screen->copyPalette(0, 2);
- _screen->fadePalette(_screen->getPalette(0).getData(), 2);
+ _screen->fadePalette(_screen->getPalette(0), 2);
}
#pragma mark -
Modified: scummvm/trunk/engines/kyra/gui_mr.cpp
===================================================================
--- scummvm/trunk/engines/kyra/gui_mr.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/gui_mr.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -693,7 +693,7 @@
printAlbumPageText();
_screen->copyRegion(0, 0, 0, 0, 320, 200, 2, 0, Screen::CR_NO_P_CHECK);
_screen->updateScreen();
- _screen->fadePalette(_screen->getPalette(0).getData(), 9);
+ _screen->fadePalette(_screen->getPalette(0), 9);
processAlbum();
@@ -708,7 +708,7 @@
_screen->copyBlockToPage(4, 0, 0, 320, 200, _album.backUpPage);
_screen->copyPalette(0, 1);
- _screen->fadePalette(_screen->getPalette(0).getData(), 9);
+ _screen->fadePalette(_screen->getPalette(0), 9);
delete[] _album.backUpRect;
_album.backUpRect = 0;
Modified: scummvm/trunk/engines/kyra/kyra_hof.cpp
===================================================================
--- scummvm/trunk/engines/kyra/kyra_hof.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/kyra_hof.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -921,7 +921,7 @@
palIndex *= 3;
memcpy(_messagePal, _screen->getPalette(0).getData() + palIndex, 3);
_screen->getPalette(0).copy(_screen->getPalette(0), palIndex / 3, 1, 255);
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
}
int x = _text->getCenterStringX(string, 0, 320);
@@ -979,7 +979,7 @@
if (updatePalette) {
_screen->getPalette(0).copy(_messagePal, 0, 1, 255);
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
} else {
_fadeMessagePalette = false;
}
@@ -1145,12 +1145,12 @@
++ptr;
++src;
}
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
unkVar1 = true;
_charPalEntry = palEntry;
} else if (unkVar1 || !unk1) {
_screen->getPalette(0).copy(_scenePal, palEntry << 4, 16, 112);
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
unkVar1 = false;
}
}
@@ -1667,9 +1667,9 @@
if (paletteFade) {
snd_playSoundEffect((state == 0) ? 0x6B : 0x66);
- _screen->fadePalette(_screen->getPalette(2).getData(), 0x4B, &_updateFunctor);
+ _screen->fadePalette(_screen->getPalette(2), 0x4B, &_updateFunctor);
} else {
- _screen->setScreenPalette(_screen->getPalette(2).getData());
+ _screen->setScreenPalette(_screen->getPalette(2));
_screen->updateScreen();
}
@@ -1835,11 +1835,11 @@
file->seek(index*18, SEEK_SET);
file->read(_screen->getPalette(0).getData()+723, 18);
snd_playSoundEffect(0x6A);
- _screen->fadePalette(_screen->getPalette(0).getData(), 0x1E, &_updateFunctor);
+ _screen->fadePalette(_screen->getPalette(0), 0x1E, &_updateFunctor);
file->seek(0, SEEK_SET);
file->read(_screen->getPalette(0).getData()+723, 18);
delete file;
- _screen->fadePalette(_screen->getPalette(0).getData(), 0x1E, &_updateFunctor);
+ _screen->fadePalette(_screen->getPalette(0), 0x1E, &_updateFunctor);
}
void KyraEngine_HoF::resetCauldronStateTable(int idx) {
Modified: scummvm/trunk/engines/kyra/kyra_mr.cpp
===================================================================
--- scummvm/trunk/engines/kyra/kyra_mr.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/kyra_mr.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -231,7 +231,7 @@
_screen->setScreenDim(0);
_res->loadFileToBuf("PALETTE.COL", _screen->getPalette(0).getData(), 768);
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
return Common::kNoError;
}
@@ -267,7 +267,7 @@
_screen->_curPage = 0;
_screen->clearPage(0);
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
// XXX
playMenuAudioFile();
@@ -390,10 +390,9 @@
_soundDigital->stopAllSounds();
_screen->showMouse();
- uint8 pal[768];
// Taken from original, it used '1' here too
- memset(pal, 1, sizeof(pal));
- _screen->setScreenPalette(pal);
+ memset(_screen->getPalette(0).getData(), 1, 768);
+ _screen->setScreenPalette(_screen->getPalette(0));
_screen->clearPage(0);
_screen->copyPalette(0, 1);
_wasPlayingVQA = true;
@@ -919,7 +918,7 @@
}
}
_charPalUpdate = true;
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
_lastCharPalLayer = layer;
} else if (_charPalUpdate || !unk1) {
memcpy(dst, src, 72);
@@ -936,7 +935,7 @@
}
}
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
_charPalUpdate = false;
}
}
Modified: scummvm/trunk/engines/kyra/lol.cpp
===================================================================
--- scummvm/trunk/engines/kyra/lol.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/lol.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -1677,7 +1677,7 @@
void LoLEngine::setPaletteBrightness(uint8 *palette, int brightness, int modifier) {
generateBrightnessPalette(palette, _screen->getPalette(1).getData(), brightness, modifier);
- _screen->fadePalette(_screen->getPalette(1).getData(), 5, 0);
+ _screen->fadePalette(_screen->getPalette(1), 5, 0);
_screen->_fadeFlag = 0;
}
@@ -2617,7 +2617,7 @@
mov->close();
delete mov;
- _screen->setScreenPalette(_screen->getPalette(1).getData());
+ _screen->setScreenPalette(_screen->getPalette(1));
_screen->copyPage(12, 2);
_screen->copyPage(12, 0);
updateDrawPage2();
@@ -2836,20 +2836,22 @@
}
void LoLEngine::callbackProcessMagicLightning(WSAMovie_v2 *mov, int x, int y) {
- uint8 *tpal = new uint8[768];
if (_lightningDiv == 2)
shakeScene(1, 2, 3, 0);
- uint8 *p1 = _screen->getPalette(1).getData();
+ const Palette &p1 = _screen->getPalette(1);
if (_lightningSfxFrame % _lightningDiv) {
_screen->setScreenPalette(p1);
} else {
- memcpy(tpal, p1, 768);
+ Palette tpal(p1.getNumColors());
+ tpal.copy(p1);
+
for (int i = 6; i < 384; i++) {
uint16 v = (tpal[i] * 120) / 64;
tpal[i] = (v < 64) ? v : 63;
}
+
_screen->setScreenPalette(tpal);
}
@@ -2864,7 +2866,6 @@
}
_lightningSfxFrame++;
- delete[] tpal;
}
void LoLEngine::drinkBezelCup(int numUses, int charNum) {
@@ -3871,7 +3872,7 @@
_screen->copyPage(2, 0);
_screen->updateScreen();
- _screen->fadePalette(_screen->getPalette(3).getData(), 10);
+ _screen->fadePalette(_screen->getPalette(3), 10);
uint32 delayTimer = _system->getMillis() + 8 * _tickLength;
while (!exitAutomap && !shouldQuit()) {
Modified: scummvm/trunk/engines/kyra/scene_hof.cpp
===================================================================
--- scummvm/trunk/engines/kyra/scene_hof.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/scene_hof.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -670,13 +670,13 @@
if (_noScriptEnter) {
memset(_screen->getPalette(0).getData(), 0, 384);
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
}
_screen->copyRegion(0, 0, 0, 0, 320, 144, 2, 0, Screen::CR_NO_P_CHECK);
if (_noScriptEnter) {
- _screen->setScreenPalette(_screen->getPalette(1).getData());
+ _screen->setScreenPalette(_screen->getPalette(1));
_screen->getPalette(0).copy(_screen->getPalette(1), 0, 128);
}
@@ -696,7 +696,7 @@
void KyraEngine_HoF::fadeScenePal(int srcIndex, int delayTime) {
_screen->getPalette(0).copy(_scenePal, srcIndex << 4, 16, 112);
- _screen->fadePalette(_screen->getPalette(0).getData(), delayTime, &_updateFunctor);
+ _screen->fadePalette(_screen->getPalette(0), delayTime, &_updateFunctor);
}
#pragma mark -
Modified: scummvm/trunk/engines/kyra/scene_lok.cpp
===================================================================
--- scummvm/trunk/engines/kyra/scene_lok.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/scene_lok.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -772,7 +772,7 @@
if (_unkScreenVar1 && !queryGameFlag(0xF0)) {
_screen->getPalette(2).clear();
if (_currentCharacter->sceneId != 117 || !queryGameFlag(0xB3))
- _screen->setScreenPalette(_screen->getPalette(2).getData());
+ _screen->setScreenPalette(_screen->getPalette(2));
}
if (_unkScreenVar2 == 1)
@@ -787,7 +787,7 @@
if (_currentCharacter->sceneId >= 229 && _currentCharacter->sceneId <= 245 && (_brandonStatusBit & 1))
_screen->copyPalette(0, 10);
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
}
} else {
if (_unkScreenVar1 && !queryGameFlag(0xA0)) {
@@ -797,7 +797,7 @@
col >>= 2;
_screen->getPalette(0)[684+i] = col;
}
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
}
if (_unkScreenVar2 == 1)
@@ -808,7 +808,7 @@
if (_unkScreenVar1 && _paletteChanged) {
if (!queryGameFlag(0xA0)) {
_screen->getPalette(0).copy(_screen->getPalette(1), 228, 20);
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
} else {
_screen->getPalette(0).clear();
}
Modified: scummvm/trunk/engines/kyra/scene_lol.cpp
===================================================================
--- scummvm/trunk/engines/kyra/scene_lol.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/scene_lol.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -1417,7 +1417,7 @@
initDialogueSequence(fieldType, 0);
if (fadeFlag) {
- _screen->fadePalette(_screen->getPalette(3).getData(), 10);
+ _screen->fadePalette(_screen->getPalette(3), 10);
_screen->_fadeFlag = 0;
}
@@ -1435,7 +1435,7 @@
if (fadeFlag) {
_screen->getPalette(3).copy(_screen->getPalette(0), 128);
_screen->loadSpecialColors(_screen->getPalette(3).getData());
- _screen->fadePalette(_screen->getPalette(3).getData(), 10);
+ _screen->fadePalette(_screen->getPalette(3), 10);
_screen->_fadeFlag = 0;
}
Modified: scummvm/trunk/engines/kyra/scene_mr.cpp
===================================================================
--- scummvm/trunk/engines/kyra/scene_mr.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/scene_mr.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -609,14 +609,14 @@
if (_noScriptEnter) {
memset(_screen->getPalette(0).getData(), 0, 432);
if (!_wasPlayingVQA)
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
}
_screen->copyRegion(0, 0, 0, 0, 320, 200, 2, 0, Screen::CR_NO_P_CHECK);
if (_noScriptEnter) {
if (!_wasPlayingVQA)
- _screen->setScreenPalette(_screen->getPalette(2).getData());
+ _screen->setScreenPalette(_screen->getPalette(2));
_screen->getPalette(0).copy(_screen->getPalette(2), 0, 144);
if (_wasPlayingVQA) {
_screen->fadeFromBlack(0x3C);
Modified: scummvm/trunk/engines/kyra/screen.cpp
===================================================================
--- scummvm/trunk/engines/kyra/screen.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/screen.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -124,8 +124,10 @@
memset(_palettes, 0, sizeof(_palettes));
- _screenPalette = new Palette(768);
+ _screenPalette = new Palette(256);
assert(_screenPalette);
+ _tempPalette = new Palette(_screenPalette->getNumColors());
+ assert(_tempPalette);
if (_vm->gameFlags().platform == Common::kPlatformAmiga) {
for (int i = 0; i < 7; ++i) {
@@ -134,12 +136,12 @@
}
} else {
for (int i = 0; i < 4; ++i) {
- _palettes[i] = new Palette(768);
+ _palettes[i] = new Palette(256);
assert(_palettes[i]);
}
}
- setScreenPalette(getPalette(0).getData());
+ setScreenPalette(getPalette(0));
_curDim = 0;
_charWidth = 0;
@@ -490,26 +492,25 @@
}
void Screen::fadeFromBlack(int delay, const UpdateFunctor *upFunc) {
- fadePalette(getPalette(0).getData(), delay, upFunc);
+ fadePalette(getPalette(0), delay, upFunc);
}
void Screen::fadeToBlack(int delay, const UpdateFunctor *upFunc) {
- uint8 blackPal[768];
- memset(blackPal, 0, 768);
- fadePalette(blackPal, delay, upFunc);
+ _tempPalette->clear();
+ fadePalette(*_tempPalette, delay, upFunc);
}
-void Screen::fadePalette(const uint8 *palData, int delay, const UpdateFunctor *upFunc) {
+void Screen::fadePalette(const Palette &pal, int delay, const UpdateFunctor *upFunc) {
updateScreen();
int diff = 0, delayInc = 0;
- getFadeParams(palData, delay, delayInc, diff);
+ getFadeParams(pal, delay, delayInc, diff);
int delayAcc = 0;
while (!_vm->shouldQuit()) {
delayAcc += delayInc;
- int refreshed = fadePalStep(palData, diff);
+ int refreshed = fadePalStep(pal, diff);
if (upFunc && upFunc->isValid())
(*upFunc)();
@@ -524,7 +525,7 @@
}
if (_vm->shouldQuit()) {
- setScreenPalette(palData);
+ setScreenPalette(pal);
if (upFunc && upFunc->isValid())
(*upFunc)();
else
@@ -532,12 +533,11 @@
}
}
-void Screen::getFadeParams(const uint8 *palette, int delay, int &delayInc, int &diff) {
+void Screen::getFadeParams(const Palette &pal, int delay, int &delayInc, int &diff) {
uint8 maxDiff = 0;
- const int colors = (_vm->gameFlags().platform == Common::kPlatformAmiga ? 32 : 256) * 3;
- for (int i = 0; i < colors; ++i) {
- diff = ABS(palette[i] - (*_screenPalette)[i]);
+ for (int i = 0; i < pal.getNumColors() * 3; ++i) {
+ diff = ABS(pal[i] - (*_screenPalette)[i]);
maxDiff = MAX<uint8>(maxDiff, diff);
}
@@ -553,17 +553,14 @@
}
}
-int Screen::fadePalStep(const uint8 *palette, int diff) {
- const int colors = (_vm->gameFlags().platform == Common::kPlatformAmiga ? 32 : (_use16ColorMode ? 16 : 256)) * 3;
+int Screen::fadePalStep(const Palette &pal, int diff) {
+ _tempPalette->copy(*_screenPalette);
- uint8 fadePal[768];
- memcpy(fadePal, _screenPalette->getData(), colors);
-
bool needRefresh = false;
- for (int i = 0; i < colors; ++i) {
- int c1 = palette[i];
- int c2 = fadePal[i];
+ for (int i = 0; i < pal.getNumColors() * 3; ++i) {
+ int c1 = pal[i];
+ int c2 = (*_tempPalette)[i];
if (c1 != c2) {
needRefresh = true;
if (c1 > c2) {
@@ -578,12 +575,12 @@
c2 = c1;
}
- fadePal[i] = (uint8)c2;
+ (*_tempPalette)[i] = (uint8)c2;
}
}
if (needRefresh)
- setScreenPalette(fadePal);
+ setScreenPalette(*_tempPalette);
return needRefresh ? 1 : 0;
}
@@ -592,7 +589,7 @@
getPalette(0)[index * 3 + 0] = red;
getPalette(0)[index * 3 + 1] = green;
getPalette(0)[index * 3 + 2] = blue;
- setScreenPalette(getPalette(0).getData());
+ setScreenPalette(getPalette(0));
}
void Screen::getRealPalette(int num, uint8 *dst) {
@@ -613,35 +610,18 @@
}
}
-void Screen::setScreenPalette(const uint8 *palData) {
- const int colors = (_vm->gameFlags().platform == Common::kPlatformAmiga ? 32 : 256);
-
+void Screen::setScreenPalette(const Palette &pal) {
uint8 screenPal[256 * 4];
- _screenPalette->copy(palData, 0, colors);
+ _screenPalette->copy(pal);
- if (_use16ColorMode && _vm->gameFlags().platform == Common::kPlatformPC98) {
- for (int l = 0; l < 1024; l += 64) {
- const uint8 *tp = palData;
- for (int i = 0; i < 16; ++i) {
- screenPal[l + 4 * i + 0] = palData[1];
- screenPal[l + 4 * i + 1] = palData[0];
- screenPal[l + 4 * i + 2] = palData[2];
- screenPal[l + 4 * i + 3] = 0;
- palData += 3;
- }
- palData = tp;
- }
- } else {
- for (int i = 0; i < colors; ++i) {
- screenPal[4 * i + 0] = (palData[0] << 2) | (palData[0] & 3);
- screenPal[4 * i + 1] = (palData[1] << 2) | (palData[1] & 3);
- screenPal[4 * i + 2] = (palData[2] << 2) | (palData[2] & 3);
- screenPal[4 * i + 3] = 0;
- palData += 3;
- }
+ for (int i = 0; i < pal.getNumColors(); ++i) {
+ screenPal[4 * i + 0] = (pal[i * 3 + 0] << 2) | (pal[i * 3 + 0] & 3);
+ screenPal[4 * i + 1] = (pal[i * 3 + 1] << 2) | (pal[i * 3 + 1] & 3);
+ screenPal[4 * i + 2] = (pal[i * 3 + 2] << 2) | (pal[i * 3 + 2] & 3);
+ screenPal[4 * i + 3] = 0;
}
- _system->setPalette(screenPal, 0, colors);
+ _system->setPalette(screenPal, 0, pal.getNumColors());
}
void Screen::copyToPage0(int y, int h, uint8 page, uint8 *seqBuf) {
Modified: scummvm/trunk/engines/kyra/screen.h
===================================================================
--- scummvm/trunk/engines/kyra/screen.h 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/screen.h 2009-06-22 02:36:54 UTC (rev 41743)
@@ -223,13 +223,12 @@
void fadeFromBlack(int delay=0x54, const UpdateFunctor *upFunc = 0);
void fadeToBlack(int delay=0x54, const UpdateFunctor *upFunc = 0);
- void fadePalette(const uint8 *palData, int delay, const UpdateFunctor *upFunc = 0);
- virtual void getFadeParams(const uint8 *palette, int delay, int &delayInc, int &diff);
- int fadePalStep(const uint8 *palette, int diff);
+ void fadePalette(const Palette &pal, int delay, const UpdateFunctor *upFunc = 0);
+ virtual void getFadeParams(const Palette &pal, int delay, int &delayInc, int &diff);
+ int fadePalStep(const Palette &pal, int diff);
void setPaletteIndex(uint8 index, uint8 red, uint8 green, uint8 blue);
- void setScreenPalette(const uint8 *palData);
- const uint8 *getScreenPalette() const { return _screenPalette->getData(); }
+ void setScreenPalette(const Palette &pal);
void getRealPalette(int num, uint8 *dst);
Palette &getPalette(int num);
@@ -362,6 +361,7 @@
Palette *_screenPalette;
Palette *_palettes[7];
+ Palette *_tempPalette;
Font _fonts[FID_NUM];
uint8 _textColorsMap[16];
Modified: scummvm/trunk/engines/kyra/screen_lok.cpp
===================================================================
--- scummvm/trunk/engines/kyra/screen_lok.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/screen_lok.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -81,14 +81,14 @@
void Screen_LoK::fadeSpecialPalette(int palIndex, int startIndex, int size, int fadeTime) {
assert(_vm->palTable1()[palIndex]);
- uint8 tempPal[768];
- memcpy(tempPal, getPalette(0).getData(), 768);
- memcpy(&tempPal[startIndex*3], _vm->palTable1()[palIndex], size*3);
+ Palette tempPal(getPalette(0).getNumColors());
+ tempPal.copy(getPalette(0));
+ tempPal.copy(_vm->palTable1()[palIndex], 0, size, startIndex);
fadePalette(tempPal, fadeTime*18);
getPalette(0).copy(tempPal, startIndex, size);
- setScreenPalette(getPalette(0).getData());
+ setScreenPalette(getPalette(0));
_system->updateScreen();
}
Modified: scummvm/trunk/engines/kyra/screen_lol.cpp
===================================================================
--- scummvm/trunk/engines/kyra/screen_lol.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/screen_lol.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -282,14 +282,13 @@
if (_fadeFlag == 1)
return;
- uint8 *tpal = new uint8[768];
+ Palette tpal(getPalette(0).getNumColors());
+ tpal.copy(getPalette(0));
- memcpy(tpal, getPalette(0).getData(), 768);
- memset(tpal, 0, 384);
- loadSpecialColors(tpal);
+ loadSpecialColors(tpal.getData());
fadePalette(tpal, delay);
+
fillRect(112, 0, 288, 120, 0);
- delete[] tpal;
_fadeFlag = 1;
}
@@ -839,7 +838,7 @@
void Screen_LoL::fadeToPalette1(int delay) {
loadSpecialColors(getPalette(1).getData());
- fadePalette(getPalette(1).getData(), delay);
+ fadePalette(getPalette(1), delay);
_fadeFlag = 0;
}
@@ -897,22 +896,22 @@
p++;
}
- uint8 tpal[768];
- memcpy(tpal, _screenPalette->getData(), 768);
- memcpy(tpal + dstColorIndex * 3, tmpPalEntry, 3);
- setScreenPalette(tpal);
+ _tempPalette->copy(*_screenPalette);
+ _tempPalette->copy(*_screenPalette);
+ _tempPalette->copy(tmpPalEntry, 0, 1, dstColorIndex);
+ setScreenPalette(*_tempPalette);
updateScreen();
return res;
}
bool Screen_LoL::fadePaletteStep(uint8 *pal1, uint8 *pal2, uint32 elapsedTime, uint32 targetTime) {
- uint8 tpal[768];
- uint8 *p1 = getPalette(1).getData();
+ Palette &p1 = getPalette(1);
bool res = false;
for (int i = 0; i < 768; i++) {
uint8 out = 0;
+
if (elapsedTime < targetTime) {
int32 d = ((pal2[i] & 0x3f) - (pal1[i] & 0x3f));
if (d)
@@ -925,10 +924,10 @@
res = false;
}
- tpal[i] = out;
+ (*_tempPalette)[i] = out;
}
- setScreenPalette(tpal);
+ setScreenPalette(*_tempPalette);
updateScreen();
return res;
Modified: scummvm/trunk/engines/kyra/screen_v2.cpp
===================================================================
--- scummvm/trunk/engines/kyra/screen_v2.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/screen_v2.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -114,12 +114,11 @@
return r;
}
-void Screen_v2::getFadeParams(const uint8 *palette, int delay, int &delayInc, int &diff) {
+void Screen_v2::getFadeParams(const Palette &pal, int delay, int &delayInc, int &diff) {
int maxDiff = 0;
diff = 0;
- int len = _use16ColorMode ? 48 : 768;
- for (int i = 0; i < len; ++i) {
- diff = ABS(palette[i] - (*_screenPalette)[i]);
+ for (int i = 0; i < pal.getNumColors() * 3; ++i) {
+ diff = ABS(pal[i] - (*_screenPalette)[i]);
maxDiff = MAX(maxDiff, diff);
}
Modified: scummvm/trunk/engines/kyra/screen_v2.h
===================================================================
--- scummvm/trunk/engines/kyra/screen_v2.h 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/screen_v2.h 2009-06-22 02:36:54 UTC (rev 41743)
@@ -44,7 +44,7 @@
void applyOverlay(int x, int y, int w, int h, int pageNum, const uint8 *overlay);
int findLeastDifferentColor(const uint8 *paletteEntry, const uint8 *palette, uint16 numColors, bool skipSpecialColors = false);
- virtual void getFadeParams(const uint8 *palette, int delay, int &delayInc, int &diff);
+ virtual void getFadeParams(const Palette &pal, int delay, int &delayInc, int &diff);
// shape handling
uint8 *getPtrToShape(uint8 *shpFile, int shape);
Modified: scummvm/trunk/engines/kyra/script_hof.cpp
===================================================================
--- scummvm/trunk/engines/kyra/script_hof.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/script_hof.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -149,7 +149,7 @@
_screen->copyPalette(2, 0);
_screen->loadPalette(palfile, _screen->getPalette(2).getData());
_screen->fillRect(0, 0, 319, 199, 207);
- _screen->setScreenPalette(_screen->getPalette(2).getData());
+ _screen->setScreenPalette(_screen->getPalette(2));
_screen->copyRegion(0, 0, 0, 0, 320, 200, 2, 0);
if (!scumm_stricmp(cpsfile, "_MEANWIL.CPS") && _flags.lang == Common::JA_JPN) {
Screen::FontId o = _screen->setFont(Screen::FID_6_FNT);
@@ -567,7 +567,7 @@
int KyraEngine_HoF::o2_setPalette384(EMCState *script) {
debugC(3, kDebugLevelScriptFuncs, "KyraEngine_HoF::o2_setPalette384(%p) ()", (const void *)script);
_screen->getPalette(0).copy(_screen->getPalette(1), 0, 128);
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
return 0;
}
@@ -798,7 +798,7 @@
}
_screen->copyRegion(0, 0, 0, 0, 320, 200, 2, 0, Screen::CR_NO_P_CHECK);
- _screen->fadePalette(_screen->getPalette(0).getData(), 0x14);
+ _screen->fadePalette(_screen->getPalette(0), 0x14);
_screen->setMouseCursor(0, 0, getShapePtr(0));
setMousePos(280, 160);
@@ -820,7 +820,7 @@
_screen->fadeToBlack(0x14);
restorePage0();
_screen->copyPalette(0, 2);
- _screen->fadePalette(_screen->getPalette(0).getData(), 0x14);
+ _screen->fadePalette(_screen->getPalette(0), 0x14);
setHandItem(_itemInHand);
_screen->showMouse();
@@ -1130,7 +1130,7 @@
for (int i = 1; i < 768; i += 3)
_screen->getPalette(0)[i] = 0;
snd_playSoundEffect(106);
- _screen->fadePalette(_screen->getPalette(0).getData(), 90, &_updateFunctor);
+ _screen->fadePalette(_screen->getPalette(0), 90, &_updateFunctor);
_screen->copyPalette(0, 2);
for (int i = 0; i < 768; i += 3) {
@@ -1140,10 +1140,10 @@
_screen->getPalette(0)[i + 2] = 63;
}
snd_playSoundEffect(106);
- _screen->fadePalette(_screen->getPalette(0).getData(), 90, &_updateFunctor);
+ _screen->fadePalette(_screen->getPalette(0), 90, &_updateFunctor);
_screen->copyPalette(0, 2);
- _screen->fadePalette(_screen->getPalette(0).getData(), 30, &_updateFunctor);
+ _screen->fadePalette(_screen->getPalette(0), 30, &_updateFunctor);
return 0;
}
@@ -1262,7 +1262,7 @@
int KyraEngine_HoF::o2_processPaletteIndex(EMCState *script) {
debugC(3, kDebugLevelScriptFuncs, "KyraEngine_HoF::o2_processPaletteIndex(%p) (%d, %d, %d, %d, %d, %d)", (const void *)script, stackPos(0), stackPos(1), stackPos(2), stackPos(3), stackPos(4), stackPos(5));
- uint8 *palette = _screen->getPalette(0).getData();
+ Palette &palette = _screen->getPalette(0);
const int index = stackPos(0);
const bool updatePalette = (stackPos(4) != 0);
@@ -1410,7 +1410,7 @@
for (int i = 0; i < 6; i++)
_text->printText(strings[i], _text->getCenterStringX(strings[i], 1, 319), y + i * 10, 255, 207, 0);
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
_screen->updateScreen();
_eventList.clear();
Modified: scummvm/trunk/engines/kyra/script_lok.cpp
===================================================================
--- scummvm/trunk/engines/kyra/script_lok.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/script_lok.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -233,7 +233,7 @@
if (stackPos(0) == 13) {
// TODO: Check this!
_screen->copyPalette(0, 12);
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
}
} else {
warning("KyraEngine_LoK::o1_fadeSpecialPalette not implemented");
@@ -1503,37 +1503,38 @@
int KyraEngine_LoK::o1_fadeEntirePalette(EMCState *script) {
debugC(3, kDebugLevelScriptFuncs, "KyraEngine_LoK::o1_fadeEntirePalette(%p) (%d, %d)", (const void *)script, stackPos(0), stackPos(1));
int cmd = stackPos(0);
- uint8 *fadePal = 0;
+ int fadePal = 0;
+
if (_flags.platform == Common::kPlatformAmiga) {
if (cmd == 0) {
_screen->getPalette(2).clear();
- fadePal = _screen->getPalette(2).getData();
+ fadePal = 2;
_screen->copyPalette(4, 0);
} else if (cmd == 1) {
- fadePal = _screen->getPalette(0).getData();
+ fadePal = 0;
_screen->copyPalette(0, 4);
} else if (cmd == 2) {
- fadePal = _screen->getPalette(0).getData();
+ fadePal = 0;
_screen->getPalette(2).clear();
}
} else {
if (cmd == 0) {
+ fadePal = 2;
_screen->getPalette(2).clear();
- fadePal = _screen->getPalette(2).getData();
_screen->copyPalette(3, 0);
} else if (cmd == 1) {
- //fadePal = _screen->getPalette(3);
+ //fadePal = 3;
warning("unimplemented o1_fadeEntirePalette function");
return 0;
} else if (cmd == 2) {
_screen->getPalette(2).clear();
_screen->copyPalette(0, 1);
- fadePal = _screen->getPalette(0).getData();
+ fadePal = 0;
}
}
- _screen->fadePalette(fadePal, stackPos(1));
+ _screen->fadePalette(_screen->getPalette(fadePal), stackPos(1));
return 0;
}
Modified: scummvm/trunk/engines/kyra/script_lol.cpp
===================================================================
--- scummvm/trunk/engines/kyra/script_lol.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/script_lol.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -602,7 +602,7 @@
int LoLEngine::olol_fadePalette(EMCState *script) {
debugC(3, kDebugLevelScriptFuncs, "LoLEngine::olol_fadePalette(%p)", (const void *)script);
- _screen->fadePalette(_screen->getPalette(3).getData(), 10);
+ _screen->fadePalette(_screen->getPalette(3), 10);
_screen->_fadeFlag = 0;
return 1;
}
@@ -866,7 +866,7 @@
debugC(3, kDebugLevelScriptFuncs, "LoLEngine::olol_fadeSequencePalette(%p)", (const void *)script);
_screen->getPalette(3).copy(_screen->getPalette(0), 128);
_screen->loadSpecialColors(_screen->getPalette(3).getData());
- _screen->fadePalette(_screen->getPalette(3).getData(), 10);
+ _screen->fadePalette(_screen->getPalette(3), 10);
_screen->_fadeFlag = 0;
return 1;
}
@@ -2003,7 +2003,7 @@
int LoLEngine::olol_restoreFadePalette(EMCState *script) {
debugC(3, kDebugLevelScriptFuncs, "LoLEngine::olol_restoreFadePalette(%p)", (const void *)script);
_screen->getPalette(0).copy(_screen->getPalette(1), 0, 128);
- _screen->fadePalette(_screen->getPalette(0).getData(), 10);
+ _screen->fadePalette(_screen->getPalette(0), 10);
_screen->_fadeFlag = 0;
return 1;
}
@@ -2101,27 +2101,29 @@
int LoLEngine::olol_paletteFlash(EMCState *script) {
debugC(3, kDebugLevelScriptFuncs, "LoLEngine::olol_paletteFlash(%p) (%d)", (const void *)script, stackPos(0));
- uint8 *s = _screen->getPalette(1).getData();
- uint8 *d = _screen->getPalette(3).getData();
+ Palette &p1 = _screen->getPalette(1);
+ Palette &p2 = _screen->getPalette(3);
+
uint8 ovl[256];
- generateFlashPalette(s, d, stackPos(0));
- _screen->loadSpecialColors(s);
- _screen->loadSpecialColors(d);
+ generateFlashPalette(p1.getData(), p2.getData(), stackPos(0));
+ _screen->loadSpecialColors(p1.getData());
+ _screen->loadSpecialColors(p2.getData());
if (_smoothScrollModeNormal) {
for (int i = 0; i < 256; i++)
ovl[i] = i;
ovl[1] = 6;
+
_screen->copyRegion(112, 0, 112, 0, 176, 120, 0, 2);
_screen->applyOverlay(112, 0, 176, 120, 0, ovl);
}
- _screen->setScreenPalette(d);
+ _screen->setScreenPalette(p2);
_screen->updateScreen();
delay(2 * _tickLength);
- _screen->setScreenPalette(s);
+ _screen->setScreenPalette(p1);
if (_smoothScrollModeNormal)
_screen->copyRegion(112, 0, 112, 0, 176, 120, 2, 0);
@@ -2139,6 +2141,8 @@
_screen->hideMouse();
+ // TODO: This function could need some major cleanup to work with our
+ // new palette code without needless conversions.
uint8 *fadeTab = new uint8[21504];
uint8 *tpal1 = fadeTab;
uint8 *tpal2 = tpal1 + 768;
@@ -2152,19 +2156,25 @@
_screen->loadPalette("LITEPAL3.COL", tpal1);
_screen->generateFadeTable(tpal2, tpal4, tpal1, 4);
+ Palette pal(768);
+
for (int i = 0; i < 21; i++) {
uint32 etime = _system->getMillis() + 20 * _tickLength;
mov->displayFrame(i, 0, 0, 0, 0, 0, 0);
_screen->updateScreen();
- _screen->setScreenPalette(tpal3);
+
+ pal.copy(tpal3, 0, 256);
+ _screen->setScreenPalette(pal);
tpal3 += 768;
+
if (i == 2 || i == 5 || i == 8 || i == 11 || i == 13 || i == 15 || i == 17 || i == 19)
snd_playSoundEffect(95, -1);
delayUntil(etime);
}
+ pal.copy(tpal3, 0, 256);
snd_playSoundEffect(91, -1);
- _screen->fadePalette(tpal3, 300);
+ _screen->fadePalette(pal, 300);
tpal3 += 768;
for (int i = 22; i < 38; i++) {
@@ -2173,7 +2183,9 @@
_screen->updateScreen();
if (i == 22 || i == 24 || i == 28 || i == 32) {
snd_playSoundEffect(131, -1);
- _screen->setScreenPalette(tpal3);
+
+ pal.copy(tpal3, 0, 256);
+ _screen->setScreenPalette(pal);
tpal3 += 768;
}
delayUntil(etime);
@@ -2289,7 +2301,7 @@
int LoLEngine::tlol_setupPaletteFade(const TIM *tim, const uint16 *param) {
debugC(3, kDebugLevelScriptFuncs, "LoLEngine::t2_playSoundEffect(%p, %p) (%d)", (const void *)tim, (const void *)param, param[0]);
- _screen->getFadeParams(_screen->getPalette(0).getData(), param[0], _tim->_palDelayInc, _tim->_palDiff);
+ _screen->getFadeParams(_screen->getPalette(0), param[0], _tim->_palDelayInc, _tim->_palDiff);
_tim->_palDelayAcc = 0;
return 1;
}
@@ -2305,7 +2317,7 @@
debugC(3, kDebugLevelScriptFuncs, "LoLEngine::tlol_setupPaletteFadeEx(%p, %p) (%d)", (const void *)tim, (const void *)param, param[0]);
_screen->copyPalette(0, 1);
- _screen->getFadeParams(_screen->getPalette(0).getData(), param[0], _tim->_palDelayInc, _tim->_palDiff);
+ _screen->getFadeParams(_screen->getPalette(0), param[0], _tim->_palDelayInc, _tim->_palDiff);
_tim->_palDelayAcc = 0;
return 1;
}
@@ -2390,7 +2402,7 @@
case 1:
_screen->getPalette(3).copy(_screen->getPalette(0), 128);
_screen->loadSpecialColors(_screen->getPalette(3).getData());
- _screen->fadePalette(_screen->getPalette(3).getData(), 10);
+ _screen->fadePalette(_screen->getPalette(3), 10);
_screen->_fadeFlag = 0;
break;
@@ -2400,7 +2412,7 @@
case 3:
_screen->loadSpecialColors(_screen->getPalette(3).getData());
- _screen->fadePalette(_screen->getPalette(3).getData(), 10);
+ _screen->fadePalette(_screen->getPalette(3), 10);
_screen->_fadeFlag = 0;
break;
@@ -2414,7 +2426,7 @@
case 5:
_screen->loadSpecialColors(_screen->getPalette(3).getData());
- _screen->fadePalette(_screen->getPalette(1).getData(), 10);
+ _screen->fadePalette(_screen->getPalette(1), 10);
_screen->_fadeFlag = 0;
break;
@@ -2549,9 +2561,11 @@
int LoLEngine::tlol_fadeInPalette(const TIM *tim, const uint16 *param) {
debugC(3, kDebugLevelScriptFuncs, "LoLEngine::tlol_fadeInPalette(%p, %p) (%d, %d)", (const void *)tim, (const void *)param, param[0], param[1]);
const char *bitmap = (const char *)(tim->text + READ_LE_UINT16(tim->text + (param[0]<<1)));
- uint8 palette[768];
- _screen->loadBitmap(bitmap, 3, 3, palette);
- _screen->fadePalette(palette, param[1]);
+
+ Palette pal(_screen->getPalette(0).getNumColors());
+ _screen->loadBitmap(bitmap, 3, 3, pal.getData());
+ _screen->fadePalette(pal, param[1]);
+
return 1;
}
Modified: scummvm/trunk/engines/kyra/script_tim.cpp
===================================================================
--- scummvm/trunk/engines/kyra/script_tim.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/script_tim.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -439,9 +439,9 @@
}
if (!fadePalette && !_palDiff) {
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
} else {
- _screen->getFadeParams(_screen->getPalette(0).getData(), fadePalette, _palDelayInc, _palDiff);
+ _screen->getFadeParams(_screen->getPalette(0), fadePalette, _palDelayInc, _palDiff);
_palDelayAcc = 0;
}
}
@@ -504,7 +504,7 @@
}
if (wsaFlags & 2) {
- _screen->fadePalette(_screen->getPalette(1).getData(), 15, 0);
+ _screen->fadePalette(_screen->getPalette(1), 15, 0);
_screen->clearPage(_drawPage2);
if (_drawPage2)
_screen->checkedPageUpdate(8, 4);
@@ -526,10 +526,10 @@
}
if (wsaFlags & 2)
- _screen->fadePalette(_screen->getPalette(0).getData(), 30, 0);
+ _screen->fadePalette(_screen->getPalette(0), 30, 0);
} else {
if (wsaFlags & 2) {
- _screen->fadePalette(_screen->getPalette(1).getData(), 15, 0);
+ _screen->fadePalette(_screen->getPalette(1), 15, 0);
_screen->clearPage(_drawPage2);
if (_drawPage2)
_screen->checkedPageUpdate(8, 4);
@@ -547,7 +547,7 @@
}
if (wsaFlags & 2)
- _screen->fadePalette(_screen->getPalette(0).getData(), 30, 0);
+ _screen->fadePalette(_screen->getPalette(0), 30, 0);
}
return anim;
@@ -963,7 +963,7 @@
if (wsaFlags & 3) {
_screen->loadSpecialColors(_screen->getPalette(3).getData());
- _screen->fadePalette(_screen->getPalette(3).getData(), 10);
+ _screen->fadePalette(_screen->getPalette(3), 10);
_screen->_fadeFlag = 0;
}
Modified: scummvm/trunk/engines/kyra/seqplayer.cpp
===================================================================
--- scummvm/trunk/engines/kyra/seqplayer.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/seqplayer.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -247,7 +247,7 @@
else if (colNum == 4)
_screen->copyPalette(0, 3);
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
} else {
uint32 fileSize;
uint8 *srcData;
Modified: scummvm/trunk/engines/kyra/sequences_hof.cpp
===================================================================
--- scummvm/trunk/engines/kyra/sequences_hof.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/sequences_hof.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -88,7 +88,7 @@
if (cseq.flags & 2) {
_screen->loadBitmap(cseq.cpsFile, 2, 2, _screen->getPalette(0).getData());
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
} else {
_screen->setCurPage(2);
_screen->clearPage(2);
@@ -101,7 +101,7 @@
if (cseq.flags & 1) {
_seqWsa->close();
_seqWsa->open(cseq.wsaFile, 0, _screen->getPalette(0).getData());
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
_seqWsa->displayFrame(0, 2, cseq.xPos, cseq.yPos, 0, 0, 0);
}
@@ -411,16 +411,16 @@
case 200:
seq_waitForTextsTimeout();
- _screen->fadePalette(_screen->getPalette(2).getData(), 64);
+ _screen->fadePalette(_screen->getPalette(2), 64);
break;
case 201:
- _screen->setScreenPalette(_screen->getPalette(2).getData());
+ _screen->setScreenPalette(_screen->getPalette(2));
_screen->updateScreen();
_screen->applyOverlay(0, 0, 320, 200, 2, _screen->getPalette(3).getData());
_screen->copyPage(2, 12);
_screen->copyRegion(0, 0, 0, 0, 320, 200, 2, 0);
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
_screen->updateScreen();
seq_resetActiveWSA(0);
seq_resetActiveWSA(1);
@@ -1781,25 +1781,27 @@
#ifdef ENABLE_LOL
int KyraEngine_HoF::seq_lolDemoScene1(WSAMovie_v2 *wsaObj, int x, int y, int frm) {
- uint8 *tmpPal = _screen->getPalette(2).getData();
+ Palette &tmpPal = _screen->getPalette(2);
if (!(_seqFrameCounter % 100)) {
if (_seqFrameCounter == 0) {
_sound->haltTrack();
_sound->playTrack(6);
}
- memcpy(tmpPal, _screen->getPalette(0).getData(), 0x300);
+ tmpPal.copy(_screen->getPalette(0));
+
for (int i = 3; i < 0x300; i++) {
tmpPal[i] = ((int)tmpPal[i] * 120) / 64;
if (tmpPal[i] > 0x3f)
tmpPal[i] = 0x3f;
}
+
seq_playTalkText(_rnd.getRandomBit());
_screen->setScreenPalette(tmpPal);
_screen->updateScreen();
delay(8);
} else {
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
_screen->updateScreen();
if (_seqFrameCounter == 40)
seq_playTalkText(3);
@@ -1924,15 +1926,17 @@
}
if (_seqFrameCounter % 175) {
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
} else {
- uint8 *tmpPal = _screen->getPalette(2).getData();
- memcpy(tmpPal, _screen->getPalette(0).getData(), 0x300);
+ Palette &tmpPal = _screen->getPalette(2);
+ tmpPal.copy(_screen->getPalette(0));
+
for (int i = 3; i < 0x300; i++) {
tmpPal[i] = ((int)tmpPal[i] * 120) / 64;
if (tmpPal[i] > 0x3f)
tmpPal[i] = 0x3f;
}
+
seq_playTalkText(_rnd.getRandomBit());
_screen->setScreenPalette(tmpPal);
_screen->updateScreen();
@@ -2045,36 +2049,34 @@
}
void KyraEngine_HoF::seq_sequenceCommand(int command) {
- uint8 pal[768];
-
for (int i = 0; i < 8; i++)
seq_resetActiveWSA(i);
switch (command) {
case 0:
- memset(pal, 0, 0x300);
- _screen->fadePalette(pal, 36);
+ _screen->fadeToBlack(36);
_screen->getPalette(0).clear();
_screen->getPalette(1).clear();
break;
case 1:
- memset(pal, 0x3F, 0x300);
seq_playTalkText(_rnd.getRandomBit());
- _screen->fadePalette(pal, 16);
- _screen->getPalette(0).copy(pal, 0, 256);
- _screen->getPalette(1).copy(pal, 0, 256);
+
+ memset(_screen->getPalette(0).getData(), 0x3F, 0x300);
+ _screen->fadePalette(_screen->getPalette(0), 16);
+
+ _screen->copyPalette(1, 0);
break;
case 3:
_screen->copyPage(2, 0);
- _screen->fadePalette(_screen->getPalette(0).getData(), 16);
+ _screen->fadePalette(_screen->getPalette(0), 16);
_screen->copyPalette(1, 0);
break;
case 4:
_screen->copyPage(2, 0);
- _screen->fadePalette(_screen->getPalette(0).getData(), 36);
+ _screen->fadePalette(_screen->getPalette(0), 36);
_screen->copyPalette(1, 0);
break;
@@ -2093,17 +2095,17 @@
break;
case 8:
- memset(pal, 0, 0x300);
- _screen->fadePalette(pal, 16);
- _screen->getPalette(0).copy(pal, 0, 256);
- _screen->getPalette(1).copy(pal, 0, 256);
+ _screen->fadeToBlack(16);
+ _screen->getPalette(0).clear();
+ _screen->getPalette(1).clear();
delay(120 * _tickLength);
break;
- case 9:
- for (int i = 0; i < 0x100; i++) {
- int pv = (_screen->getPalette(0)[3 * i] + _screen->getPalette(0)[3 * i + 1] + _screen->getPalette(0)[3 * i + 2]) / 3;
+ case 9: {
+ Palette &pal = _screen->getPalette(0);
+ for (int i = 0; i < 256; i++) {
+ int pv = (pal[3 * i] + pal[3 * i + 1] + pal[3 * i + 2]) / 3;
pal[3 * i] = pal[3 * i + 1] = pal[3 * i + 2] = pv & 0xff;
}
@@ -2112,9 +2114,8 @@
//pal[3 * i] = pal[3 * i + 1] = pal[3 * i + 2] = 0x3f;
_screen->fadePalette(pal, 64);
- _screen->getPalette(0).copy(pal, 0, 256);
- _screen->getPalette(1).copy(pal, 0, 256);
- break;
+ _screen->copyPalette(1, 0);
+ } break;
default:
break;
@@ -2435,7 +2436,7 @@
_screen->getPalette(0)[0x2f7] = _screen->getPalette(0)[textcolor * 3];
_screen->getPalette(0)[0x2f8] = _screen->getPalette(0)[textcolor * 3 + 1];
_screen->getPalette(0)[0x2f9] = _screen->getPalette(0)[textcolor * 3 + 2];
- _screen->fadePalette(_screen->getPalette(0).getData(), 0x18);
+ _screen->fadePalette(_screen->getPalette(0), 0x18);
_seqTextColor[0] = textcolor;
_screen->setTextColorMap(colorMap);
@@ -2445,7 +2446,7 @@
_screen->copyPage(2, 0);
_screen->updateScreen();
_screen->getPalette(0)[0x2f7] = _screen->getPalette(0)[0x2f8] = _screen->getPalette(0)[0x2f9] = 0;
- _screen->fadePalette(_screen->getPalette(0).getData(), 1);
+ _screen->fadePalette(_screen->getPalette(0), 1);
_screen->copyPage(2, 12);
seq_resetAllTextEntries();
@@ -2665,7 +2666,7 @@
for (int col = 133; col > 112; col--)
_screen->getPalette(0).copy(_screen->getPalette(0), col - 1, 1, col);
_screen->getPalette(0).copy(_screen->getPalette(0), 133, 1, 112);
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
}
delayUntil(_seqSubFrameEndTimeInternal);
Modified: scummvm/trunk/engines/kyra/sequences_lok.cpp
===================================================================
--- scummvm/trunk/engines/kyra/sequences_lok.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/sequences_lok.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -167,7 +167,7 @@
if (_flags.platform == Common::kPlatformAmiga) {
_screen->copyPalette(0, 1);
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
}
if ((_seq->playSequence(_seq_KyrandiaLogo, skipFlag()) && !seq_skipSequence()) || shouldQuit()) {
@@ -251,7 +251,7 @@
_screen->loadBitmap("TEXT_ENG.CPS", 3, 3, _screen->getPalette(0).getData());
else
warning("no story graphics file found");
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
_screen->copyRegion(0, 0, 0, 0, 320, 200, 3, 0);
if (_flags.lang == Common::JA_JPN) {
@@ -569,12 +569,9 @@
_sprites->_anims[i].play = false;
_animator->sprites()[i].active = 0;
}
- uint8 tmpPal[768];
- memcpy(tmpPal, _screen->getPalette(0).getData(), 768);
- memcpy(&tmpPal[684], palTable2()[0], 60);
- _screen->fadePalette(tmpPal, 72);
_screen->getPalette(0).copy(palTable2()[0], 0, 20, 228);
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->fadePalette(_screen->getPalette(0), 72);
+ _screen->setScreenPalette(_screen->getPalette(0));
setGameFlag(0xB3);
} else {
delayWithTicks(120);
@@ -951,7 +948,7 @@
_screen->fadeSpecialPalette(32, 228, 20, 60);
delay(60 * _tickLength);
_screen->loadBitmap("GEMHEAL.CPS", 3, 3, _screen->getPalette(0).getData());
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
_screen->shuffleScreen(8, 8, 304, 128, 2, 0, 1, 0);
uint32 nextTime = _system->getMillis() + 120 * _tickLength;
_finalA = new WSAMovie_v1(this);
@@ -1012,7 +1009,7 @@
drawSentenceCommand(_homeString[0], 179);
_screen->getPalette(2).clear();
- _screen->setScreenPalette(_screen->getPalette(2).getData());
+ _screen->setScreenPalette(_screen->getPalette(2));
_seqPlayerFlag = true;
_seq->playSequence(_seq_Reunion, false);
@@ -1124,9 +1121,9 @@
_screen->setCurPage(2);
_screen->getPalette(2).clear();
- _screen->setScreenPalette(_screen->getPalette(2).getData());
+ _screen->setScreenPalette(_screen->getPalette(2));
_screen->copyRegion(8, 32, 8, 32, 312, 128, 4, 0, Screen::CR_NO_P_CHECK);
- _screen->fadePalette(_screen->getPalette(0).getData(), 0x5A);
+ _screen->fadePalette(_screen->getPalette(0), 0x5A);
Common::Event event;
bool finished = false;
@@ -1683,7 +1680,7 @@
_screen->getPalette(0)[palPos++] = kyraGemPalette[i + _kyragemFadingState.gOffset];
_screen->getPalette(0)[palPos++] = kyraGemPalette[i + _kyragemFadingState.bOffset];
}
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
_animator->_updateScreen = true;
switch (_kyragemFadingState.nextOperation) {
case 0:
Modified: scummvm/trunk/engines/kyra/sequences_lol.cpp
===================================================================
--- scummvm/trunk/engines/kyra/sequences_lol.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/sequences_lol.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -42,12 +42,12 @@
showIntro();
if (_flags.isDemo) {
- _screen->fadePalette(_screen->getPalette(1).getData(), 30, 0);
+ _screen->fadePalette(_screen->getPalette(1), 30, 0);
_screen->loadBitmap("FINAL.CPS", 2, 2, _screen->getPalette(0).getData());
_screen->copyRegion(0, 0, 0, 0, 320, 200, 2, 0, Screen::CR_NO_P_CHECK);
- _screen->fadePalette(_screen->getPalette(0).getData(), 30, 0);
+ _screen->fadePalette(_screen->getPalette(0), 30, 0);
delayWithTicks(300);
- _screen->fadePalette(_screen->getPalette(1).getData(), 60, 0);
+ _screen->fadePalette(_screen->getPalette(1), 60, 0);
setupPrologueData(false);
return -1;
@@ -66,7 +66,7 @@
_screen->fprintString("SVM %s", 300 - width, 193, 0x67, 0x00, 0x04, gScummVMVersion);
_screen->setFont(Screen::FID_9_FNT);
- _screen->fadePalette(_screen->getPalette(0).getData(), 0x1E);
+ _screen->fadePalette(_screen->getPalette(0), 0x1E);
_screen->updateScreen();
_eventList.clear();
@@ -186,7 +186,7 @@
delete _chargenWSA; _chargenWSA = 0;
_screen->getPalette(0).clear();
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
if (shouldQuit())
return;
@@ -204,7 +204,7 @@
showStarcraftLogo();
_screen->getPalette(0).clear();
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
_screen->clearPage(0);
_screen->clearPage(4);
@@ -234,8 +234,8 @@
palNextFadeStep = _system->getMillis() + ((_tim->_palDelayAcc >> 8) * _tickLength);
_tim->_palDelayAcc &= 0xFF;
- if (!_screen->fadePalStep(_screen->getPalette(0).getData(), _tim->_palDiff)) {
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ if (!_screen->fadePalStep(_screen->getPalette(0), _tim->_palDiff)) {
+ _screen->setScreenPalette(_screen->getPalette(0));
_tim->_palDiff = 0;
}
}
@@ -259,7 +259,7 @@
delete _tim;
_tim = 0;
- _screen->fadePalette(_screen->getPalette(1).getData(), 30, 0);
+ _screen->fadePalette(_screen->getPalette(1), 30, 0);
}
int LoLEngine::chooseCharacter() {
@@ -306,7 +306,7 @@
if (_flags.use16ColorMode)
_screen->loadPalette("LOL.NOL", _screen->getPalette(0).getData());
- _screen->fadePalette(_screen->getPalette(0).getData(), 30, 0);
+ _screen->fadePalette(_screen->getPalette(0), 30, 0);
bool kingIntro = true;
while (!shouldQuit()) {
@@ -728,7 +728,7 @@
_sound->loadSoundFile("SOUND.DAT");
} else {
_screen->getPalette(0).clear();
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
if (shouldQuit())
return;
@@ -744,7 +744,7 @@
_tim = new TIMInterpreter(this, _screen, _system);
_screen->getPalette(0).clear();
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
_screen->clearPage(0);
_screen->clearPage(4);
@@ -773,8 +773,8 @@
palNextFadeStep = _system->getMillis() + ((_tim->_palDelayAcc >> 8) * _tickLength);
_tim->_palDelayAcc &= 0xFF;
- if (!_screen->fadePalStep(_screen->getPalette(0).getData(), _tim->_palDiff)) {
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ if (!_screen->fadePalStep(_screen->getPalette(0), _tim->_palDiff)) {
+ _screen->setScreenPalette(_screen->getPalette(0));
_tim->_palDiff = 0;
}
}
@@ -828,7 +828,7 @@
if (maxDifficulty)
_tim->displayText(0x8000, 0, 0xDC);
_screen->updateScreen();
- _screen->fadePalette(_screen->getPalette(0).getData(), 30, 0);
+ _screen->fadePalette(_screen->getPalette(0), 30, 0);
while (!checkInput(0) && !shouldQuit())
delay(_tickLength);
@@ -907,7 +907,7 @@
assert(monsterPal);
_screen->getPalette(0).copy(monsterPal, 0, 40, 88);
- _screen->fadePalette(_screen->getPalette(0).getData(), 30);
+ _screen->fadePalette(_screen->getPalette(0), 30);
uint32 waitTimer = _system->getMillis();
@@ -1049,7 +1049,7 @@
doorRedraw = false;
} else {
if (!monsterAnimFrame)
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
_screen->copyRegion(0, 0, 0, 0, 320, 200, 2, page, Screen::CR_NO_P_CHECK);
@@ -1130,7 +1130,7 @@
loadOutroShapes(curShapeFile, shapes);
_screen->getPalette(0).copy(monsterPal, curShapeFile * 40, 40, 88);
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
needNewShape = false;
}
Modified: scummvm/trunk/engines/kyra/text_lok.cpp
===================================================================
--- scummvm/trunk/engines/kyra/text_lok.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/text_lok.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -333,7 +333,7 @@
_currSentenceColor[1] = _screen->getPalette(0)[766] = _screen->getPalette(0)[color*3+1];
_currSentenceColor[2] = _screen->getPalette(0)[767] = _screen->getPalette(0)[color*3+2];
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
_startSentencePalIndex = 0;
}
@@ -371,7 +371,7 @@
_screen->getPalette(0)[765] = _currSentenceColor[0];
_screen->getPalette(0)[766] = _currSentenceColor[1];
_screen->getPalette(0)[767] = _currSentenceColor[2];
- _screen->setScreenPalette(_screen->getPalette(0).getData());
+ _screen->setScreenPalette(_screen->getPalette(0));
if (finished) {
_fadeText = false;
Modified: scummvm/trunk/engines/kyra/vqa.cpp
===================================================================
--- scummvm/trunk/engines/kyra/vqa.cpp 2009-06-22 02:36:16 UTC (rev 41742)
+++ scummvm/trunk/engines/kyra/vqa.cpp 2009-06-22 02:36:54 UTC (rev 41743)
@@ -520,9 +520,8 @@
// The frame has been decoded
- if (_frameInfo[frameNum] & 0x80000000) {
- _vm->screen()->setScreenPalette(_vm->screen()->getPalette(0).getData());
- }
+ if (_frameInfo[frameNum] & 0x80000000)
+ _vm->screen()->setScreenPalette(_vm->screen()->getPalette(0));
int blockPitch = _header.width / _header.blockW;
This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.
More information about the Scummvm-git-logs
mailing list