From 01581b24c0a77eba38442ed880ee069b3cd5699a Mon Sep 17 00:00:00 2001
From: Max Horn
Date: Fri, 13 Feb 2009 05:10:36 +0000
Subject: 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

svn-id: r36291
---
 backends/platform/ds/arm9/source/gbampsave.cpp | 17 ++++-------------
 1 file changed, 4 insertions(+), 13 deletions(-)

(limited to 'backends/platform')

diff --git a/backends/platform/ds/arm9/source/gbampsave.cpp b/backends/platform/ds/arm9/source/gbampsave.cpp
index 8814fc4957..385935e816 100644
--- a/backends/platform/ds/arm9/source/gbampsave.cpp
+++ b/backends/platform/ds/arm9/source/gbampsave.cpp
@@ -169,21 +169,12 @@ GBAMPSaveFile* GBAMPSaveFileManager::openSavefile(char const* name, bool saveOrL
 
 // 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;
-- 
cgit v1.2.3