Changeset 6432 for code/branches/gamestate/src/libraries/core/input/KeyBinder.cc

Timestamp:

Dec 29, 2009, 10:30:19 PM (16 years ago)

Author:

rgrieder

Message:

Changed the way config values associated with general settings (ConfigFileType::Settings) are handled:

• ConfigFileManager only handles config files listed in the ConfigFileType enum (normal enum again)
• ConfigFileManager only takes care of ConfigFiles and returns a pointer to the right one, just two functions left. —> use like: ConfigFileManager::getInstance().getConfigFile(myType)→doSomething();
• Moved all code (except for the argument completion functions) relating to ConfigFileType::Settings to a new class: SettingsConfigFile, which is a Singleton (it doesn't make sense to have multiple instances unless you start coding a lot more)
• SettingsConfigFile handles config value containers according to their section and entry in the ini file, not according to class and variables names. (In most cases it will be class and variable names though)
• SettingsConfigFile supports:
  • clear() (removes any file entries not associated to a config value container)
  • updateConfigValues() (does exactly that through the identifier)
  • config, tconfig and getConfig
  • commands listed above are exported to tolua, and tconfig, config and getConfig were given shortcuts in Lua (e.g. orxonox.config)
• If you need to organise ConfigFiles yourself, just do it without the ConfigFileManager, like the KeyBinder does.
• All getValue() functions have been split into getOrCreateValue() and getValue(), which is const
• Removed obsolete config value management code in the Identifier (it still stores and destroys them and provides access to them)

All of that leads to one HUGE advantage:
"config OutputHandler softDebugLevelInGameConsole"
works now (any further implications are up to the reader…)
(it didn't work before because the actual config value container is in the InGameConsole singleton)

File:

• code/branches/gamestate/src/libraries/core/input/KeyBinder.cc (modified) (4 diffs)

code/branches/gamestate/src/libraries/core/input/KeyBinder.cc

@@ -49 +49 @@
         : deriveTime_(0.0f)
         , filename_(filename)
+        , configFile_(NULL)
     {
         mouseRelative_[0] = 0;
@@ -93 +94 @@
             mouseAxes_[i].groupName_ = "MouseAxes";
         }
-
-        // We might not even load any bindings at all (KeyDetector for instance)
-        this->configFile_ = ConfigFileType::NoType;
 
         // initialise joy sticks separatly to allow for reloading
@@ -163 +161 @@
 
         // load the bindings if required
-        if (configFile_ != ConfigFileType::NoType)
+        if (configFile_ != NULL)
         {
             for (unsigned int iDev = oldValue; iDev < joySticks_.size(); ++iDev)
@@ -249 +247 @@
         COUT(3) << "KeyBinder: Loading key bindings..." << std::endl;
 
-        // Get a new ConfigFileType from the ConfigFileManager
-        this->configFile_ = ConfigFileManager::getInstance().getNewConfigFileType();
-
-        ConfigFileManager::getInstance().setFilename(this->configFile_, this->filename_);
+        this->configFile_ = new ConfigFile(this->filename_);
+        this->configFile_->load();
 
         // Parse bindings and create the ConfigValueContainers if necessary