Factor out save/load code from MainMenuDialog::handleCommand into new methods

svn-id: r48563

2 files changed, 47 insertions, 41 deletions

diff --git a/engines/dialogs.cpp b/engines/dialogs.cpp
index 4a2f0f9f5b..cd42c2a23d 100644
--- a/engines/dialogs.cpp
+++ b/engines/dialogs.cpp
@@ -125,49 +125,10 @@ void MainMenuDialog::handleCommand(CommandSender *sender, uint32 cmd, uint32 dat
 		close();
 		break;
 	case kLoadCmd:
-		{
-		Common::String gameId = ConfMan.get("gameid");
-
-		const EnginePlugin *plugin = 0;
-		EngineMan.findGame(gameId, &plugin);
-
-		int slot = _loadDialog->runModal(plugin, ConfMan.getActiveDomainName());
-
-		if (slot >= 0) {
-			// FIXME: For now we just ignore the return
-			// value, which is quite bad since it could
-			// be a fatal loading error, which renders
-			// the engine unusable.
-			_engine->loadGameState(slot);
-			close();
-		}
-
-		}
+		load();
 		break;
 	case kSaveCmd:
-		{
-		Common::String gameId = ConfMan.get("gameid");
-
-		const EnginePlugin *plugin = 0;
-		EngineMan.findGame(gameId, &plugin);
-
-		int slot = _saveDialog->runModal(plugin, ConfMan.getActiveDomainName());
-
-		if (slot >= 0) {
-			Common::String result(_saveDialog->getResultString());
-			if (result.empty()) {
-				// If the user was lazy and entered no save name, come up with a default name.
-				char buf[20];
-				snprintf(buf, 20, "Save %d", slot + 1);
-				_engine->saveGameState(slot, buf);
-			} else {
-				_engine->saveGameState(slot, result.c_str());
-			}
-
-			close();
-		}
-
-		}
+		save();
 		break;
 	case kOptionsCmd:
 		_optionsDialog->runModal();
@@ -232,6 +193,47 @@ void MainMenuDialog::reflowLayout() {
 	Dialog::reflowLayout();
 }
 
+void MainMenuDialog::save() {
+	Common::String gameId = ConfMan.get("gameid");
+
+	const EnginePlugin *plugin = 0;
+	EngineMan.findGame(gameId, &plugin);
+
+	int slot = _saveDialog->runModal(plugin, ConfMan.getActiveDomainName());
+
+	if (slot >= 0) {
+		Common::String result(_saveDialog->getResultString());
+		if (result.empty()) {
+			// If the user was lazy and entered no save name, come up with a default name.
+			char buf[20];
+			snprintf(buf, 20, "Save %d", slot + 1);
+			_engine->saveGameState(slot, buf);
+		} else {
+			_engine->saveGameState(slot, result.c_str());
+		}
+
+		close();
+	}
+}
+
+void MainMenuDialog::load() {
+	Common::String gameId = ConfMan.get("gameid");
+
+	const EnginePlugin *plugin = 0;
+	EngineMan.findGame(gameId, &plugin);
+
+	int slot = _loadDialog->runModal(plugin, ConfMan.getActiveDomainName());
+
+	if (slot >= 0) {
+		// FIXME: For now we just ignore the return
+		// value, which is quite bad since it could
+		// be a fatal loading error, which renders
+		// the engine unusable.
+		_engine->loadGameState(slot);
+		close();
+	}
+}
+
 enum {
 	kOKCmd = 'ok  '
 };
diff --git a/engines/dialogs.h b/engines/dialogs.h
index bd2f9dce84..6bee7c5fb1 100644
--- a/engines/dialogs.h
+++ b/engines/dialogs.h
@@ -47,6 +47,10 @@ public:
 	virtual void reflowLayout();
 
 protected:
+	void save();
+	void load();
+
+protected:
 	Engine			*_engine;
 
 	GUI::GraphicsWidget *_logo;