[Scummvm-cvs-logs] SF.net SVN: scummvm:[46070] scummvm/trunk/engines/draci/game.cpp

[spalek at users.sourceforge.net]

Revision: 46070
          http://scummvm.svn.sourceforge.net/scummvm/?rev=46070&view=rev
Author:   spalek
Date:     2009-11-22 09:21:40 +0000 (Sun, 22 Nov 2009)

Log Message:
-----------
Updated an urgent TODO to make the game playable

Modified Paths:
--------------
    scummvm/trunk/engines/draci/game.cpp

Modified: scummvm/trunk/engines/draci/game.cpp
===================================================================
--- scummvm/trunk/engines/draci/game.cpp	2009-11-22 09:17:33 UTC (rev 46069)
+++ scummvm/trunk/engines/draci/game.cpp	2009-11-22 09:21:40 UTC (rev 46070)
@@ -1207,13 +1207,16 @@
 	// animations instead of listing what to deallocate.  maybe simply
 	// deallocate everything; reloading isn't that expensive.
 	//
+	// URGENT TODO: if a game item's location changes (e.g., due to a GPL2
+	// command), its animations survive and we get assert in
+	// AnimationManager::load() next time.  we must address this before a
+	// proper clean-up to make game playable.  happens when loading game
+	// before getting the fairy tale book, playing a bit, and then doing it
+	// again.  the fairy-tale book's animations raise an assert.
+	//
 	// TODO: completely rewrite the resource management.  maybe implement
 	// usage counters?  maybe completely ignore the GPL2 hints and manage
 	// memory completely on my own?
-	//
-	// URGENT TODO: if a game item is in the hero's hands when changing
-	// locations, its animations survive and we get assert in
-	// AnimationManager::load().
 	GameObject *dragon = &_objects[kDragonObject];
 	for (uint i = dragon->_anim.size() - 1; i >= kFirstTemporaryAnimation; --i) {
 		dragon->_anim.back()->del();


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