Let’s jump right into the main focus of this week’s development: savefiles.<\/p>\n\n\n\n
The original game uses a FILE setup to read and write savefiles, which we will need to refactor as with everything else we’ve done so far. <\/p>\n\n\n\n
I will outline the process of how a save is written\/read:<\/p>\n\n\n\n
When we click “save” in the options menu, a event for I will showcase its contents:<\/p>\n\n\n\n I’ve summarized it a bit to keep it simple, but the gist of it is that we create this This is how one of those methods look like:<\/p>\n\n\n\n We may as well take a look into this The important takeaway here is that we first write the “chunk” header, which consists of the id (the 4 bytes that serve as the identifier, in this case “GLOB”) and the size. After that comes the actual contents that we want to save, that we store in a So then, how do we refactor this for portability? The answer is to use streams. More specifically, for writing into savefiles, we use You remember how we declared And then finally, once we’re done writing all of the data, we call The code for I’ve removed some debug message calls that aren’t of interest to us. As you can see, we write the chunk header first, and then write all of the individual members of As for loading, the process is very similar:<\/p>\n\n\n\n I’ve omitted some code for simplicity. Here we construct a So, all in all, it’s not very hard work. As long as one pays attention to the sizes3<\/sup>, there is not much to think about. I personally cut down some time by streamlining with Vim macros, and I’m sure someone could write a script to automate much of this, but I didn’t go that far.<\/p>\n\n\n\n Now, I’m showing all of the code commented out, but we proceeded in steps. We started with an An interesting thing to note here, is that because the code is mostly the same, it’s very easy to notice when it’s different. In the majority of the code we use methods such as This leads me to ponder that the spell system was written much later (or earlier) in development.<\/p>\n\n\n\n Now, this is all very fun, but even after doing all of that the work is not done. We still need to: Get rid of the original code; Replace sizeof by portable alternatives; Fix any crashes or bugs detected during playtest. As the saying goes, work never ends!<\/p>\n\n\n\n [1] [2] The original had methods for seeking to the next chunk when the chunk is thought to not be completed. I removed those operations because they did not seem necessary. I believe this shouldn’t be a problem for saves created locally, but I wonder if it will have an effect when loading original saves. I’m writing this here so that I don’t forget it one week later when I can’t get loading to work for mysterious reasons.<\/em><\/p>\n\n\n\n [3] Extra points if the size is knowable. There were some instances of Let’s jump right into the main focus of this week’s development: savefiles. The original game uses a FILE setup to read and write savefiles, which we will need to refactor as with everything else we’ve done so far. I will outline the process of how a save is written\/read: When we click “save” in the […]<\/p>\n","protected":false},"author":7,"featured_media":0,"comment_status":"open","ping_status":"open","sticky":false,"template":"","format":"standard","meta":{"footnotes":""},"categories":[1],"tags":[],"class_list":["post-43","post","type-post","status-publish","format-standard","hentry","category-uncategorized"],"_links":{"self":[{"href":"https:\/\/blogs.scummvm.org\/ayyg\/wp-json\/wp\/v2\/posts\/43","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/blogs.scummvm.org\/ayyg\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/blogs.scummvm.org\/ayyg\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/blogs.scummvm.org\/ayyg\/wp-json\/wp\/v2\/users\/7"}],"replies":[{"embeddable":true,"href":"https:\/\/blogs.scummvm.org\/ayyg\/wp-json\/wp\/v2\/comments?post=43"}],"version-history":[{"count":2,"href":"https:\/\/blogs.scummvm.org\/ayyg\/wp-json\/wp\/v2\/posts\/43\/revisions"}],"predecessor-version":[{"id":45,"href":"https:\/\/blogs.scummvm.org\/ayyg\/wp-json\/wp\/v2\/posts\/43\/revisions\/45"}],"wp:attachment":[{"href":"https:\/\/blogs.scummvm.org\/ayyg\/wp-json\/wp\/v2\/media?parent=43"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/blogs.scummvm.org\/ayyg\/wp-json\/wp\/v2\/categories?post=43"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/blogs.scummvm.org\/ayyg\/wp-json\/wp\/v2\/tags?post=43"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}cmdOptionsSaveGame<\/code> is sent, which eventually leads to a call for saveGameState<\/code>. Here is the GDB Backtrace:<\/p>\n\n\n\n#0 Saga2::saveGameState(short, char*)\n (saveNo=32767, saveName=0x7fffffffa170 \"\\263\\212\\265A\")\n at engines\/saga2\/loadsave.cpp:131\n#1 0x0000555555f34a1a in Saga2::cmdFileSave(Saga2::gEvent&) (ev=...)\n at engines\/saga2\/uidialog.cpp:1662\n#2 0x0000555555e130c5 in Saga2::gPanel::notify(Saga2::gEventType, int)\n (this=0x60e00002fb40, type=Saga2::gEventNewValue, value=1)\n at engines\/saga2\/panel.cpp:169\n#3 0x0000555555f464e0 in Saga2::gCompButton::pointerRelease(Saga2::gPanelMessage&)\n (this=0x60e00002fb40) at engines\/saga2\/button.cpp:553\n#4 0x0000555555e1baeb in Saga2::gToolBase::handleMouse(Common::Event&, unsigned int)\n (this=0x555557592e40 <Saga2::G_BASE>, event=..., time=71827)\n at engines\/saga2\/panel.cpp:985\n#5 0x0000555555d843e8 in Saga2::processEventLoop(bool) (updateScreen=true)\n at engines\/saga2\/main.cpp:304\n#6 0x0000555555d840b8 in Saga2::EventLoop(bool&, bool)\n (running=@0x7fffffffa5e0: false) at engines\/saga2\/main.cpp:270\n#7 0x0000555555f2ea42 in Saga2::FileDialog(short) (fileProcess=0)\n at engines\/saga2\/uidialog.cpp:760\n#8 0x0000555555f351ca in Saga2::cmdOptionsSaveGame(Saga2::gEvent&) (ev=...)\n at engines\/saga2\/uidialog.cpp:1719\n#9 0x0000555555e130c5 in Saga2::gPanel::notify(Saga2::gEventType, int)\n (this=0x60e00002e720, type=Saga2::gEventNewValue, value=1)\n at engines\/saga2\/panel.cpp:169\n#10 0x0000555555f464e0 in Saga2::gCompButton::pointerRelease(Saga2::gPanelMessage&)\n (this=0x60e00002e720) at engines\/saga2\/button.cpp:553\n#11 0x0000555555e1baeb in Saga2::gToolBase::handleMouse(Common::Event&, unsigned int)\n (this=0x555557592e40 <Saga2::G_BASE>, event=..., time=58752)\n at engines\/saga2\/panel.cpp:985\n#12 0x0000555555d843e8 in Saga2::processEventLoop(bool) (updateScreen=true)\n at engines\/saga2\/main.cpp:304<\/code><\/pre>\n\n\n\nvoid saveGameState( int16 saveNo, char *saveName )\n{\n\tpauseTimer();\n\n\ttry\n\t{\n\t\tSaveFileConstructor saveGame( saveNo, saveName );\n\n\t\tsaveGlobals( saveGame );\n\t\tsaveTimer( saveGame );\n...\n\t\tsavePaletteState( saveGame );\n\t\tsaveContainerNodes( saveGame );\n\t}\n\tcatch ( SaveFileWriteError writeError )\n\t{\n\t\t\t\/\/\tFor now this simply rethrows the exception. Eventually,\n\t\t\t\/\/\tthis must notify the user that the game was not saved and\n\t\t\t\/\/\trecover gracefully.\n\t\t\/\/throw writeError;\n\t\tthrow SystemError(cpSavFileWrite,\"Writing saved game\");\n\t}\n\n\tresumeTimer();\n}<\/code><\/pre>\n\n\n\nSaveFileConstructor<\/code> object and pass it through all of these methods, saving parts of the game into saveGame<\/code>.<\/p>\n\n\n\n\/\/-----------------------------------------------------------------------\n\/\/\tStore miscellaneous globals in a save file\n\nvoid saveGlobals( SaveFileConstructor &saveGame )\n{\n\tGlobalsArchive\tarchive;\n\n\tarchive.objectIndex\t\t\t\t\t= objectIndex;\n\tarchive.actorIndex\t\t\t\t\t= actorIndex;\n\tarchive.brotherBandingEnabled\t\t= brotherBandingEnabled;\n\tarchive.centerActorIndicatorEnabled\t= centerActorIndicatorEnabled;\n\tarchive.interruptableMotionsPaused\t= interruptableMotionsPaused;\n\tarchive.objectStatesPaused\t\t\t= objectStatesPaused;\n\tarchive.actorStatesPaused\t\t\t= actorStatesPaused;\n\tarchive.actorTasksPaused\t\t\t= actorTasksPaused;\n\tarchive.combatBehaviorEnabled\t\t= combatBehaviorEnabled;\n\tarchive.backgroundSimulationPaused\t= backgroundSimulationPaused;\n\n\tsaveGame.writeChunk(\n\t\tMakeID( 'G', 'L', 'O', 'B' ),\n\t\t&archive,\n\t\tsizeof( archive ) );\n}<\/code><\/pre>\n\n\n\nwriteChunk<\/code> method to see how exactly one writes into the file.<\/p>\n\n\n\n\/\/----------------------------------------------------------------------\n\/\/\tCreate a new chunk and write the chunk data.\n\nbool SaveFileConstructor::writeChunk( ChunkID id, void *buf, int32 size )\n{\n\t\t\/\/\tDetermine if file position is at end of previous chunk\n\tif ( posInChunk < chunkSize ) return FALSE;\n\n\tASSERT( posInChunk == chunkSize );\n\n\tSaveFileChunkInfo\tchunkHeader;\n\n\t\t\/\/\tInitialize the chunk header\n\tchunkHeader.id = id;\n\tchunkHeader.size = size;\n\n\t\t\/\/\tWrite the chunk header\n\tif ( fwrite( &chunkHeader, sizeof( chunkHeader ), 1, fileHandle ) != 1 )\n\t\tthrow SaveFileWriteError( \"Error writing save game chunk header\" );\n\n\t\t\/\/\tWrite the chunk data\n\tif ( size > 0 && fwrite( buf, size, 1, fileHandle ) != 1 )\n\t\tthrow SaveFileWriteError( \"Error writing save game data\" );\n\n\t\t\/\/\tInitialize the chunk varibles to indicate the file position is\n\t\t\/\/\tat the end of the chunk\n\tchunkSize = posInChunk = size;\n\n\t\t\/\/\tReturn success\n\treturn TRUE;\n}<\/code><\/pre>\n\n\n\nGlobalsArchive<\/code> here.<\/p>\n\n\n\nCommon::OutSaveFile<\/a><\/code> and for reading we use Common::InSaveFile<\/a><\/code>. The latter is simply a typedef<\/code> for Common::SeekableReadStream<\/code>.<\/p>\n\n\n\nsaveGame<\/code> inside of saveGameState<\/code>? We shall do the same thing here. We declare a Common::OutSaveFile *out<\/code> pointer that we initialize using OSystem’s SaveFileManager<\/code>.1<\/sup> After that, we send it as an argument to all of these methods. Obviously, that requires us to rewrite all of these individual methods.<\/p>\n\n\n\nCommon::Error saveGameState(int16 saveNo, char *saveName) {\n\tpauseTimer();\n\n\tdebugC(1, kDebugSaveload, \"Saving game\");\n\n\tCommon::OutSaveFile *out = g_vm->getSaveFileManager()->openForSaving(getSaveFileName(saveNo), false);\n\tif (!out)\n\t\treturn Common::kCreatingFileFailed;\n\n\tSaveFileHeader header;\n\theader.gameID = gameID;\n\theader.saveName = saveName;\n\n\theader.write(out);\n\n\tsaveGlobals(out);\n\tsaveTimer(out);\n...\n\tsavePaletteState(out);\n\tsaveContainerNodes(out);\n\n\tout->finalize();\n\n\tdelete out;\n\n\tresumeTimer();\n\n\treturn Common::kNoError;\n}<\/code><\/pre>\n\n\n\nout->finalize()<\/code> and delete the object.<\/p>\n\n\n\nsaveGlobals<\/code> becomes as follows, by the way:<\/p>\n\n\n\nvoid saveGlobals(Common::OutSaveFile *out) {\n\tout->write(\"GLOB\", 4);\n\tout->writeUint32LE(sizeof(GlobalsArchive));\n\n\tout->writeUint32LE(objectIndex);\n\tout->writeUint32LE(actorIndex);\n\tout->writeByte(brotherBandingEnabled);\n\tout->writeByte(centerActorIndicatorEnabled);\n\tout->writeByte(interruptableMotionsPaused);\n\tout->writeByte(objectStatesPaused);\n\tout->writeByte(actorStatesPaused);\n\tout->writeByte(actorTasksPaused);\n\tout->writeByte(combatBehaviorEnabled);\n\tout->writeByte(backgroundSimulationPaused);\n}<\/code><\/pre>\n\n\n\nGlobalsArchive<\/code> according to their order within the struct. An important thing to note here is the sizeof(GlobalsArchive)<\/code>. Due to padding shenanigans, the size of these structs may not be the same for all platforms. This obviously creates portability issues when you try to load saves from other machines. That is why we need to change each and every of one of these suspicious sizeof<\/code> operations to a constant such as kGlobalsArchiveSize<\/code>.<\/p>\n\n\n\nvoid loadSavedGameState(int16 saveNo) {\n\tuint32 loadFlags = 0;\n\n\tpauseTimer();\n\n\tCommon::InSaveFile *in = g_vm->getSaveFileManager()->openForLoading(getSaveFileName(saveNo));\n\tChunkID id;\n\tint32 chunkSize;\n\tbool notEOF;\n\n\tnotEOF = firstChunk(in, id, chunkSize);\n\twhile (notEOF) {\n\t\tswitch (id) {\n\t\tcase MKTAG('G', 'L', 'O', 'B'):\n\t\t\tloadGlobals(in);\n\t\t\tloadFlags |= loadGlobalsFlag;\n\t\t\tbreak;\n ...\n }\n \n notEOF = nextChunk(in, id, chunkSize);\n }\n...\n}<\/code><\/pre>\n\n\n\nCommon::InSaveFile<\/code> similar to before. We load in each chunk’s header with firstChunk<\/code> or nextChunk<\/code>2<\/sup>, and according to the ChunkID we load in the appropriate chunk.loadGlobals<\/code> is like this, by the way:<\/p>\n\n\n\nvoid loadGlobals(Common::InSaveFile *in) {\n\tobjectIndex = in->readUint32LE();\n\tactorIndex = in->readUint32LE();\n\tbrotherBandingEnabled = in->readByte();\n\tcenterActorIndicatorEnabled = in->readByte();\n\tinterruptableMotionsPaused = in->readByte();\n\tobjectStatesPaused = in->readByte();\n\tactorStatesPaused = in->readByte();\n\tactorTasksPaused = in->readByte();\n\tcombatBehaviorEnabled = in->readByte();\n\tbackgroundSimulationPaused = in->readByte();\n}<\/code><\/pre>\n\n\n\n#if 0<\/code> block surrounding all of the save methods and took things out of it one by one. It took a while, but because of that I learned a new skill: Vim macros.<\/p>\n\n\n\nvoid *archive(...)<\/code> or constructX(void**)<\/code>, but in spellio.cpp<\/code> the save\/loading methods are named quite different:<\/p>\n\n\n\n\/\/ ------------------------------------------------------------------\n\/\/ serialize active spells\n\nvoid saveSpellState( SaveFileConstructor &saveGame )\n{\n\tactiveSpells.save(saveGame);\n}\n\n\/\/ ------------------------------------------------------------------\n\/\/ read serialized active spells\n\nvoid loadSpellState( SaveFileReader &saveGame )\n{\n\tactiveSpells.load(saveGame);\n}<\/code><\/pre>\n\n\n\n
\n\n\n\n
\n\n\n\ngetSaveFileName(int16)<\/code> is a method that returns the filename according to the slot. For instance getSaveFileName(0) returns “000.SAV” as a Common::String.<\/p>\n\n\n\nenum SpellID<\/code> being saved into the archive. This is a problem because enums could have all kinds of size depending on the compiler. Will this have to be reverse engineered !?<\/p>\n","protected":false},"excerpt":{"rendered":"