As far as I can tell, there's no longer any reason for PartBuffer and AnimData

to be packed, so I've removed the packing and added the 'refresh' field to
AnimData instead of having it as a separate array.

On the other hand, animHeaderStruct and animHeader2Struct *should* have been
packed, but weren't. I've changed they way they are initialised so that the
packing should no longer matter for them either.

svn-id: r22651

1 files changed, 1 insertions, 8 deletions

diff --git a/engines/cine/various.cpp b/engines/cine/various.cpp
index 4ea8f8712f..a36dd7b229 100644
--- a/engines/cine/various.cpp
+++ b/engines/cine/various.cpp
@@ -54,13 +54,6 @@ uint16 var5;
 
 int16 buildObjectListCommand(void);
 
-// TODO: This could/should be a field in the AnimData struct, but as long as it
-// is marked as packed I don't dare add anything to it without knowing more
-// about how it's used. (It's quite possible that it no longer needs to be
-// packed.)
-
-bool refreshAnimData[NUM_MAX_ANIMDATA];
-
 void drawString(const char *string, byte param) {
 }
 
@@ -760,7 +753,7 @@ int16 makeLoad(char *saveName) {
 		animDataTable[i].fileIdx = fHandle->readSint16BE();
 		animDataTable[i].frameIdx = fHandle->readSint16BE();
 		fHandle->read(animDataTable[i].name, 10);
-		refreshAnimData[i] = (fHandle->readByte() != 0);
+		animDataTable[i].refresh = (fHandle->readByte() != 0);
 	}
 
 	// TODO: handle screen params (really required ?)