From eef8e4de22e799eba2a9011ea15bdb3fc6e3faec Mon Sep 17 00:00:00 2001
From: Willem Jan Palenstijn <wjp@usecode.org>
Date: Sat, 26 Nov 2011 17:07:40 +0100
Subject: [PATCH] DREAMWEB: Convert 'loadposition'

---
 engines/dreamweb/saveload.cpp | 121 ++++++++++++----------------------
 engines/dreamweb/stubs.cpp    |   8 +++
 engines/dreamweb/stubs.h      |   1 +
 3 files changed, 50 insertions(+), 80 deletions(-)

diff --git a/engines/dreamweb/saveload.cpp b/engines/dreamweb/saveload.cpp
index 42f78a1786ec..60b72365cf8f 100644
--- a/engines/dreamweb/saveload.cpp
+++ b/engines/dreamweb/saveload.cpp
@@ -102,53 +102,35 @@ void DreamGenContext::doload() {
 		}
 
 
-		// TODO: proper scheme for filename, in a separate function
-		//Common::String filename = ConfMan.getActiveDomainName() + Common::String::format(".d%02d", savegameId);
-		Common::String filename = Common::String::format("DREAMWEB.D%02d", savegameId);
-		debug(1, "Loading from filename: %s", filename.c_str());
-		engine->openSaveFileForReading(filename);
-
 		// TODO: The below is duplicated from Loadposition
 		data.word(kTimecount) = 0;
 		clearchanges();
 
-		ds = cs;
-		dx = kFileheader;
-		cx = kHeaderlen;
-		savefileread();
-		es = cs;
-		di = kFiledata;
-		ax = savegameId;
+		openforload(savegameId);
+
+		engine->readFromSaveFile(cs.ptr(kFileheader, kHeaderlen), kHeaderlen);
+
+		// read segment lengths from savegame file header
+		int len[6];
+		for (int i = 0; i < 6; ++i)
+			len[i] = cs.word(kFiledata + 2*i);
+		if (len[0] != 17)
+			::error("Error loading save: description buffer isn't 17 bytes");
+
 		if (savegameId < 7) {
-			cx = 17;
-			_mul(cx);
-			ds = data;
-			dx = kSavenames;
-			_add(dx, ax);
-			loadseg();
+			engine->readFromSaveFile(data.ptr(kSavenames + 17*savegameId, len[0]), len[0]);
 		} else {
-			// For potential support of more than 7 savegame slots,
+			// For support of more than 7 savegame slots,
 			// loading into the savenames buffer isn't always possible
-			// Emulate a loadseg call:
 			uint8 namebuf[17];
-			engine->readFromFile(namebuf, 17);
-			_add(di, 2);
+			engine->readFromSaveFile(namebuf, 17);
 		}
-		ds = data; 
-		dx = kStartvars;
-		loadseg();
-		ds = data.word(kExtras);
-		dx = kExframedata;
-		loadseg();
-		ds = data.word(kBuffers);
-		dx = kListofchanges;
-		loadseg();
-		ds = data;
-		dx = kMadeuproomdat;
-		loadseg();
-		ds = cs;
-		dx = kReelroutines;
-		loadseg();
+		engine->readFromSaveFile(data.ptr(kStartvars, len[1]), len[1]);
+		engine->readFromSaveFile(segRef(data.word(kExtras)).ptr(kExframedata, len[2]), len[2]);
+		engine->readFromSaveFile(segRef(data.word(kBuffers)).ptr(kListofchanges, len[3]), len[3]);
+		engine->readFromSaveFile(data.ptr(kMadeuproomdat, len[4]), len[4]);
+		engine->readFromSaveFile(cs.ptr(kReelroutines, len[5]), len[5]);
+
 		closefile();
 		data.byte(kGetback) = 1;
 	}
@@ -418,50 +400,29 @@ void DreamGenContext::saveposition() {
 }
 
 void DreamGenContext::loadposition() {
-	STACK_CHECK;
 	data.word(kTimecount) = 0;
 	clearchanges();
-	al = data.byte(kCurrentslot);
-	ah = 0;
-	push(ax);
-	cx = 13;
-	_mul(cx);
-	dx = data;
-	ds = dx;
-	dx = 8698;
-	_add(dx, ax);
-	openfilefromc();
-	ds = cs;
-	dx = 6091;
-	cx = (6187-6091);
-	savefileread();
-	es = cs;
-	di = 6141;
-	ax = pop();
-	cx = 17;
-	_mul(cx);
-	dx = data;
-	ds = dx;
-	dx = 8579;
-	_add(dx, ax);
-	loadseg();
-	dx = data;
-	ds = dx;
-	dx = 0;
-	loadseg();
-	ds = data.word(kExtras);
-	dx = (0);
-	loadseg();
-	ds = data.word(kBuffers);
-	dx = (0+(228*13)+32+60+(32*32)+(11*10*3)+768+768+768+(32*32)+(128*5)+(80*5)+(100*5)+(12*5)+(46*40)+(5*80));
-	loadseg();
-	dx = data;
-	ds = dx;
-	dx = 7979;
-	loadseg();
-	ds = cs;
-	dx = 534;
-	loadseg();
+
+	unsigned int slot = data.byte(kCurrentslot);
+
+	openforload(slot);
+
+	engine->readFromSaveFile(cs.ptr(kFileheader, kHeaderlen), kHeaderlen);
+
+	// read segment lengths from savegame file header
+	int len[6];
+	for (int i = 0; i < 6; ++i)
+		len[i] = cs.word(kFiledata + 2*i);
+	if (len[0] != 17)
+		::error("Error loading save: description buffer isn't 17 bytes");
+
+	engine->readFromSaveFile(data.ptr(kSavenames + 17*slot, len[0]), len[0]);
+	engine->readFromSaveFile(data.ptr(kStartvars, len[1]), len[1]);
+	engine->readFromSaveFile(segRef(data.word(kExtras)).ptr(kExframedata, len[2]), len[2]);
+	engine->readFromSaveFile(segRef(data.word(kBuffers)).ptr(kListofchanges, len[3]), len[3]);
+	engine->readFromSaveFile(data.ptr(kMadeuproomdat, len[4]), len[4]);
+	engine->readFromSaveFile(cs.ptr(kReelroutines, len[5]), len[5]);
+
 	closefile();
 }
 
diff --git a/engines/dreamweb/stubs.cpp b/engines/dreamweb/stubs.cpp
index a41d5c1dc169..01a76573308e 100644
--- a/engines/dreamweb/stubs.cpp
+++ b/engines/dreamweb/stubs.cpp
@@ -413,6 +413,14 @@ void DreamGenContext::openforsave(unsigned int slot) {
 	engine->openSaveFileForWriting(filename);
 }
 
+void DreamGenContext::openforload(unsigned int slot) {
+	//Common::String filename = ConfMan.getActiveDomainName() + Common::String::format(".d%02d", savegameId);
+	Common::String filename = Common::String::format("DREAMWEB.D%02d", slot);
+	debug(1, "openforload(%s)", filename.c_str());
+	engine->openSaveFileForReading(filename);
+}
+
+
 void DreamGenContext::openfilenocheck() {
 	const char *name = (const char *)ds.ptr(dx, 13);
 	debug(1, "checksavefile(%s)", name);
diff --git a/engines/dreamweb/stubs.h b/engines/dreamweb/stubs.h
index f86b0e0f5b29..24233b42fd75 100644
--- a/engines/dreamweb/stubs.h
+++ b/engines/dreamweb/stubs.h
@@ -348,4 +348,5 @@
 	void loadposition();
 	void saveposition();
 	void openforsave(unsigned int slot);
+	void openforload(unsigned int slot);