[Scummvm-cvs-logs] scummvm master -> b39b15485bbf4e53a248e04423ecd93f990d4740

m-kiewitz m_kiewitz at users.sourceforge.net
Fri May 15 05:09:48 CEST 2015


This automated email contains information about 1 new commit which have been
pushed to the 'scummvm' repo located at https://github.com/scummvm/scummvm .

Summary:
b39b15485b AGI: mouse support cleanup


Commit: b39b15485bbf4e53a248e04423ecd93f990d4740
    https://github.com/scummvm/scummvm/commit/b39b15485bbf4e53a248e04423ecd93f990d4740
Author: Martin Kiewitz (m_kiewitz at users.sourceforge.net)
Date: 2015-05-15T05:10:08+02:00

Commit Message:
AGI: mouse support cleanup

Changed paths:
    engines/agi/agi.cpp
    engines/agi/detection.cpp



diff --git a/engines/agi/agi.cpp b/engines/agi/agi.cpp
index 93a00ec..85a8e17 100644
--- a/engines/agi/agi.cpp
+++ b/engines/agi/agi.cpp
@@ -508,7 +508,7 @@ AgiBase::AgiBase(OSystem *syst, const AGIGameDescription *gameDesc) : Engine(sys
 	// Assign default values to the config manager, in case settings are missing
 	ConfMan.registerDefault("originalsaveload", "false");
 	ConfMan.registerDefault("altamigapalette", "false");
-	ConfMan.registerDefault("enablemouse", "true");
+	ConfMan.registerDefault("mousesupport", "true");
 
 	_noSaveLoadAllowed = false;
 
@@ -564,9 +564,8 @@ AgiEngine::AgiEngine(OSystem *syst, const AGIGameDescription *gameDesc) : AgiBas
 	memset(&_mouse, 0, sizeof(struct Mouse));
 
 	_game.mouseEnabled = true;
-	if (!ConfMan.getBool("enablemouse")) {
-		// we actually do disable mouse instead of enabling it in case the option is set
-		//  that's because we do not show the option at all for all Amiga games and certain fanmade games
+	if (!ConfMan.getBool("mousesupport")) {
+		// we effectively disable the mouse for games, that explicitly do not want mouse support to be enabled
 		_game.mouseEnabled = false;
 	}
 
diff --git a/engines/agi/detection.cpp b/engines/agi/detection.cpp
index a44053d..823ec7b 100644
--- a/engines/agi/detection.cpp
+++ b/engines/agi/detection.cpp
@@ -138,27 +138,6 @@ static const PlainGameDescriptor agiGames[] = {
 	{0, 0}
 };
 
-static const ExtraGuiOption agiExtraGuiOption = {
-	_s("Use original save/load screens"),
-	_s("Use the original save/load screens, instead of the ScummVM ones"),
-	"originalsaveload",
-	false
-};
-
-static const ExtraGuiOption agiExtraGuiOptionAmiga = {
-	_s("Use an alternative palette"),
-	_s("Use an alternative palette, common for all Amiga games. This was the old behavior"),
-	"altamigapalette",
-	false
-};
-
-static const ExtraGuiOption agiExtraGuiOptionEnableMouse = {
-	_s("Enable mouse"),
-	_s("Enables mouse. Games, that require a mouse, will have mouse always enabled."),
-	"enablemouse",
-	true
-};
-
 #include "agi/detection_tables.h"
 
 static const ADExtraGuiOptionsMap optionsList[] = {
@@ -185,9 +164,9 @@ static const ADExtraGuiOptionsMap optionsList[] = {
 	{
 		GAMEOPTION_DISABLE_MOUSE,
 		{
-			_s("Enable mouse"),
-			_s("Enables mouse. Games, that require a mouse, will have mouse always enabled."),
-			"enablemouse",
+			_s("Mouse support"),
+			_s("Enables mouse support. Allows to use mouse for movement and in game menus."),
+			"mousesupport",
 			true
 		}
 	},
@@ -216,7 +195,6 @@ public:
 
 	virtual bool hasFeature(MetaEngineFeature f) const;
 	virtual bool createInstance(OSystem *syst, Engine **engine, const ADGameDescription *desc) const;
-	//virtual const ExtraGuiOptions getExtraGuiOptions(const Common::String &target) const;
 	virtual SaveStateList listSaves(const char *target) const;
 	virtual int getMaximumSaveSlot() const;
 	virtual void removeSaveState(const char *target, int slot) const;
@@ -275,19 +253,6 @@ bool AgiMetaEngine::createInstance(OSystem *syst, Engine **engine, const ADGameD
 	return res;
 }
 
-// TODO: original 2 gui engine options are not shown for games, that were detected before our change
-//        because gui-options from the detection-table are saved internally.
-//        I also can't set those 2 options manually in case they were not set by optionsList[]
-#if 0
-const ExtraGuiOptions AgiMetaEngine::getExtraGuiOptions(const Common::String &target) const {
-	ExtraGuiOptions options;
-	options.push_back(agiExtraGuiOption);
-	if (target.contains("-amiga"))
-		options.push_back(agiExtraGuiOptionAmiga);
-	return options;
-}
-#endif
-
 SaveStateList AgiMetaEngine::listSaves(const char *target) const {
 	const uint32 AGIflag = MKTAG('A','G','I',':');
 	Common::SaveFileManager *saveFileMan = g_system->getSavefileManager();






More information about the Scummvm-git-logs mailing list