thirdparty/scummvm
2
0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Cleanup: Removed obsolete TODOs

Browse source 
svn-id: r28682
This commit is contained in:
Filippos Karapetis 2007-08-20 20:00:40 +00:00
parent a6171ec349
commit c50c20b2c2
5 changed files with 4 additions and 13 deletions
Download patch file Download diff file Expand all files Collapse all files

3
engines/saga/displayinfo.h
View file

@ -280,7 +280,7 @@ static const GameDisplayInfo ITE_DisplayInfo = {
}; };
//TODO: ihnm // IHNM
#define IHNM_CONVERSE_MAX_TEXT_WIDTH (485 - 8) #define IHNM_CONVERSE_MAX_TEXT_WIDTH (485 - 8)
#define IHNM_CONVERSE_TEXT_HEIGHT 10 #define IHNM_CONVERSE_TEXT_HEIGHT 10
#define IHNM_CONVERSE_TEXT_LINES 11 #define IHNM_CONVERSE_TEXT_LINES 11
@ -354,7 +354,6 @@ static PanelButton IHNM_LoadPanelButtons[] = {
}; };
static PanelButton IHNM_SavePanelButtons[] = { static PanelButton IHNM_SavePanelButtons[] = {
// TODO
{kPanelButtonSave, 25,79, 80,25, kTextSave,'s',0, 0,0,0}, {kPanelButtonSave, 25,79, 80,25, kTextSave,'s',0, 0,0,0},
{kPanelButtonSave, 155,79, 80,25, kTextCancel,'c',0, 0,0,0}, {kPanelButtonSave, 155,79, 80,25, kTextCancel,'c',0, 0,0,0},
{kPanelButtonSaveEdit, 26,57, 209,17, 0,'-',0, 0,0,0}, {kPanelButtonSaveEdit, 26,57, 209,17, 0,'-',0, 0,0,0},

4
engines/saga/interface.cpp
View file

@ -420,8 +420,8 @@ void Interface::setMode(int mode) {
switch (_panelMode) { switch (_panelMode) {
case kPanelMain: case kPanelMain:
if (_vm->getGameType() == GType_IHNM) // FIXME: Implement IHNM differences from ExecuteInventoryPanel for IHNM (though I believe they're already
warning("FIXME: Implement IHNM differences from ExecuteInventoryPanel"); // implemented)
_mainPanel.currentButton = NULL; _mainPanel.currentButton = NULL;
break; break;

3
engines/saga/itedata.h
View file

@ -41,9 +41,6 @@ enum ActorFlags {
kNoScale = 0x80 // (1<<7) Actor is not scaled kNoScale = 0x80 // (1<<7) Actor is not scaled
}; };
// TODO: This doesn't quite correspond to the original Actor struct, so I'm not
// sure if I got it right.
struct ActorTableData { struct ActorTableData {
byte flags; byte flags;
byte nameIndex; byte nameIndex;

2
engines/saga/saveload.cpp
View file

@ -237,8 +237,8 @@ void SagaEngine::load(const char *fileName) {
// Some older saves were not written in an endian safe fashion. // Some older saves were not written in an endian safe fashion.
// We try to detect this here by checking for extremly high version values. // We try to detect this here by checking for extremly high version values.
// If found, we retry with the data swapped. // If found, we retry with the data swapped.
// FIXME: Maybe display a warning/error message instead?
if (_saveHeader.version > 0xFFFFFF) { if (_saveHeader.version > 0xFFFFFF) {
warning("This savegame is not endian safe, retrying with the data swapped");
_saveHeader.version = SWAP_BYTES_32(_saveHeader.version); _saveHeader.version = SWAP_BYTES_32(_saveHeader.version);
} }

5
engines/saga/scene.cpp
View file

@ -811,11 +811,6 @@ void Scene::loadScene(LoadSceneParams *loadSceneParams) {
_vm->_sound->stopAll(); _vm->_sound->stopAll();
// FIXME: Does IHNM use scene background music, or is all the
// music scripted? At the very least, it shouldn't try
// to start song 0 at the beginning of the game, since
// it's the end credits music.
if (_vm->getGameType() == GType_ITE) { if (_vm->getGameType() == GType_ITE) {
if (_sceneDescription.musicResourceId >= 0) { if (_sceneDescription.musicResourceId >= 0) {
event.type = kEvTOneshot; event.type = kEvTOneshot;