{"id":300,"date":"2026-07-07T16:50:06","date_gmt":"2026-07-07T16:50:06","guid":{"rendered":"https:\/\/blogs.scummvm.org\/mohit\/?p=300"},"modified":"2026-07-07T17:33:26","modified_gmt":"2026-07-07T17:33:26","slug":"closing-the-dungeon-gates","status":"publish","type":"post","link":"https:\/\/blogs.scummvm.org\/mohit\/2026\/07\/07\/closing-the-dungeon-gates\/","title":{"rendered":"Closing the Dungeon Gates"},"content":{"rendered":"

Hello and welcome back! This is Week 6 of my GSoC journey, and\u00a0I’m excited to share that I’ve finally completed the Dungeon Master engine!<\/strong><\/p>\n

Fixing the Endianness Problem<\/span><\/h4>\n

As I mentioned in the last blog, using READ_LE_UINT16<\/code> and WRITE_LE_UINT16<\/code> macros had a deeper issue \u2014 loadDungeonFile()<\/code> reads Big-Endian file data and stores it into _thingData<\/code> as native-endian uint16s<\/code>, so on a Big-Endian machine the whole setup breaks regardless of the macro fix.<\/p>\n

So I spent the first half of this week replacing those byte offset macros by parsing the file data directly into respective arrays of structs at load time. This involved removing all raw byte-to-struct pointer casting (e.g., (Door *)rawDat<\/code>) from the codebase and updating the file loading and saving routines to read and write fields one at a time using stream operations like readUint16BE<\/code> and writeUint16BE<\/code>.<\/p>\n

Uncovering Hidden Bugs<\/span><\/h4>\n

These changes also brought some new freezes and crashes to the surface \u2014 bugs that were silently hiding behind the old unsafe code. Raw pointer arithmetic was allowing silent reads and writes to out-of-bounds memory, which switching to Common::Array<\/code> now catches immediately as crashes or assertions. Unsafe casting was hiding cases where one object type was being read as another, which strict C++ type assertions now catch instantly.<\/p>\n

Debugging the\u00a0Endgame<\/span><\/h4>\n

The endgame sequence had quite a few bugs hiding in it. When you capture the Lord of Chaos and fuse him with The Firestaff, the game unlinks the fluxcage \u2014 but in fuseSequence()<\/code>, curThing<\/code> was never being reset after that, causing the loop to run forever.<\/p>\n

When the Lord of Chaos was turned back to normal, the message displayed was also corrupted \u2014 wrkString<\/code> was declared outside the loop so it kept accumulating characters from previous words, and a stray \\0<\/code> append was preventing it from reading past the first word. Moving wrkString<\/code> inside the loop and removing \\0<\/code> fixed the issue.<\/p>\n

Another text bug had only the second row showing up in the message area \u2014 the first row was being cleared before it could scroll up. createNewRow()<\/code> was clearing the line buffer before the scroll animation could finish, so the fix was to wait in a small delay loop until _isScrolling<\/code> is false before proceeding.<\/p>\n

Beyond that, I also added support for returning to the launcher after closing the game, along with other small fixes to get the engine to a solid, playable state.<\/p>\n

Crash When Renaming Champion in the Hall of Champions<\/span><\/h4>\n

Clicking outside the character select table during champion renaming was calculating an invalid character index from out-of-bounds coordinates, causing a crash. The fix was straightforward \u2014 enabling a bounds check that was previously disabled, so clicks outside the table are now simply ignored.<\/p>\n

Implementing Extended Saves<\/span><\/h4>\n

I also implemented extended saves for the DM engine, allowing the game to make use of ScummVM’s extended save functionality.<\/p>\n

That was it for this week! It’s been a long road getting the DM engine to this point, but it’s rewarding to finally see the work of the previous contributor and Strangerke pay off after 10 years. You can find the previous contributor’s blog here<\/a>.<\/p>\n

Thanks to Sev for helping me throughout this.<\/p>\n

I’ll see you in the next blog as I dive into a new engine. Until then, bye! \ud83d\udc4b<\/p>\n","protected":false},"excerpt":{"rendered":"

Hello and welcome back! This is Week 6 of my GSoC journey, and\u00a0I’m excited to share that I’ve finally completed the Dungeon Master engine! Fixing the Endianness Problem As I mentioned in the last blog, using READ_LE_UINT16 and WRITE_LE_UINT16 macros had a deeper issue \u2014 loadDungeonFile() reads Big-Endian file data and stores it into _thingData […]<\/p>\n","protected":false},"author":30,"featured_media":0,"comment_status":"open","ping_status":"open","sticky":false,"template":"","format":"standard","meta":{"footnotes":""},"categories":[10],"tags":[],"class_list":["post-300","post","type-post","status-publish","format-standard","hentry","category-week-6"],"_links":{"self":[{"href":"https:\/\/blogs.scummvm.org\/mohit\/wp-json\/wp\/v2\/posts\/300","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/blogs.scummvm.org\/mohit\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/blogs.scummvm.org\/mohit\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/blogs.scummvm.org\/mohit\/wp-json\/wp\/v2\/users\/30"}],"replies":[{"embeddable":true,"href":"https:\/\/blogs.scummvm.org\/mohit\/wp-json\/wp\/v2\/comments?post=300"}],"version-history":[{"count":20,"href":"https:\/\/blogs.scummvm.org\/mohit\/wp-json\/wp\/v2\/posts\/300\/revisions"}],"predecessor-version":[{"id":320,"href":"https:\/\/blogs.scummvm.org\/mohit\/wp-json\/wp\/v2\/posts\/300\/revisions\/320"}],"wp:attachment":[{"href":"https:\/\/blogs.scummvm.org\/mohit\/wp-json\/wp\/v2\/media?parent=300"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/blogs.scummvm.org\/mohit\/wp-json\/wp\/v2\/categories?post=300"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/blogs.scummvm.org\/mohit\/wp-json\/wp\/v2\/tags?post=300"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}