From b4a2268bd1be0d5fb56b39eb301a1ee7fbf0caa5 Mon Sep 17 00:00:00 2001 From: stephena Date: Mon, 6 Mar 2006 02:26:16 +0000 Subject: [PATCH] First pass at conditional saving of the settings file. Basically, the settings will only be saved when they haven't changed from the defaults. This is probably of limited use on ordinary systems, but is helpful for smaller systems using flash-RAM as storage (WinCE/GP2X), since multiple saves wears out the hardware quicker. This breaks the OSX port; I'll fix it in a day or two. I'm hoping this will resolve the issues of certain settings not being saved in OSX as well. git-svn-id: svn://svn.code.sf.net/p/stella/code/trunk@1017 8b62c5a3-ac7e-4cc8-8f21-d9a121418aba --- stella/src/cheat/CheatManager.cxx | 4 +- stella/src/common/SoundSDL.cxx | 4 +- stella/src/emucore/Settings.cxx | 433 +++++++++++++++++---------- stella/src/emucore/Settings.hxx | 80 ++--- stella/src/gui/LauncherDialog.cxx | 4 +- stella/src/macosx/SettingsMACOSX.cxx | 38 +-- stella/src/macosx/SettingsMACOSX.hxx | 13 +- stella/src/unix/SettingsUNIX.cxx | 6 +- 8 files changed, 352 insertions(+), 230 deletions(-) diff --git a/stella/src/cheat/CheatManager.cxx b/stella/src/cheat/CheatManager.cxx index 28d00c51a..f4ee8e398 100644 --- a/stella/src/cheat/CheatManager.cxx +++ b/stella/src/cheat/CheatManager.cxx @@ -13,7 +13,7 @@ // See the file "license" for information on usage and redistribution of // this file, and for a DISCLAIMER OF ALL WARRANTIES. // -// $Id: CheatManager.cxx,v 1.7 2006-01-11 01:17:08 stephena Exp $ +// $Id: CheatManager.cxx,v 1.8 2006-03-06 02:26:15 stephena Exp $ //============================================================================ #include @@ -288,7 +288,7 @@ void CheatManager::loadCheats(const string& md5sum) // (and remove the key from the settings, so they won't get set again) string cheats = myOSystem->settings().getString("cheat"); if(cheats != "") - myOSystem->settings().setString("cheat", "", false); + myOSystem->settings().setString("cheat", ""); CheatCodeMap::iterator iter = myCheatMap.find(md5sum); if(iter == myCheatMap.end() && cheats == "") diff --git a/stella/src/common/SoundSDL.cxx b/stella/src/common/SoundSDL.cxx index e7bb025c5..5f3f28222 100644 --- a/stella/src/common/SoundSDL.cxx +++ b/stella/src/common/SoundSDL.cxx @@ -13,7 +13,7 @@ // See the file "license" for information on usage and redistribution of // this file, and for a DISCLAIMER OF ALL WARRANTIES. // -// $Id: SoundSDL.cxx,v 1.28 2006-01-06 00:31:55 stephena Exp $ +// $Id: SoundSDL.cxx,v 1.29 2006-03-06 02:26:16 stephena Exp $ //============================================================================ #ifdef SOUND_SUPPORT @@ -272,7 +272,7 @@ void SoundSDL::setChannels(uInt32 channels) if(channels == 1 || channels == 2) { myNumChannels = channels; - myOSystem->settings().setInt("channels", myNumChannels, false); + myOSystem->settings().setInt("channels", myNumChannels); } } diff --git a/stella/src/emucore/Settings.cxx b/stella/src/emucore/Settings.cxx index c9fed30ba..ad6ed1296 100644 --- a/stella/src/emucore/Settings.cxx +++ b/stella/src/emucore/Settings.cxx @@ -13,7 +13,7 @@ // See the file "license" for information on usage and redistribution of // this file, and for a DISCLAIMER OF ALL WARRANTIES. // -// $Id: Settings.cxx,v 1.78 2006-03-05 01:18:42 stephena Exp $ +// $Id: Settings.cxx,v 1.79 2006-03-06 02:26:16 stephena Exp $ //============================================================================ #include @@ -32,64 +32,59 @@ Settings::Settings(OSystem* osystem) // Add this settings object to the OSystem myOSystem->attach(this); - // First create the settings array - myCapacity = 40; - mySettings = new Setting[myCapacity]; - mySize = 0; + // Add options that are common to all versions of Stella + setInternal("video", "soft"); + setInternal("dirtyrects", "true"); + setInternal("ppblend", "77"); - // Now fill it with options that are common to all versions of Stella - set("video", "soft"); - set("dirtyrects", "true"); - set("ppblend", "77"); + setInternal("gl_filter", "nearest"); + setInternal("gl_aspect", "2.0"); + setInternal("gl_fsmax", "false"); + setInternal("gl_lib", ""); - set("gl_filter", "nearest"); - set("gl_aspect", "2.0"); - set("gl_fsmax", "false"); - set("gl_lib", ""); + setInternal("zoom", "2"); + setInternal("fullscreen", "false"); + setInternal("center", "true"); + setInternal("grabmouse", "false"); + setInternal("palette", "standard"); + setInternal("debugheight", "0"); - set("zoom", "2"); - set("fullscreen", "false"); - set("center", "true"); - set("grabmouse", "false"); - set("palette", "standard"); - set("debugheight", "0"); + setInternal("sound", "true"); + setInternal("fragsize", "512"); + setInternal("freq", "31400"); + setInternal("tiafreq", "31400"); + setInternal("volume", "100"); + setInternal("clipvol", "true"); - set("sound", "true"); - set("fragsize", "512"); - set("freq", "31400"); - set("tiafreq", "31400"); - set("volume", "100"); - set("clipvol", "true"); + setInternal("keymap", ""); + setInternal("joymap", ""); + setInternal("joyaxismap", ""); + setInternal("joyhatmap", ""); + setInternal("paddle", "0"); + setInternal("sa1", "left"); + setInternal("sa2", "right"); + setInternal("joymouse", "false"); + setInternal("p1speed", "50"); + setInternal("p2speed", "50"); + setInternal("p3speed", "50"); + setInternal("p4speed", "50"); - set("keymap", ""); - set("joymap", ""); - set("joyaxismap", ""); - set("joyhatmap", ""); - set("paddle", "0"); - set("sa1", "left"); - set("sa2", "right"); - set("joymouse", "false"); - set("p1speed", "50"); - set("p2speed", "50"); - set("p3speed", "50"); - set("p4speed", "50"); + setInternal("showinfo", "false"); - set("showinfo", "false"); + setInternal("ssdir", ""); + setInternal("ssname", "romname"); + setInternal("sssingle", "false"); - set("ssdir", ""); - set("ssname", "romname"); - set("sssingle", "false"); - - set("romdir", ""); - set("lastrom", ""); - set("modtime", ""); // romdir last modification time + setInternal("romdir", ""); + setInternal("lastrom", ""); + setInternal("modtime", ""); // romdir last modification time } // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - Settings::~Settings() { - // Free the settings array - delete[] mySettings; + myInternalSettings.clear(); + myExternalSettings.clear(); } // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - @@ -135,8 +130,8 @@ void Settings::loadConfig() continue; // Only settings which have been previously set are valid - if(contains(key)) - set(key, value); + if(int idx = getInternalPos(key) != -1) + setInternal(key, value, idx, true); } in.close(); @@ -163,27 +158,27 @@ bool Settings::loadCommandLine(int argc, char** argv) } else if(key == "listrominfo") { - set(key, "true", false); // this confusing line means set 'listrominfo' - return true; // to true, but don't save to the settings file + setExternal(key, "true"); + return true; } else if(key == "debug") // this doesn't make Stella exit { - set(key, "true", false); // don't save this to the config file either + setExternal(key, "true"); return true; } else if(key == "holdreset") // this doesn't make Stella exit { - set(key, "true", false); // don't save this to the config file either + setExternal(key, "true"); return true; } else if(key == "holdselect") // this doesn't make Stella exit { - set(key, "true", false); // don't save this to the config file either + setExternal(key, "true"); return true; } else if(key == "holdbutton0") // this doesn't make Stella exit { - set(key, "true", false); // don't save this to the config file either + setExternal(key, "true"); return true; } @@ -196,7 +191,10 @@ bool Settings::loadCommandLine(int argc, char** argv) // Settings read from the commandline must not be saved to // the rc-file, unless they were previously set - set(key, value, false); + if(int idx = getInternalPos(key) != -1) + setInternal(key, value, idx, true); + else + setExternal(key, value); } return true; @@ -210,57 +208,57 @@ void Settings::validate() s = getString("video"); if(s != "soft" && s != "gl") - set("video", "soft"); + setInternal("video", "soft"); #ifdef DISPLAY_OPENGL s = getString("gl_filter"); if(s != "linear" && s != "nearest") - set("gl_filter", "nearest"); + setInternal("gl_filter", "nearest"); float f = getFloat("gl_aspect"); if(f < 1.1 || f > 2.0) - set("gl_aspect", "2.0"); + setInternal("gl_aspect", "2.0"); #endif #ifdef SOUND_SUPPORT i = getInt("fragsize"); if(i != 256 && i != 512 && i != 1024 && i != 2048 && i != 4096) #ifdef WIN32 - set("fragsize", "2048"); + setInternal("fragsize", "2048"); #else - set("fragsize", "512"); + setInternal("fragsize", "512"); #endif i = getInt("volume"); if(i < 0 || i > 100) - set("volume", "100"); + setInternal("volume", "100"); i = getInt("freq"); if(i < 0 || i > 48000) - set("freq", "31400"); + setInternal("freq", "31400"); i = getInt("tiafreq"); if(i < 0 || i > 48000) - set("tiafreq", "31400"); + setInternal("tiafreq", "31400"); #endif i = getInt("zoom"); if(i < 1 || i > 6) - set("zoom", "2"); + setInternal("zoom", "2"); i = getInt("paddle"); if(i < 0 || i > 3) - set("paddle", "0"); + setInternal("paddle", "0"); s = getString("palette"); if(s != "standard" && s != "original" && s != "z26") - set("palette", "standard"); + setInternal("palette", "standard"); i = getInt("ppblend"); - if(i < 0) set("ppblend", "0"); - if(i > 100) set("ppblend", "100"); + if(i < 0) setInternal("ppblend", "0"); + if(i > 100) setInternal("ppblend", "100"); s = getString("romname"); if(s != "romname" && s != "md5sum") - set("ssname", "romname"); + setInternal("ssname", "romname"); } // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - @@ -365,6 +363,22 @@ void Settings::usage() // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - void Settings::saveConfig() { + + // Do a quick scan of the internal settings to see if any have + // changed. If not, we don't need to save them at all. + bool settingsChanged = false; + for(unsigned int i = 0; i < myInternalSettings.size(); ++i) + { + if(myInternalSettings[i].value != myInternalSettings[i].initialValue) + { + settingsChanged = true; + break; + } + } + + if(!settingsChanged) + return; + ofstream out(myOSystem->configOutputFilename().c_str()); if(!out || !out.is_open()) { @@ -388,153 +402,254 @@ void Settings::saveConfig() << ";" << endl; // Write out each of the key and value pairs - for(unsigned int i = 0; i < mySize; ++i) - if(mySettings[i].save) - out << mySettings[i].key << " = " << mySettings[i].value << endl; + for(unsigned int i = 0; i < myInternalSettings.size(); ++i) + { + out << myInternalSettings[i].key << " = " << + myInternalSettings[i].value << endl; + } out.close(); } // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -void Settings::set(const string& key, const string& value, bool save) -{ - // See if the setting already exists - for(unsigned int i = 0; i < mySize; ++i) - { - // If a key is already present in the array, then we assume - // that it was set by the emulation core and must be saved - // to the rc-file. - if(key == mySettings[i].key) - { - mySettings[i].value = value; - mySettings[i].save = true; - return; - } - } - - // See if the array needs to be resized - if(mySize == myCapacity) - { - // Yes, so we'll make the array twice as large - Setting* newSettings = new Setting[myCapacity * 2]; - - for(unsigned int i = 0; i < mySize; ++i) - { - newSettings[i] = mySettings[i]; - } - - delete[] mySettings; - - mySettings = newSettings; - myCapacity *= 2; - } - - // Add new property to the array - mySettings[mySize].key = key; - mySettings[mySize].value = value; - mySettings[mySize].save = save; - - ++mySize; -} - -// - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -void Settings::setInt(const string& key, const int value, bool save) +void Settings::setInt(const string& key, const int value) { ostringstream stream; stream << value; - set(key, stream.str(), save); + if(int idx = getInternalPos(key) != -1) + setInternal(key, stream.str(), idx); + else + setExternal(key, stream.str()); } // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -void Settings::setFloat(const string& key, const float value, bool save) +void Settings::setFloat(const string& key, const float value) { ostringstream stream; stream << value; - set(key, stream.str(), save); + if(int idx = getInternalPos(key) != -1) + setInternal(key, stream.str(), idx); + else + setExternal(key, stream.str()); } // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -void Settings::setBool(const string& key, const bool value, bool save) +void Settings::setBool(const string& key, const bool value) { ostringstream stream; stream << value; - set(key, stream.str(), save); + if(int idx = getInternalPos(key) != -1) + setInternal(key, stream.str(), idx); + else + setExternal(key, stream.str()); } // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -void Settings::setString(const string& key, const string& value, bool save) +void Settings::setString(const string& key, const string& value) { - ostringstream stream; - stream << value; - - set(key, stream.str(), save); + if(int idx = getInternalPos(key) != -1) + setInternal(key, value, idx); + else + setExternal(key, value); } // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - int Settings::getInt(const string& key) const { // Try to find the named setting and answer its value - for(unsigned int i = 0; i < mySize; ++i) - if(key == mySettings[i].key) - return (int) atoi(mySettings[i].value.c_str()); - - return 0; + int idx = -1; + if((idx = getInternalPos(key)) != -1) + return (int) atoi(myInternalSettings[idx].value.c_str()); + else if((idx = getExternalPos(key)) != -1) + return (int) atoi(myExternalSettings[idx].value.c_str()); + else + return 0; } // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - float Settings::getFloat(const string& key) const { // Try to find the named setting and answer its value - for(unsigned int i = 0; i < mySize; ++i) - if(key == mySettings[i].key) - return (float) atof(mySettings[i].value.c_str()); - - return -1.0; + int idx = -1; + if((idx = getInternalPos(key)) != -1) + return (float) atof(myInternalSettings[idx].value.c_str()); + else if((idx = getExternalPos(key)) != -1) + return (float) atof(myExternalSettings[idx].value.c_str()); + else + return -1.0; } // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - bool Settings::getBool(const string& key) const { // Try to find the named setting and answer its value - for(unsigned int i = 0; i < mySize; ++i) + int idx = -1; + if((idx = getInternalPos(key)) != -1) { - if(key == mySettings[i].key) - { - if(mySettings[i].value == "1" || mySettings[i].value == "true") - return true; - else if(mySettings[i].value == "0" || mySettings[i].value == "false") - return false; - else - return false; - } + if(myInternalSettings[idx].value == "1" || + myInternalSettings[idx].value == "true") + return true; + else if(myInternalSettings[idx].value == "0" || + myInternalSettings[idx].value == "false") + return false; + else + return false; } - - return false; + else if((idx = getExternalPos(key)) != -1) + { + if(myInternalSettings[idx].value == "1" || + myInternalSettings[idx].value == "true") + return true; + else if(myInternalSettings[idx].value == "0" || + myInternalSettings[idx].value == "false") + return false; + else + return false; + } + else + return false; } // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - const string& Settings::getString(const string& key) const { // Try to find the named setting and answer its value - for(unsigned int i = 0; i < mySize; ++i) - if(key == mySettings[i].key) - return mySettings[i].value; - - return EmptyString; + int idx = -1; + if((idx = getInternalPos(key)) != -1) + return myInternalSettings[idx].value; + else if((idx = getExternalPos(key)) != -1) + return myExternalSettings[idx].value; + else + return EmptyString; } // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -bool Settings::contains(const string& key) +int Settings::getInternalPos(const string& key) const { - // Try to find the named setting - for(unsigned int i = 0; i < mySize; ++i) - if(key == mySettings[i].key) - return true; + for(unsigned int i = 0; i < myInternalSettings.size(); ++i) + if(myInternalSettings[i].key == key) + return i; - return false; + return -1; +} + +// - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - +int Settings::getExternalPos(const string& key) const +{ + for(unsigned int i = 0; i < myExternalSettings.size(); ++i) + if(myExternalSettings[i].key == key) + return i; + + return -1; +} + +// - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - +int Settings::setInternal(const string& key, const string& value, + int pos, bool useAsInitial) +{ + int idx = -1; + + if(pos != -1 && pos >= 0 && pos < (int)myInternalSettings.size() && + myInternalSettings[pos].key == key) + { + idx = pos; + } + else + { + for(unsigned int i = 0; i < myInternalSettings.size(); ++i) + { + if(myInternalSettings[i].key == key) + { + idx = i; + break; + } + } + } + + if(idx != -1) + { + myInternalSettings[idx].key = key; + myInternalSettings[idx].value = value; + if(useAsInitial) myInternalSettings[idx].initialValue = value; + + /*cerr << "modify internal: key = " << key + << ", value = " << value + << " @ index = " << idx + << endl;*/ + } + else + { + Setting setting; + setting.key = key; + setting.value = value; + if(useAsInitial) setting.initialValue = value; + + myInternalSettings.push_back(setting); + idx = myInternalSettings.size() - 1; + + /*cerr << "insert internal: key = " << key + << ", value = " << value + << " @ index = " << idx + << endl;*/ + } + + return idx; +} + +// - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - +int Settings::setExternal(const string& key, const string& value, + int pos, bool useAsInitial) +{ + int idx = -1; + + if(pos != -1 && pos >= 0 && pos < (int)myExternalSettings.size() && + myExternalSettings[pos].key == key) + { + idx = pos; + } + else + { + for(unsigned int i = 0; i < myExternalSettings.size(); ++i) + { + if(myExternalSettings[i].key == key) + { + idx = i; + break; + } + } + } + + if(idx != -1) + { + myExternalSettings[idx].key = key; + myExternalSettings[idx].value = value; + if(useAsInitial) myExternalSettings[idx].initialValue = value; + + /*cerr << "modify external: key = " << key + << ", value = " << value + << " @ index = " << idx + << endl;*/ + } + else + { + Setting setting; + setting.key = key; + setting.value = value; + if(useAsInitial) setting.initialValue = value; + + myExternalSettings.push_back(setting); + idx = myExternalSettings.size() - 1; + + /*cerr << "insert external: key = " << key + << ", value = " << value + << " @ index = " << idx + << endl;*/ + } + + return idx; } // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - diff --git a/stella/src/emucore/Settings.hxx b/stella/src/emucore/Settings.hxx index cd1adcb62..a84aff23a 100644 --- a/stella/src/emucore/Settings.hxx +++ b/stella/src/emucore/Settings.hxx @@ -13,7 +13,7 @@ // See the file "license" for information on usage and redistribution of // this file, and for a DISCLAIMER OF ALL WARRANTIES. // -// $Id: Settings.hxx,v 1.26 2005-11-19 22:26:13 stephena Exp $ +// $Id: Settings.hxx,v 1.27 2006-03-06 02:26:16 stephena Exp $ //============================================================================ #ifndef SETTINGS_HXX @@ -21,6 +21,7 @@ class OSystem; +#include "Array.hxx" #include "bspf.hxx" @@ -28,7 +29,7 @@ class OSystem; This class provides an interface for accessing frontend specific settings. @author Stephen Anthony - @version $Id: Settings.hxx,v 1.26 2005-11-19 22:26:13 stephena Exp $ + @version $Id: Settings.hxx,v 1.27 2006-03-06 02:26:16 stephena Exp $ */ class Settings { @@ -113,59 +114,32 @@ class Settings @param key The key of the setting @param value The value to assign to the setting - @param save Whether this setting should be saved to the rc-file. */ - void setInt(const string& key, const int value, bool save = true); + void setInt(const string& key, const int value); /** Set the value associated with key to the given value. @param key The key of the setting @param value The value to assign to the setting - @param save Whether this setting should be saved to the rc-file. */ - void setFloat(const string& key, const float value, bool save = true); + void setFloat(const string& key, const float value); /** Set the value associated with key to the given value. @param key The key of the setting @param value The value to assign to the setting - @param save Whether this setting should be saved to the rc-file. */ - void setBool(const string& key, const bool value, bool save = true); + void setBool(const string& key, const bool value); /** Set the value associated with key to the given value. @param key The key of the setting @param value The value to assign to the setting - @param save Whether this setting should be saved to the rc-file. */ - void setString(const string& key, const string& value, bool save = true); - - protected: - void set(const string& key, const string& value, bool save = true); - - // The parent OSystem object - OSystem* myOSystem; - - // Structure used for storing settings - struct Setting - { - string key; - string value; - bool save; - }; - - // Pointer to a dynamically allocated array of settings - Setting* mySettings; - - // Size of the settings array (i.e. the number of pairs) - unsigned int mySize; - - // Test whether the given setting is present in the array - bool contains(const string& key); + void setString(const string& key, const string& value); private: // Copy constructor isn't supported by this class so make it private @@ -175,15 +149,49 @@ class Settings Settings& operator = (const Settings&); // Trim leading and following whitespace from a string - string trim(string& str) + static string trim(string& str) { string::size_type first = str.find_first_not_of(' '); return (first == string::npos) ? string() : str.substr(first, str.find_last_not_of(' ')-first+1); } - // Current capacity of the settings array - unsigned int myCapacity; + protected: + // The parent OSystem object + OSystem* myOSystem; + + // Structure used for storing settings + struct Setting + { + string key; + string value; + string initialValue; + }; + typedef Common::Array SettingsArray; + + const SettingsArray& getInternalSettings() const + { return myInternalSettings; } + const SettingsArray& getExternalSettings() const + { return myExternalSettings; } + + /** Get position in specified array of 'key' */ + int getInternalPos(const string& key) const; + int getExternalPos(const string& key) const; + + /** Add key,value pair to specified array at specified position */ + int setInternal(const string& key, const string& value, + int pos = -1, bool useAsInitial = false); + int setExternal(const string& key, const string& value, + int pos = -1, bool useAsInitial = false); + + private: + // Holds key,value pairs that are necessary for Stella to + // function and must be saved on each program exit. + SettingsArray myInternalSettings; + + // Holds auxiliary key,value pairs that shouldn't be saved on + // program exit. + SettingsArray myExternalSettings; }; #endif diff --git a/stella/src/gui/LauncherDialog.cxx b/stella/src/gui/LauncherDialog.cxx index 67c2df8df..e0272060e 100644 --- a/stella/src/gui/LauncherDialog.cxx +++ b/stella/src/gui/LauncherDialog.cxx @@ -13,7 +13,7 @@ // See the file "license" for information on usage and redistribution of // this file, and for a DISCLAIMER OF ALL WARRANTIES. // -// $Id: LauncherDialog.cxx,v 1.40 2006-03-05 01:18:42 stephena Exp $ +// $Id: LauncherDialog.cxx,v 1.41 2006-03-06 02:26:16 stephena Exp $ // // Based on code from ScummVM - Scumm Interpreter // Copyright (C) 2002-2004 The ScummVM project @@ -54,7 +54,7 @@ LauncherDialog::LauncherDialog(OSystem* osystem, DialogContainer* parent, mySelectedItem(0) { const GUI::Font& font = instance()->launcherFont(); -// const GUI::Font& font = instance()->font(); + const int fontHeight = font.getFontHeight(); const int bwidth = (_w - 2 * 10 - 8 * (4 - 1)) / 4; const int bheight = font.getLineHeight() + 4; diff --git a/stella/src/macosx/SettingsMACOSX.cxx b/stella/src/macosx/SettingsMACOSX.cxx index ae8c59d0c..8996c7037 100644 --- a/stella/src/macosx/SettingsMACOSX.cxx +++ b/stella/src/macosx/SettingsMACOSX.cxx @@ -13,7 +13,7 @@ // See the file "license" for information on usage and redistribution of // this file, and for a DISCLAIMER OF ALL WARRANTIES. // -// $Id: SettingsMACOSX.cxx,v 1.8 2006-01-15 23:43:40 markgrebe Exp $ +// $Id: SettingsMACOSX.cxx,v 1.9 2006-03-06 02:26:16 stephena Exp $ //============================================================================ #include @@ -32,17 +32,17 @@ #include "SettingsMACOSX.hxx" extern "C" { -void prefsSetString(char *key, char *value); -void prefsGetString(char *key, char *value); -void prefsSave(void); + void prefsSetString(char *key, char *value); + void prefsGetString(char *key, char *value); + void prefsSave(void); } // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - SettingsMACOSX::SettingsMACOSX(OSystem* osystem) : Settings(osystem) { - set("video", "gl"); // Use opengl mode by default - set("gl_lib", "libGL.so"); + setInternal("video", "gl"); // Use opengl mode by default + setInternal("gl_lib", "libGL.so"); } // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - @@ -58,11 +58,14 @@ void SettingsMACOSX::loadConfig() // Write out each of the key and value pairs for(uInt32 i = 0; i < mySize; ++i) - { - prefsGetString((char *) mySettings[i].key.c_str(),cvalue); - if (cvalue[0] != 0) - mySettings[i].value.assign(cvalue); - } + { + prefsGetString((char *) mySettings[i].key.c_str(), cvalue); + if(cvalue[0] != 0) + { + mySettings[i].value.assign(cvalue); + mySettings[i].save = true; + } + } } // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - @@ -70,11 +73,12 @@ void SettingsMACOSX::saveConfig() { // Write out each of the key and value pairs for(uInt32 i = 0; i < mySize; ++i) + { if(mySettings[i].save) - { - prefsSetString((char *) mySettings[i].key.c_str(), - (char *) mySettings[i].value.c_str()); - } - prefsSave(); + { + prefsSetString((char *) mySettings[i].key.c_str(), + (char *) mySettings[i].value.c_str()); + } + } + prefsSave(); } - diff --git a/stella/src/macosx/SettingsMACOSX.hxx b/stella/src/macosx/SettingsMACOSX.hxx index 6dc25fa02..8c03b2d41 100644 --- a/stella/src/macosx/SettingsMACOSX.hxx +++ b/stella/src/macosx/SettingsMACOSX.hxx @@ -13,22 +13,19 @@ // See the file "license" for information on usage and redistribution of // this file, and for a DISCLAIMER OF ALL WARRANTIES. // -// $Id: SettingsMACOSX.hxx,v 1.3 2005-06-16 01:11:29 stephena Exp $ +// $Id: SettingsMACOSX.hxx,v 1.4 2006-03-06 02:26:16 stephena Exp $ //============================================================================ #ifndef SETTINGS_MAC_OSX_HXX #define SETTINGS_MAX_OSX_HXX -#include "bspf.hxx" - -class Console; - +class OSystem; /** This class defines Macintosh OSX system specific settings. @author Mark Grebe - @version $Id: SettingsMACOSX.hxx,v 1.3 2005-06-16 01:11:29 stephena Exp $ + @version $Id: SettingsMACOSX.hxx,v 1.4 2006-03-06 02:26:16 stephena Exp $ */ class SettingsMACOSX : public Settings { @@ -55,9 +52,7 @@ class SettingsMACOSX : public Settings This method should be called to save the current settings to the standard Mac preferences. */ - - void saveConfig(); - + void saveConfig(); }; #endif diff --git a/stella/src/unix/SettingsUNIX.cxx b/stella/src/unix/SettingsUNIX.cxx index 34ddda953..1980034ea 100644 --- a/stella/src/unix/SettingsUNIX.cxx +++ b/stella/src/unix/SettingsUNIX.cxx @@ -13,7 +13,7 @@ // See the file "license" for information on usage and redistribution of // this file, and for a DISCLAIMER OF ALL WARRANTIES. // -// $Id: SettingsUNIX.cxx,v 1.16 2006-01-14 21:36:29 stephena Exp $ +// $Id: SettingsUNIX.cxx,v 1.17 2006-03-06 02:26:16 stephena Exp $ //============================================================================ #include "bspf.hxx" @@ -25,8 +25,8 @@ SettingsUNIX::SettingsUNIX(OSystem* osystem) : Settings(osystem) { // This argument is only valid for Linux/UNIX, and will eventually be removed - set("accurate", "false"); - set("gl_lib", "libGL.so"); + setInternal("accurate", "false"); + setInternal("gl_lib", "libGL.so"); } // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -