libretro/scummvm
1
0
Fork 0
mirror of https://github.com/libretro/scummvm.git synced 2025-02-25 13:42:37 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

TINSEL: Add playTime to saved game and display it

Browse Source 
Also bumps the saved game format version to 3.

I changed the order of how the data is saved so that the playtime could
be read in without skipping as much data.

This is because querySaveMetaInfos only needs the metafields where as
saveload::DoRestore() needs more of the header.

A crash was happening when trying to delete a saved game from the launcher.
It is because the engine is not initalized. I assign a dummy value to
resolve the issue.

Loading saved games from previous versions works.

When an old version saved game is loaded it will start with zero playtime.

Old saved games are shown as not having playtime data.
This commit is contained in:
David Fioramonti 2018-07-06 15:37:46 -07:00 committed by Eugene Sandulenko
parent 5cf3b75deb
commit f06cf65479
2 changed files with 29 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
engines/tinsel/detection.cpp
View File

@ -113,6 +113,7 @@ bool TinselMetaEngine::hasFeature(MetaEngineFeature f) const {
(f == kSupportsDeleteSave) ||
(f == kSimpleSavesNames) ||
(f == kSavesSupportMetaInfo) ||
(f == kSavesSupportPlayTime) ||
(f == kSavesSupportCreationDate);
}
@ -134,16 +135,7 @@ bool Tinsel::TinselEngine::hasFeature(EngineFeature f) const {
SaveStateDescriptor TinselMetaEngine::querySaveMetaInfos(const char *target, int slot) const {
Common::String fileName;
if (slot < 0)
return SaveStateDescriptor();
else if (slot < 10)
fileName = Common::String::format("%s.00%d", target, slot);
else if (slot < 100)
fileName = Common::String::format("%s.0%d", target, slot);
else if (slot < 1000)
fileName = Common::String::format("%s.%d", target, slot);
else
warning("Too many slots!");
fileName = Common::String::format("%s.%03u", target, slot);
Common::InSaveFile *file = g_system->getSavefileManager()->openForLoading(fileName);
@ -153,7 +145,7 @@ SaveStateDescriptor TinselMetaEngine::querySaveMetaInfos(const char *target, int
file->readUint32LE(); // skip id
file->readUint32LE(); // skip size
file->readUint32LE(); // skip version
uint32 ver = file->readUint32LE();
char saveDesc[Tinsel::SG_DESC_LEN];
file->read(saveDesc, sizeof(saveDesc));
@ -165,9 +157,16 @@ SaveStateDescriptor TinselMetaEngine::querySaveMetaInfos(const char *target, int
int8 tm_mday = file->readSByte();
int8 tm_hour = file->readSByte();
int8 tm_min = file->readSByte();
file->readSByte(); // skip secs
desc.setSaveDate(1900+tm_year, tm_mon, tm_mday);
desc.setSaveTime(tm_hour, tm_min);
if (ver >= 3) {
uint32 playTime = file->readUint32LE(); // playTime in seconds
desc.setPlayTime(playTime);
}
delete file;
return desc;
}

21
engines/tinsel/saveload.cpp
View File

@ -55,7 +55,7 @@ namespace Tinsel {
* only saves/loads those which are valid for the version of the savegame
* which is being loaded/saved currently.
*/
#define CURRENT_VER 2
#define CURRENT_VER 3
//----------------- GLOBAL GLOBAL DATA --------------------
@ -86,6 +86,7 @@ struct SaveGameHeader {
uint32 ver;
char desc[SG_DESC_LEN];
TimeDate dateTime;
uint32 playTime;
bool scnFlag;
byte language;
uint16 numInterpreters; // Savegame version 2 or later only
@ -94,7 +95,7 @@ struct SaveGameHeader {
enum {
DW1_SAVEGAME_ID = 0x44575399, // = 'DWSc' = "DiscWorld 1 ScummVM"
DW2_SAVEGAME_ID = 0x44573253, // = 'DW2S' = "DiscWorld 2 ScummVM"
SAVEGAME_HEADER_SIZE = 4 + 4 + 4 + SG_DESC_LEN + 7 + 1 + 1 + 2
SAVEGAME_HEADER_SIZE = 4 + 4 + 4 + SG_DESC_LEN + 7 + 4 + 1 + 1 + 2
};
#define SAVEGAME_ID (TinselV2 ? (uint32)DW2_SAVEGAME_ID : (uint32)DW1_SAVEGAME_ID)
@ -152,6 +153,11 @@ static bool syncSaveGameHeader(Common::Serializer &s, SaveGameHeader &hdr) {
syncTime(s, hdr.dateTime);
if (hdr.ver >= 3)
s.syncAsUint32LE(hdr.playTime);
else
hdr.playTime = 0;
int tmp = hdr.size - s.bytesSynced();
// NOTE: We can't use SAVEGAME_ID here when attempting to remove a saved game from the launcher,
@ -184,7 +190,10 @@ static bool syncSaveGameHeader(Common::Serializer &s, SaveGameHeader &hdr) {
hdr.numInterpreters = NUM_INTERPRET;
s.syncAsUint16LE(hdr.numInterpreters);
} else {
hdr.numInterpreters = (TinselV2 ? 70 : 64) - 20;
if(_vm) // See comment above about bug #3387551
hdr.numInterpreters = (TinselV2 ? 70 : 64) - 20;
else
hdr.numInterpreters = 50; // This value doesn't matter since the saved game is being deleted.
}
// Skip over any extra bytes
@ -495,6 +504,11 @@ static bool DoRestore() {
return false;
}
if (hdr.ver >= 3)
_vm->setTotalPlayTime(hdr.playTime);
else
_vm->setTotalPlayTime(0);
// Load in the data. For older savegame versions, we potentially need to load the data twice, once
// for pre 1.5 savegames, and if that fails, a second time for 1.5 savegames
int numInterpreters = hdr.numInterpreters;
@ -589,6 +603,7 @@ static void DoSave() {
memset(hdr.desc, 0, SG_DESC_LEN);
Common::strlcpy(hdr.desc, g_SaveSceneDesc, SG_DESC_LEN);
g_system->getTimeAndDate(hdr.dateTime);
hdr.playTime = _vm->getTotalPlayTime();
hdr.scnFlag = _vm->getFeatures() & GF_SCNFILES;
hdr.language = _vm->_config->_language;