[Scummvm-cvs-logs] SF.net SVN: scummvm:[46868] scummvm/trunk/engines/saga/script.h

sev at users.sourceforge.net sev at users.sourceforge.net
Sat Jan 2 01:22:46 CET 2010


Revision: 46868
          http://scummvm.svn.sourceforge.net/scummvm/?rev=46868&view=rev
Author:   sev
Date:     2010-01-02 00:22:46 +0000 (Sat, 02 Jan 2010)

Log Message:
-----------
Removing now redundant comment. The fix was correct.

Modified Paths:
--------------
    scummvm/trunk/engines/saga/script.h

Modified: scummvm/trunk/engines/saga/script.h
===================================================================
--- scummvm/trunk/engines/saga/script.h	2010-01-02 00:07:20 UTC (rev 46867)
+++ scummvm/trunk/engines/saga/script.h	2010-01-02 00:22:46 UTC (rev 46868)
@@ -333,15 +333,6 @@
 	}
 	void setNoPendingVerb() {
 		_pendingVerb = getVerbType(kVerbNone);
-		// TODO: Someone with knowledge of SAGA should review this.
-		// This initially looked like:
-		// _currentObject[0] = _currentObject[0] = ID_NOTHING;
-		// and thus was an undefined operation on _currentObject[0]
-		// according to the C(++) standard.
-		// Now the question is whether this should really reset
-		// both objets or just _currentObject[0].
-		// In case the current code is ok, just remove the TODO,
-		// otherwise fix the code and remove the TODO.
 		_currentObject[0] = _currentObject[1] = ID_NOTHING;
 		setPointerVerb();
 	}


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