[Scummvm-cvs-logs] SF.net SVN: scummvm:[36291] scummvm/trunk/backends/platform/ds/arm9/source/ gbampsave.cpp

[fingolfin at users.sourceforge.net]

Revision: 36291
          http://scummvm.svn.sourceforge.net/scummvm/?rev=36291&view=rev
Author:   fingolfin
Date:     2009-02-13 05:10:36 +0000 (Fri, 13 Feb 2009)

Log Message:
-----------
DS: Removed code from GBAMPSaveFileManager::getSavePath() which was for compatibility with 0.6.1 config files (NDS port 0.1 was based on 0.7.1); added FIXME comment

Modified Paths:
--------------
    scummvm/trunk/backends/platform/ds/arm9/source/gbampsave.cpp

Modified: scummvm/trunk/backends/platform/ds/arm9/source/gbampsave.cpp
===================================================================
--- scummvm/trunk/backends/platform/ds/arm9/source/gbampsave.cpp	2009-02-13 05:09:46 UTC (rev 36290)
+++ scummvm/trunk/backends/platform/ds/arm9/source/gbampsave.cpp	2009-02-13 05:10:36 UTC (rev 36291)
@@ -169,21 +169,12 @@
 
 // This method copied from an old version of the savefile.cpp, since it's been removed from there and
 // placed in default-saves.cpp, where I cannot call it.
+// FIXME: Does it even make sense to change the "savepath" on the NDS? Considering
+// that nothing sets a default value for the "savepath" either, wouldn't it better
+// to return a fixed path here?
 const char *GBAMPSaveFileManager::getSavePath() const {
-	const char *dir = NULL;
-
 	// Try to use game specific savepath from config
-	dir = ConfMan.get("savepath").c_str();
-
-	// Work around a bug (#999122) in the original 0.6.1 release of
-	// ScummVM, which would insert a bad savepath value into config files.
-	if (0 == strcmp(dir, "None")) {
-		ConfMan.removeKey("savepath", ConfMan.getActiveDomainName());
-		ConfMan.flushToDisk();
-		dir = ConfMan.get("savepath").c_str();
-	}
-
-
+	const char *dir = ConfMan.get("savepath").c_str();
 	assert(dir);
 
 	return dir;


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