[Scummvm-cvs-logs] SF.net SVN: scummvm:[50730] scummvm/trunk/engines/sci/engine/savegame.cpp

fingolfin at users.sourceforge.net fingolfin at users.sourceforge.net
Wed Jul 7 01:24:37 CEST 2010


Revision: 50730
          http://scummvm.svn.sourceforge.net/scummvm/?rev=50730&view=rev
Author:   fingolfin
Date:     2010-07-06 23:24:36 +0000 (Tue, 06 Jul 2010)

Log Message:
-----------
SCI: Add version to obsolete string

This way, if we ever get rid of support for savegames with
version <= 23, we can also get rid of this syncString()
call. Without this change, it would have "infected" every
new savegame version, forever.

Modified Paths:
--------------
    scummvm/trunk/engines/sci/engine/savegame.cpp

Modified: scummvm/trunk/engines/sci/engine/savegame.cpp
===================================================================
--- scummvm/trunk/engines/sci/engine/savegame.cpp	2010-07-06 22:29:45 UTC (rev 50729)
+++ scummvm/trunk/engines/sci/engine/savegame.cpp	2010-07-06 23:24:36 UTC (rev 50730)
@@ -286,7 +286,7 @@
 	s.skip(4, VER(9), VER(9));	// OBSOLETE: Used to be savegame_version
 
 	Common::String tmp;
-	s.syncString(tmp);			// OBSOLETE: Used to be game_version
+	s.syncString(tmp, VER(9), VER(23));			// OBSOLETE: Used to be game_version
 	s.skip(4, VER(9), VER(9));	// OBSOLETE: Used to be version
 
 	// OBSOLETE: Saved menus. Skip all of the saved data
@@ -746,7 +746,7 @@
 				if (!baseObj)
 					error("Clone entry without a base class: %d", j);
 			}	// end for
-			}	// end if
+		}	// end if
 	}	// end for
 }
 


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.




More information about the Scummvm-git-logs mailing list