From 5bb449b2bf443c30dd9fa312a426c81d40baf9dc Mon Sep 17 00:00:00 2001
From: Paul Gilbert
Date: Mon, 1 Jun 2009 08:46:39 +0000
Subject: Removed the remaining two FIXME's - my analysis of the data indicates
 no issues with structure packing, and all multi-byte data accesses are
 wrapped using loadShort/saveShort, which is endian safe

svn-id: r41096
---
 engines/cruise/saveload.cpp | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/engines/cruise/saveload.cpp b/engines/cruise/saveload.cpp
index 4230829bcf..0084b10cb9 100644
--- a/engines/cruise/saveload.cpp
+++ b/engines/cruise/saveload.cpp
@@ -258,7 +258,6 @@ static void syncOverlays2(Common::Serializer &s) {
 			// save BSS
 			s.syncAsSint16LE(ovlData->sizeOfData4);
 			if (ovlData->sizeOfData4)
-				// FIXME: Endian and structure packing problems for this data pointer
 				s.syncBytes(ovlData->data4Ptr, ovlData->sizeOfData4);
 
 			// save variables
@@ -344,8 +343,6 @@ void syncScript(Common::Serializer &s, scriptInstanceStruct *entry) {
 		s.syncAsSint16LE(ptr->varA);
 
 		if (ptr->varA) {
-			// FIXME: This code is not endian safe, and breaks if struct
-			// packing changes. Read/write the members one by one instead.
 			if (s.isLoading())
 				ptr->var6 = (byte *)mallocAndZero(ptr->varA);
 			s.syncBytes(ptr->var6, ptr->varA);
-- 
cgit v1.2.3