Merge pull request #2688 from ace20022/cleanup_settings
authorSascha Montellese <sascha.montellese@gmail.com>
Wed, 3 Jul 2013 20:31:33 +0000 (13:31 -0700)
committerSascha Montellese <sascha.montellese@gmail.com>
Wed, 3 Jul 2013 20:31:33 +0000 (13:31 -0700)
[Settings] Clean up things found by Cppcheck

26 files changed:
xbmc/settings/AdvancedSettings.cpp
xbmc/settings/AdvancedSettings.h
xbmc/settings/DisplaySettings.cpp
xbmc/settings/DisplaySettings.h
xbmc/settings/MediaSettings.cpp
xbmc/settings/MediaSettings.h
xbmc/settings/MediaSourceSettings.cpp
xbmc/settings/MediaSourceSettings.h
xbmc/settings/Setting.cpp
xbmc/settings/Setting.h
xbmc/settings/SettingConditions.cpp
xbmc/settings/SettingConditions.h
xbmc/settings/SettingControl.cpp
xbmc/settings/SettingDependency.h
xbmc/settings/SettingSection.cpp
xbmc/settings/Settings.cpp
xbmc/settings/SettingsManager.cpp
xbmc/settings/SettingsManager.h
xbmc/settings/SkinSettings.cpp
xbmc/settings/SkinSettings.h
xbmc/settings/dialogs/GUIDialogContentSettings.cpp
xbmc/settings/windows/GUIControlSettings.cpp
xbmc/settings/windows/GUIControlSettings.h
xbmc/settings/windows/GUIWindowSettingsCategory.cpp
xbmc/settings/windows/GUIWindowSettingsScreenCalibration.cpp
xbmc/settings/windows/GUIWindowTestPattern.cpp

index 983d764..a128fa8 100644 (file)
@@ -376,7 +376,6 @@ void CAdvancedSettings::Initialize()
   m_logEnableAirtunes = false;
   m_airTunesPort = 36666;
   m_airPlayPort = 36667;
-  m_initialized = true;
 
   m_databaseMusic.Reset();
   m_databaseVideo.Reset();
@@ -402,6 +401,8 @@ void CAdvancedSettings::Initialize()
   #endif
 
   m_userAgent = g_sysinfo.GetUserAgent();
+
+  m_initialized = true;
 }
 
 bool CAdvancedSettings::Load()
index dc0c854..67bfbb6 100644 (file)
@@ -300,7 +300,6 @@ class CAdvancedSettings : public ISettingCallback, public ISettingsHandler
     int m_iEdlCommBreakAutowait;    // seconds
     int m_iEdlCommBreakAutowind;    // seconds
 
-    bool m_bFirstLoop;
     int m_curlconnecttimeout;
     int m_curllowspeedtime;
     int m_curlretries;
index 7b77337..49b50f4 100644 (file)
@@ -525,7 +525,7 @@ void CDisplaySettings::SettingOptionsRefreshRatesFiller(const CSetting *setting,
   vector<REFRESHRATE> refreshrates = g_Windowing.RefreshRates(resInfo.iScreen, resInfo.iScreenWidth, resInfo.iScreenHeight, resInfo.dwFlags);
 
   bool match = false;
-  for (vector<REFRESHRATE>::const_iterator refreshrate = refreshrates.begin(); refreshrate != refreshrates.end(); refreshrate++)
+  for (vector<REFRESHRATE>::const_iterator refreshrate = refreshrates.begin(); refreshrate != refreshrates.end(); ++refreshrate)
   {
     std::string screenmode = GetStringFromResolution((RESOLUTION)refreshrate->ResInfo_Index, refreshrate->RefreshRate);
     if (!match && StringUtils::EqualsNoCase(((CSettingString*)setting)->GetValue(), screenmode))
@@ -550,7 +550,7 @@ void CDisplaySettings::SettingOptionsResolutionsFiller(const CSetting *setting,
   {
     std::map<RESOLUTION, RESOLUTION_INFO> resolutionInfos;
     vector<RESOLUTION_WHR> resolutions = g_Windowing.ScreenResolutions(info.iScreen, info.fRefreshRate);
-    for (vector<RESOLUTION_WHR>::const_iterator resolution = resolutions.begin(); resolution != resolutions.end(); resolution++)
+    for (vector<RESOLUTION_WHR>::const_iterator resolution = resolutions.begin(); resolution != resolutions.end(); ++resolution)
     {
       list.push_back(make_pair(
         StringUtils::Format("%dx%d%s", resolution->width, resolution->height,
index 446e6a0..056fc07 100644 (file)
@@ -97,7 +97,7 @@ public:
 protected:
   CDisplaySettings();
   CDisplaySettings(const CDisplaySettings&);
-  CDisplaySettings const& operator=(CDisplaySettings const&);
+  CDisplaySettings& operator=(CDisplaySettings const&);
   virtual ~CDisplaySettings();
 
   DisplayMode GetCurrentDisplayMode() const;
index ae659a2..2195f8b 100644 (file)
@@ -56,7 +56,7 @@ CMediaSettings::CMediaSettings()
   m_additionalSubtitleDirectoryChecked = 0;
 
   m_musicNeedsUpdate = 0;
-  m_musicNeedsUpdate = 0;
+  m_videoNeedsUpdate = 0;
 }
 
 CMediaSettings::~CMediaSettings()
index b2dad89..5e460d7 100644 (file)
@@ -96,7 +96,7 @@ public:
 protected:
   CMediaSettings();
   CMediaSettings(const CMediaSettings&);
-  CMediaSettings const& operator=(CMediaSettings const&);
+  CMediaSettings& operator=(CMediaSettings const&);
   virtual ~CMediaSettings();
 
   static std::string GetWatchedContent(const std::string &content);
index f9335ea..ce0a1ee 100644 (file)
@@ -187,7 +187,7 @@ void CMediaSourceSettings::SetDefaultSource(const std::string &type, const std::
 }
 
 // NOTE: This function does NOT save the sources.xml file - you need to call SaveSources() separately.
-bool CMediaSourceSettings::UpdateSource(const std::string &strType, const std::string strOldName, const std::string &strUpdateChild, const std::string &strUpdateValue)
+bool CMediaSourceSettings::UpdateSource(const std::string &strType, const std::string &strOldName, const std::string &strUpdateChild, const std::string &strUpdateValue)
 {
   VECSOURCES *pShares = GetSources(strType);
   if (pShares == NULL)
@@ -223,7 +223,7 @@ bool CMediaSourceSettings::UpdateSource(const std::string &strType, const std::s
   return false;
 }
 
-bool CMediaSourceSettings::DeleteSource(const std::string &strType, const std::string &strName, const std::string strPath, bool virtualSource /* = false */)
+bool CMediaSourceSettings::DeleteSource(const std::string &strType, const std::string &strName, const std::string &strPath, bool virtualSource /* = false */)
 {
   VECSOURCES *pShares = GetSources(strType);
   if (pShares == NULL)
@@ -331,10 +331,7 @@ bool CMediaSourceSettings::GetSource(const std::string &category, const TiXmlNod
       {
         // translate special tags
         if (!strPath.empty() && strPath.at(0) == '$')
-        {
-          string strPathOld(strPath);
           strPath = CUtil::TranslateSpecialSource(strPath);
-        }
 
         // need to check path validity again as CUtil::TranslateSpecialSource() may have failed
         if (!strPath.empty())
@@ -366,7 +363,7 @@ bool CMediaSourceSettings::GetSource(const std::string &category, const TiXmlNod
   else
   {
     // validate the paths
-    for (vector<string>::const_iterator path = vecPaths.begin(); path != vecPaths.end(); path++)
+    for (vector<string>::const_iterator path = vecPaths.begin(); path != vecPaths.end(); ++path)
     {
       CURL url(*path);
       string protocol = url.GetProtocol();
index 1119d54..8f1bf78 100644 (file)
@@ -46,15 +46,15 @@ public:
   const std::string& GetDefaultSource(const std::string &type) const;
   void SetDefaultSource(const std::string &type, const std::string &source);
 
-  bool UpdateSource(const std::string &strType, const std::string strOldName, const std::string &strUpdateChild, const std::string &strUpdateValue);
-  bool DeleteSource(const std::string &strType, const std::string &strName, const std::string strPath, bool virtualSource = false);
+  bool UpdateSource(const std::string &strType, const std::string &strOldName, const std::string &strUpdateChild, const std::string &strUpdateValue);
+  bool DeleteSource(const std::string &strType, const std::string &strName, const std::string &strPath, bool virtualSource = false);
   bool AddShare(const std::string &type, const CMediaSource &share);
   bool UpdateShare(const std::string &type, const std::string &oldName, const CMediaSource &share);
 
 protected:
   CMediaSourceSettings();
   CMediaSourceSettings(const CMediaSourceSettings&);
-  CMediaSourceSettings const& operator=(CMediaSourceSettings const&);
+  CMediaSourceSettings& operator=(CMediaSourceSettings const&);
   virtual ~CMediaSourceSettings();
 
 private:
index 9bfeb0d..a620092 100644 (file)
@@ -541,7 +541,7 @@ bool CSettingInt::CheckValidity(int value) const
   {
     //if the setting is an std::map, check if we got a valid value before assigning it
     bool ok = false;
-    for (StaticIntegerSettingOptions::const_iterator it = m_options.begin(); it != m_options.end(); it++)
+    for (StaticIntegerSettingOptions::const_iterator it = m_options.begin(); it != m_options.end(); ++it)
     {
       if (it->second == value)
       {
@@ -664,7 +664,7 @@ void CSettingInt::copy(const CSettingInt &setting)
   m_optionsFiller = setting.m_optionsFiller;
 }
 
-bool CSettingInt::fromString(const std::string &strValue, int &value) const
+bool CSettingInt::fromString(const std::string &strValue, int &value)
 {
   if (strValue.empty())
     return false;
@@ -830,7 +830,7 @@ void CSettingNumber::copy(const CSettingNumber &setting)
   m_max = setting.m_max;
 }
 
-bool CSettingNumber::fromString(const std::string &strValue, double &value) const
+bool CSettingNumber::fromString(const std::string &strValue, double &value)
 {
   if (strValue.empty())
     return false;
index 25a0713..0130595 100644 (file)
@@ -204,7 +204,7 @@ public:
 
 private:
   void copy(const CSettingInt &setting);
-  bool fromString(const std::string &strValue, int &value) const;
+  static bool fromString(const std::string &strValue, int &value);
 
   int m_value;
   int m_default;
@@ -253,7 +253,7 @@ public:
 
 private:
   virtual void copy(const CSettingNumber &setting);
-  bool fromString(const std::string &strValue, double &value) const;
+  static bool fromString(const std::string &strValue, double &value);
 
   double m_value;
   double m_default;
index d06aac9..b6daf6d 100644 (file)
@@ -57,7 +57,7 @@ bool CSettingConditionCombination::Check() const
 {
   bool ok = false;
   for (CBooleanLogicOperations::const_iterator operation = m_operations.begin();
-       operation != m_operations.end(); operation++)
+       operation != m_operations.end(); ++operation)
   {
     if (*operation == NULL)
       continue;
@@ -73,7 +73,7 @@ bool CSettingConditionCombination::Check() const
   }
 
   for (CBooleanLogicValues::const_iterator value = m_values.begin();
-       value != m_values.end(); value++)
+       value != m_values.end(); ++value)
   {
     if (*value == NULL)
       continue;
@@ -122,7 +122,7 @@ void CSettingConditionsManager::AddCondition(const std::string &identifier, Sett
   m_conditions.insert(SettingConditionPair(identifier, condition));
 }
 
-bool CSettingConditionsManager::Check(const std::string &condition, const std::string &value /* = "" */, const std::string settingId /* = "" */) const
+bool CSettingConditionsManager::Check(const std::string &condition, const std::string &value /* = "" */, const std::string &settingId /* = "" */) const
 {
   if (condition.empty())
     return false;
index 4b2fe77..1b3327e 100644 (file)
@@ -93,7 +93,7 @@ public:
   void AddCondition(const std::string &condition);
   void AddCondition(const std::string &identifier, SettingConditionCheck condition);
 
-  bool Check(const std::string &condition, const std::string &value = "", const std::string settingId = "") const;
+  bool Check(const std::string &condition, const std::string &value = "", const std::string &settingId = "") const;
 
 private:
   CSettingConditionsManager(const CSettingConditionsManager&);
index 6f297f2..59b67d2 100644 (file)
@@ -120,7 +120,7 @@ bool CSettingControl::setAttributes(const std::string &strAttributes)
   std::vector<std::string> attributeList = StringUtils::Split(strAttributes, ",");
 
   int controlAttributes = SettingControlAttributeNone;
-  for (std::vector<std::string>::const_iterator attribute = attributeList.begin(); attribute != attributeList.end(); attribute++)
+  for (std::vector<std::string>::const_iterator attribute = attributeList.begin(); attribute != attributeList.end(); ++attribute)
   {
     if (StringUtils::EqualsNoCase(*attribute, "hidden"))
       controlAttributes |= (int)SettingControlAttributeHidden;
index fb2e096..0f85ba2 100644 (file)
@@ -49,7 +49,8 @@ class CSettingDependencyCondition : public CSettingConditionItem
 public:
   CSettingDependencyCondition(CSettingsManager *settingsManager = NULL)
     : CSettingConditionItem(settingsManager),
-      m_operator(SettingDependencyOperatorEquals)
+      m_target(SettingDependencyTargetNone),  
+      m_operator(SettingDependencyOperatorEquals)      
   { }
   virtual ~CSettingDependencyCondition() { }
 
index fa8ed07..0e4c00b 100644 (file)
@@ -46,14 +46,14 @@ template<class T> void addISetting(const TiXmlNode *node, const T &item, std::ve
 
   if (positionId != NULL && strlen(positionId) > 0 && position >= 0)
   {
-    for (typename std::vector<T>::iterator it = items.begin(); it != items.end(); it++)
+    for (typename std::vector<T>::iterator it = items.begin(); it != items.end(); ++it)
     {
       if (!StringUtils::EqualsNoCase((*it)->GetId(), positionId))
         continue;
 
       typename std::vector<T>::iterator positionIt = it;
       if (position == 1)
-        positionIt++;
+        ++positionIt;
 
       items.insert(positionIt, item);
       return;
@@ -69,7 +69,7 @@ CSettingGroup::CSettingGroup(const std::string &id, CSettingsManager *settingsMa
 
 CSettingGroup::~CSettingGroup()
 {
-  for (SettingList::const_iterator setting = m_settings.begin(); setting != m_settings.end(); setting++)
+  for (SettingList::const_iterator setting = m_settings.begin(); setting != m_settings.end(); ++setting)
     delete *setting;
   m_settings.clear();
 }
@@ -87,7 +87,7 @@ bool CSettingGroup::Deserialize(const TiXmlNode *node, bool update /* = false */
     if (CSettingCategory::DeserializeIdentification(settingElement, settingId))
     {
       CSetting *setting = NULL;
-      for (SettingList::iterator itSetting = m_settings.begin(); itSetting != m_settings.end(); itSetting++)
+      for (SettingList::iterator itSetting = m_settings.begin(); itSetting != m_settings.end(); ++itSetting)
       {
         if ((*itSetting)->GetId() == settingId)
         {
@@ -133,7 +133,7 @@ SettingList CSettingGroup::GetSettings(SettingLevel level) const
 {
   SettingList settings;
 
-  for (SettingList::const_iterator it = m_settings.begin(); it != m_settings.end(); it++)
+  for (SettingList::const_iterator it = m_settings.begin(); it != m_settings.end(); ++it)
   {
     if ((*it)->GetLevel() <= level && (*it)->IsVisible())
       settings.push_back(*it);
@@ -150,7 +150,7 @@ CSettingCategory::CSettingCategory(const std::string &id, CSettingsManager *sett
 
 CSettingCategory::~CSettingCategory()
 {
-  for (SettingGroupList::const_iterator it = m_groups.begin(); it != m_groups.end(); it++)
+  for (SettingGroupList::const_iterator it = m_groups.begin(); it != m_groups.end(); ++it)
     delete *it;
 
   m_groups.clear();
@@ -183,7 +183,7 @@ bool CSettingCategory::Deserialize(const TiXmlNode *node, bool update /* = false
     if (CSettingGroup::DeserializeIdentification(groupNode, groupId))
     {
       CSettingGroup *group = NULL;
-      for (SettingGroupList::iterator itGroup = m_groups.begin(); itGroup != m_groups.end(); itGroup++)
+      for (SettingGroupList::iterator itGroup = m_groups.begin(); itGroup != m_groups.end(); ++itGroup)
       {
         if ((*itGroup)->GetId() == groupId)
         {
@@ -219,7 +219,7 @@ SettingGroupList CSettingCategory::GetGroups(SettingLevel level) const
 {
   SettingGroupList groups;
 
-  for (SettingGroupList::const_iterator it = m_groups.begin(); it != m_groups.end(); it++)
+  for (SettingGroupList::const_iterator it = m_groups.begin(); it != m_groups.end(); ++it)
   {
     if ((*it)->IsVisible() && (*it)->GetSettings(level).size() > 0)
       groups.push_back(*it);
@@ -240,7 +240,7 @@ CSettingSection::CSettingSection(const std::string &id, CSettingsManager *settin
 
 CSettingSection::~CSettingSection()
 {
-  for (SettingCategoryList::const_iterator it = m_categories.begin(); it != m_categories.end(); it++)
+  for (SettingCategoryList::const_iterator it = m_categories.begin(); it != m_categories.end(); ++it)
     delete *it;
 
   m_categories.clear();
@@ -269,7 +269,7 @@ bool CSettingSection::Deserialize(const TiXmlNode *node, bool update /* = false
     if (CSettingCategory::DeserializeIdentification(categoryNode, categoryId))
     {
       CSettingCategory *category = NULL;
-      for (SettingCategoryList::iterator itCategory = m_categories.begin(); itCategory != m_categories.end(); itCategory++)
+      for (SettingCategoryList::iterator itCategory = m_categories.begin(); itCategory != m_categories.end(); ++itCategory)
       {
         if ((*itCategory)->GetId() == categoryId)
         {
@@ -305,7 +305,7 @@ SettingCategoryList CSettingSection::GetCategories(SettingLevel level) const
 {
   SettingCategoryList categories;
 
-  for (SettingCategoryList::const_iterator it = m_categories.begin(); it != m_categories.end(); it++)
+  for (SettingCategoryList::const_iterator it = m_categories.begin(); it != m_categories.end(); ++it)
   {
     if ((*it)->IsVisible() && (*it)->GetGroups(level).size() > 0)
       categories.push_back(*it);
index 3c8ff5e..7c15049 100644 (file)
@@ -302,7 +302,7 @@ bool CSettings::Load(const TiXmlElement *root, bool hide /* = false */)
   // if necessary hide all the loaded settings
   if (success && hide && loadedSettings != NULL)
   {
-    for(std::map<std::string, CSetting*>::const_iterator setting = loadedSettings->begin(); setting != loadedSettings->end(); setting++)
+    for(std::map<std::string, CSetting*>::const_iterator setting = loadedSettings->begin(); setting != loadedSettings->end(); ++setting)
       setting->second->SetVisible(false);
 
     delete loadedSettings;
index 5763db7..963cc9e 100644 (file)
@@ -78,13 +78,13 @@ bool CSettingsManager::Initialize(const TiXmlElement *root)
           m_sections[section->GetId()] = section;
 
         // get all settings and add them to the settings map
-        for (SettingCategoryList::const_iterator categoryIt = section->GetCategories().begin(); categoryIt != section->GetCategories().end(); categoryIt++)
+        for (SettingCategoryList::const_iterator categoryIt = section->GetCategories().begin(); categoryIt != section->GetCategories().end(); ++categoryIt)
         {
           (*categoryIt)->CheckVisible();
-          for (SettingGroupList::const_iterator groupIt = (*categoryIt)->GetGroups().begin(); groupIt != (*categoryIt)->GetGroups().end(); groupIt++)
+          for (SettingGroupList::const_iterator groupIt = (*categoryIt)->GetGroups().begin(); groupIt != (*categoryIt)->GetGroups().end(); ++groupIt)
           {
             (*groupIt)->CheckVisible();
-            for (SettingList::const_iterator settingIt = (*groupIt)->GetSettings().begin(); settingIt != (*groupIt)->GetSettings().end(); settingIt++)
+            for (SettingList::const_iterator settingIt = (*groupIt)->GetSettings().begin(); settingIt != (*groupIt)->GetSettings().end(); ++settingIt)
             {
               (*settingIt)->CheckVisible();
 
@@ -117,16 +117,16 @@ bool CSettingsManager::Initialize(const TiXmlElement *root)
     sectionNode = sectionNode->NextSibling(XML_SECTION);
   }
 
-  for (SettingMap::iterator itSettingDep = m_settings.begin(); itSettingDep != m_settings.end(); itSettingDep++)
+  for (SettingMap::iterator itSettingDep = m_settings.begin(); itSettingDep != m_settings.end(); ++itSettingDep)
   {
     if (itSettingDep->second.setting == NULL)
       continue;
 
     const SettingDependencies& deps = itSettingDep->second.setting->GetDependencies();
-    for (SettingDependencies::const_iterator depIt = deps.begin(); depIt != deps.end(); depIt++)
+    for (SettingDependencies::const_iterator depIt = deps.begin(); depIt != deps.end(); ++depIt)
     {
       std::set<std::string> settingIds = depIt->GetSettings();
-      for (std::set<std::string>::const_iterator itSettingId = settingIds.begin(); itSettingId != settingIds.end(); itSettingId++)
+      for (std::set<std::string>::const_iterator itSettingId = settingIds.begin(); itSettingId != settingIds.end(); ++itSettingId)
       {
         SettingMap::iterator setting = m_settings.find(*itSettingId);
         if (setting == m_settings.end())
@@ -193,7 +193,7 @@ bool CSettingsManager::Save(TiXmlNode *root) const
   }
 
   // save any ISubSettings implementations
-  for (std::set<ISubSettings*>::const_iterator it = m_subSettings.begin(); it != m_subSettings.end(); it++)
+  for (std::set<ISubSettings*>::const_iterator it = m_subSettings.begin(); it != m_subSettings.end(); ++it)
   {
     if (!(*it)->Save(root))
       return false;
@@ -214,7 +214,7 @@ void CSettingsManager::Unload()
   // OnSettingChanging() and OnSettingChanged()
   m_loaded = false;
 
-  for (SettingMap::iterator setting = m_settings.begin(); setting != m_settings.end(); setting++)
+  for (SettingMap::iterator setting = m_settings.begin(); setting != m_settings.end(); ++setting)
     setting->second.setting->Reset();
 
   OnSettingsUnloaded();
@@ -226,13 +226,13 @@ void CSettingsManager::Clear()
   Unload();
 
   m_settings.clear();
-  for (SettingSectionMap::iterator section = m_sections.begin(); section != m_sections.end(); section++)
+  for (SettingSectionMap::iterator section = m_sections.begin(); section != m_sections.end(); ++section)
     delete section->second;
   m_sections.clear();
 
   OnSettingsCleared();
 
-  for (std::set<ISubSettings*>::const_iterator it = m_subSettings.begin(); it != m_subSettings.end(); it++)
+  for (std::set<ISubSettings*>::const_iterator it = m_subSettings.begin(); it != m_subSettings.end(); ++it)
     (*it)->Clear();
 
   m_initialized = false;
@@ -272,7 +272,7 @@ void CSettingsManager::RegisterCallback(ISettingCallback *callback, const std::s
   if (callback == NULL)
     return;
 
-  for (std::set<std::string>::const_iterator settingIt = settingList.begin(); settingIt != settingList.end(); settingIt++)
+  for (std::set<std::string>::const_iterator settingIt = settingList.begin(); settingIt != settingList.end(); ++settingIt)
   {
     std::string id = *settingIt;
     StringUtils::ToLower(id);
@@ -295,7 +295,7 @@ void CSettingsManager::RegisterCallback(ISettingCallback *callback, const std::s
 void CSettingsManager::UnregisterCallback(ISettingCallback *callback)
 {
   CSingleLock lock(m_critical);
-  for (SettingMap::iterator settingIt = m_settings.begin(); settingIt != m_settings.end(); settingIt++)
+  for (SettingMap::iterator settingIt = m_settings.begin(); settingIt != m_settings.end(); ++settingIt)
     settingIt->second.callbacks.erase(callback);
 }
 
@@ -586,7 +586,7 @@ bool CSettingsManager::Serialize(TiXmlNode *parent) const
   if (parent == NULL)
     return false;
 
-  for (SettingMap::const_iterator it = m_settings.begin(); it != m_settings.end(); it++)
+  for (SettingMap::const_iterator it = m_settings.begin(); it != m_settings.end(); ++it)
   {
     if (it->second.setting->GetType() == SettingTypeAction)
       continue;
@@ -631,7 +631,7 @@ bool CSettingsManager::Deserialize(const TiXmlNode *node, std::map<std::string,
   if (node == NULL)
     return false;
 
-  for (SettingMap::iterator it = m_settings.begin(); it != m_settings.end(); it++)
+  for (SettingMap::iterator it = m_settings.begin(); it != m_settings.end(); ++it)
   {
     if (LoadSetting(node, it->second.setting) && loadedSettings != NULL)
       loadedSettings->insert(make_pair(it->first, it->second.setting));
@@ -659,7 +659,7 @@ bool CSettingsManager::OnSettingChanging(const CSetting *setting)
 
   for (CallbackSet::iterator callback = settingData.callbacks.begin();
         callback != settingData.callbacks.end();
-        callback++)
+        ++callback)
   {
     if (!(*callback)->OnSettingChanging(setting))
       return false;
@@ -684,14 +684,14 @@ void CSettingsManager::OnSettingChanged(const CSetting *setting)
     
   for (CallbackSet::iterator callback = settingData.callbacks.begin();
         callback != settingData.callbacks.end();
-        callback++)
+        ++callback)
     (*callback)->OnSettingChanged(setting);
 
   // now handle any settings which depend on the changed setting
   const SettingDependencyMap& deps = GetDependencies(setting);
-  for (SettingDependencyMap::const_iterator depsIt = deps.begin(); depsIt != deps.end(); depsIt++)
+  for (SettingDependencyMap::const_iterator depsIt = deps.begin(); depsIt != deps.end(); ++depsIt)
   {
-    for (SettingDependencies::const_iterator depIt = depsIt->second.begin(); depIt != depsIt->second.end(); depIt++)
+    for (SettingDependencies::const_iterator depIt = depsIt->second.begin(); depIt != depsIt->second.end(); ++depIt)
       UpdateSettingByDependency(depsIt->first, *depIt);
   }
 }
@@ -712,7 +712,7 @@ void CSettingsManager::OnSettingAction(const CSetting *setting)
 
   for (CallbackSet::iterator callback = settingData.callbacks.begin();
         callback != settingData.callbacks.end();
-        callback++)
+        ++callback)
     (*callback)->OnSettingAction(setting);
 }
 
@@ -733,7 +733,7 @@ bool CSettingsManager::OnSettingUpdate(CSetting* &setting, const char *oldSettin
   bool ret = false;
   for (CallbackSet::iterator callback = settingData.callbacks.begin();
         callback != settingData.callbacks.end();
-        callback++)
+        ++callback)
     ret |= (*callback)->OnSettingUpdate(setting, oldSettingId, oldSettingNode);
 
   return ret;
@@ -755,7 +755,7 @@ void CSettingsManager::OnSettingPropertyChanged(const CSetting *setting, const c
 
   for (CallbackSet::iterator callback = settingData.callbacks.begin();
         callback != settingData.callbacks.end();
-        callback++)
+        ++callback)
     (*callback)->OnSettingPropertyChanged(setting, propertyName);
 }
 
@@ -782,7 +782,7 @@ CSetting* CSettingsManager::CreateSetting(const std::string &settingType, const
 bool CSettingsManager::OnSettingsLoading()
 {
   CSingleLock lock(m_critical);
-  for (SettingsHandlers::const_iterator it = m_settingsHandlers.begin(); it != m_settingsHandlers.end(); it++)
+  for (SettingsHandlers::const_iterator it = m_settingsHandlers.begin(); it != m_settingsHandlers.end(); ++it)
   {
     if (!(*it)->OnSettingsLoading())
       return false;
@@ -794,14 +794,14 @@ bool CSettingsManager::OnSettingsLoading()
 void CSettingsManager::OnSettingsLoaded()
 {
   CSingleLock lock(m_critical);
-  for (SettingsHandlers::const_iterator it = m_settingsHandlers.begin(); it != m_settingsHandlers.end(); it++)
+  for (SettingsHandlers::const_iterator it = m_settingsHandlers.begin(); it != m_settingsHandlers.end(); ++it)
     (*it)->OnSettingsLoaded();
 }
 
 bool CSettingsManager::OnSettingsSaving() const
 {
   CSingleLock lock(m_critical);
-  for (SettingsHandlers::const_iterator it = m_settingsHandlers.begin(); it != m_settingsHandlers.end(); it++)
+  for (SettingsHandlers::const_iterator it = m_settingsHandlers.begin(); it != m_settingsHandlers.end(); ++it)
   {
     if (!(*it)->OnSettingsSaving())
       return false;
@@ -813,21 +813,21 @@ bool CSettingsManager::OnSettingsSaving() const
 void CSettingsManager::OnSettingsSaved() const
 {
   CSingleLock lock(m_critical);
-  for (SettingsHandlers::const_iterator it = m_settingsHandlers.begin(); it != m_settingsHandlers.end(); it++)
+  for (SettingsHandlers::const_iterator it = m_settingsHandlers.begin(); it != m_settingsHandlers.end(); ++it)
     (*it)->OnSettingsSaved();
 }
 
 void CSettingsManager::OnSettingsCleared()
 {
   CSingleLock lock(m_critical);
-  for (SettingsHandlers::const_iterator it = m_settingsHandlers.begin(); it != m_settingsHandlers.end(); it++)
+  for (SettingsHandlers::const_iterator it = m_settingsHandlers.begin(); it != m_settingsHandlers.end(); ++it)
     (*it)->OnSettingsCleared();
 }
 
 bool CSettingsManager::Load(const TiXmlNode *settings)
 {
   bool ok = true;
-  for (std::set<ISubSettings*>::const_iterator it = m_subSettings.begin(); it != m_subSettings.end(); it++)
+  for (std::set<ISubSettings*>::const_iterator it = m_subSettings.begin(); it != m_subSettings.end(); ++it)
     ok &= (*it)->Load(settings);
 
   return ok;
@@ -871,13 +871,13 @@ bool CSettingsManager::UpdateSettings(const TiXmlNode *root)
 {
   bool updated = false;
 
-  for (SettingMap::iterator setting = m_settings.begin(); setting != m_settings.end(); setting++)
+  for (SettingMap::iterator setting = m_settings.begin(); setting != m_settings.end(); ++setting)
   {
     const std::set<CSettingUpdate>& updates = setting->second.setting->GetUpdates();
     if (updates.empty())
       continue;
 
-    for (std::set<CSettingUpdate>::const_iterator update = updates.begin(); update != updates.end(); update++)
+    for (std::set<CSettingUpdate>::const_iterator update = updates.begin(); update != updates.end(); ++update)
       updated |= UpdateSetting(root, setting->second.setting, *update);
   }
 
index a8b923d..71e5360 100644 (file)
@@ -364,7 +364,7 @@ private:
   bool Serialize(TiXmlNode *parent) const;
   bool Deserialize(const TiXmlNode *node, std::map<std::string, CSetting*> *loadedSettings = NULL);
 
-  bool LoadSetting(const TiXmlNode *node, CSetting *setting);
+  static bool LoadSetting(const TiXmlNode *node, CSetting *setting);
   bool UpdateSettings(const TiXmlNode *root);
   bool UpdateSetting(const TiXmlNode *node, CSetting *setting, const CSettingUpdate& update);
   void UpdateSettingByDependency(const std::string &settingId, const CSettingDependency &dependency);
index bc665ed..b893ece 100644 (file)
@@ -55,7 +55,7 @@ int CSkinSettings::TranslateString(const string &setting)
 
   CSingleLock lock(m_critical);
   // run through and see if we have this setting
-  for (map<int, CSkinString>::const_iterator it = m_strings.begin(); it != m_strings.end(); it++)
+  for (map<int, CSkinString>::const_iterator it = m_strings.begin(); it != m_strings.end(); ++it)
   {
     if (StringUtils::EqualsNoCase(settingName, it->second.name))
       return it->first;
@@ -100,7 +100,7 @@ int CSkinSettings::TranslateBool(const string &setting)
 
   CSingleLock lock(m_critical);
   // run through and see if we have this setting
-  for (map<int, CSkinBool>::const_iterator it = m_bools.begin(); it != m_bools.end(); it++)
+  for (map<int, CSkinBool>::const_iterator it = m_bools.begin(); it != m_bools.end(); ++it)
   {
     if (StringUtils::EqualsNoCase(settingName, it->second.name))
       return it->first;
@@ -147,7 +147,7 @@ void CSkinSettings::Reset(const string &setting)
 
   CSingleLock lock(m_critical);
   // run through and see if we have this setting as a string
-  for (map<int, CSkinString>::iterator it = m_strings.begin(); it != m_strings.end(); it++)
+  for (map<int, CSkinString>::iterator it = m_strings.begin(); it != m_strings.end(); ++it)
   {
     if (StringUtils::EqualsNoCase(settingName, it->second.name))
     {
@@ -157,7 +157,7 @@ void CSkinSettings::Reset(const string &setting)
   }
 
   // and now check for the skin bool
-  for (map<int, CSkinBool>::iterator it = m_bools.begin(); it != m_bools.end(); it++)
+  for (map<int, CSkinBool>::iterator it = m_bools.begin(); it != m_bools.end(); ++it)
   {
     if (StringUtils::EqualsNoCase(settingName, it->second.name))
     {
@@ -173,13 +173,13 @@ void CSkinSettings::Reset()
 
   CSingleLock lock(m_critical);
   // clear all the settings and strings from this skin.
-  for (map<int, CSkinBool>::iterator it = m_bools.begin(); it != m_bools.end(); it++)
+  for (map<int, CSkinBool>::iterator it = m_bools.begin(); it != m_bools.end(); ++it)
   {
     if (StringUtils::StartsWith(it->second.name, currentSkin))
       it->second.value = false;
   }
 
-  for (map<int, CSkinString>::iterator it = m_strings.begin(); it != m_strings.end(); it++)
+  for (map<int, CSkinString>::iterator it = m_strings.begin(); it != m_strings.end(); ++it)
   {
     if (StringUtils::StartsWith(it->second.name, currentSkin))
       it->second.value.clear();
index d7f2fab..eb7729b 100644 (file)
@@ -68,7 +68,7 @@ public:
 protected:
   CSkinSettings();
   CSkinSettings(const CSkinSettings&);
-  CSkinSettings const& operator=(CSkinSettings const&);
+  CSkinSettings& operator=(CSkinSettings const&);
   virtual ~CSkinSettings();
 
   std::string GetCurrentSkin() const;
index fd5a86f..a0a1883 100644 (file)
@@ -44,6 +44,12 @@ CGUIDialogContentSettings::CGUIDialogContentSettings(void)
   : CGUIDialogSettings(WINDOW_DIALOG_CONTENT_SETTINGS, "DialogContentSettings.xml"), m_origContent(CONTENT_NONE)
 {
   m_bNeedSave = false;
+  m_bShowScanSettings = false;
+  m_bScanRecursive = false;
+  m_bUseDirNames = false;
+  m_bSingleItem = false;
+  m_bExclude = false;
+  m_bNoUpdate = false;
   m_content = CONTENT_NONE;
   m_vecItems = new CFileItemList;
 }
index 6105076..050014f 100644 (file)
@@ -173,7 +173,7 @@ void CGUIControlSpinExSetting::FillControl()
         if (pSettingString->GetOptionsType() == SettingOptionsTypeDynamic)
         {
           DynamicStringSettingOptions options = pSettingString->UpdateDynamicOptions();
-          for (std::vector< std::pair<std::string, std::string> >::const_iterator option = options.begin(); option != options.end(); option++)
+          for (std::vector< std::pair<std::string, std::string> >::const_iterator option = options.begin(); option != options.end(); ++option)
             m_pSpin->AddLabel(option->first, option->second);
 
           m_pSpin->SetStringValue(pSettingString->GetValue());
@@ -195,7 +195,7 @@ void CGUIControlSpinExSetting::FillIntegerSettingControl()
     case SettingOptionsTypeStatic:
     {
       const StaticIntegerSettingOptions& options = pSettingInt->GetOptions();
-      for (StaticIntegerSettingOptions::const_iterator it = options.begin(); it != options.end(); it++)
+      for (StaticIntegerSettingOptions::const_iterator it = options.begin(); it != options.end(); ++it)
         m_pSpin->AddLabel(g_localizeStrings.Get(it->first), it->second);
 
       break;
@@ -204,7 +204,7 @@ void CGUIControlSpinExSetting::FillIntegerSettingControl()
     case SettingOptionsTypeDynamic:
     {
       DynamicIntegerSettingOptions options = pSettingInt->UpdateDynamicOptions();
-      for (DynamicIntegerSettingOptions::const_iterator option = options.begin(); option != options.end(); option++)
+      for (DynamicIntegerSettingOptions::const_iterator option = options.begin(); option != options.end(); ++option)
         m_pSpin->AddLabel(option->first, option->second);
 
       break;
@@ -341,7 +341,7 @@ bool CGUIControlListSetting::GetItems(CSetting *setting, CFileItemList &items)
         if (pSettingString->GetOptionsType() == SettingOptionsTypeDynamic)
         {
           DynamicStringSettingOptions options = pSettingString->UpdateDynamicOptions();
-          for (DynamicStringSettingOptions::const_iterator option = options.begin(); option != options.end(); option++)
+          for (DynamicStringSettingOptions::const_iterator option = options.begin(); option != options.end(); ++option)
           {
             CFileItemPtr pItem = GetItem(option->first, option->second);
 
@@ -370,7 +370,7 @@ bool CGUIControlListSetting::GetIntegerItems(CSetting *setting, CFileItemList &i
     case SettingOptionsTypeStatic:
     {
       const StaticIntegerSettingOptions& options = pSettingInt->GetOptions();
-      for (StaticIntegerSettingOptions::const_iterator it = options.begin(); it != options.end(); it++)
+      for (StaticIntegerSettingOptions::const_iterator it = options.begin(); it != options.end(); ++it)
       {
         CFileItemPtr pItem = GetItem(g_localizeStrings.Get(it->first), it->second);
 
@@ -385,7 +385,7 @@ bool CGUIControlListSetting::GetIntegerItems(CSetting *setting, CFileItemList &i
     case SettingOptionsTypeDynamic:
     {
       DynamicIntegerSettingOptions options = pSettingInt->UpdateDynamicOptions();
-      for (DynamicIntegerSettingOptions::const_iterator option = options.begin(); option != options.end(); option++)
+      for (DynamicIntegerSettingOptions::const_iterator option = options.begin(); option != options.end(); ++option)
       {
         CFileItemPtr pItem = GetItem(option->first, option->second);
 
@@ -506,7 +506,7 @@ bool CGUIControlButtonSetting::GetPath(CSettingPath *pathSetting)
 
   VECSOURCES shares;
   const std::vector<std::string>& sources = pathSetting->GetSources();
-  for (std::vector<std::string>::const_iterator source = sources.begin(); source != sources.end(); source++)
+  for (std::vector<std::string>::const_iterator source = sources.begin(); source != sources.end(); ++source)
   {
     VECSOURCES *sources = CMediaSourceSettings::Get().GetSources(*source);
     if (sources != NULL)
index c4441b4..584384c 100644 (file)
@@ -41,7 +41,7 @@ public:
   CGUIControlBaseSetting(int id, CSetting *pSetting);
   virtual ~CGUIControlBaseSetting() {}
   
-  int GetID() { return m_id; }
+  int GetID() const { return m_id; }
   CSetting* GetSetting() { return m_pSetting; }
 
   /*!
index 05e2a1f..b00cd17 100644 (file)
@@ -92,6 +92,12 @@ CGUIWindowSettingsCategory::CGUIWindowSettingsCategory(void)
     : CGUIWindow(WINDOW_SETTINGS_MYPICTURES, "SettingsCategory.xml"),
       m_settings(CSettings::Get()),
       m_iCategory(0), m_iSection(0),
+      m_pOriginalSpin(NULL),
+      m_pOriginalRadioButton(NULL),
+      m_pOriginalCategoryButton(NULL),
+      m_pOriginalButton(NULL),
+      m_pOriginalEdit(NULL),
+      m_pOriginalImage(NULL),
       m_delayedTimer(this),
       m_returningFromSkinLoad(false)
 {
@@ -272,7 +278,7 @@ bool CGUIWindowSettingsCategory::OnAction(const CAction &action)
     {
       if (CGUIDialogYesNo::ShowAndGetInput(10041, 0, 10042, 0))
       {
-        for(vector<BaseSettingControlPtr>::iterator it = m_settingControls.begin(); it != m_settingControls.end(); it++)
+        for(vector<BaseSettingControlPtr>::iterator it = m_settingControls.begin(); it != m_settingControls.end(); ++it)
         {
           CSetting *setting = (*it)->GetSetting();
           if (setting != NULL)
@@ -350,7 +356,7 @@ void CGUIWindowSettingsCategory::DoProcess(unsigned int currentTime, CDirtyRegio
     }
   }
   CGUIWindow::DoProcess(currentTime, dirtyregions);
-  if (bAlphaFaded)
+  if (control && bAlphaFaded)
   {
     control->SetFocus(false);
     if (control->GetControlType() == CGUIControl::GUICONTROL_BUTTON)
@@ -415,7 +421,7 @@ void CGUIWindowSettingsCategory::SetupControls(bool createSettings /* = true */)
 
   // go through the categories and create the necessary buttons
   int buttonIdOffset = 0;
-  for (SettingCategoryList::const_iterator category = m_categories.begin(); category != m_categories.end(); category++)
+  for (SettingCategoryList::const_iterator category = m_categories.begin(); category != m_categories.end(); ++category)
   {
     CGUIButtonControl *pButton = NULL;
     if (m_pOriginalCategoryButton->GetControlType() == CGUIControl::GUICONTROL_TOGGLEBUTTON)
@@ -461,7 +467,7 @@ void CGUIWindowSettingsCategory::FreeSettingsControls()
     control->ClearAll();
   }
 
-  for (std::vector<BaseSettingControlPtr>::iterator control = m_settingControls.begin(); control != m_settingControls.end(); control++)
+  for (std::vector<BaseSettingControlPtr>::iterator control = m_settingControls.begin(); control != m_settingControls.end(); ++control)
     (*control)->Clear();
 
   m_settingControls.clear();
@@ -530,7 +536,7 @@ void CGUIWindowSettingsCategory::CreateSettings()
   const SettingGroupList& groups = category->GetGroups(CViewStateSettings::Get().GetSettingLevel());
   int iControlID = CONTROL_START_CONTROL;
   bool first = true;
-  for (SettingGroupList::const_iterator groupIt = groups.begin(); groupIt != groups.end(); groupIt++)
+  for (SettingGroupList::const_iterator groupIt = groups.begin(); groupIt != groups.end(); ++groupIt)
   {
     if (*groupIt == NULL)
       continue;
@@ -544,7 +550,7 @@ void CGUIWindowSettingsCategory::CreateSettings()
     else
       AddSeparator(group->GetWidth(), iControlID);
 
-    for (SettingList::const_iterator settingIt = settings.begin(); settingIt != settings.end(); settingIt++)
+    for (SettingList::const_iterator settingIt = settings.begin(); settingIt != settings.end(); ++settingIt)
     {
       CSetting *pSetting = *settingIt;
       settingMap.insert(pSetting->GetId());
@@ -552,7 +558,7 @@ void CGUIWindowSettingsCategory::CreateSettings()
     }
   }
 
-  if (settingMap.size() > 0)
+  if (!settingMap.empty())
     m_settings.RegisterCallback(this, settingMap);
   
   // update our settings (turns controls on/off as appropriate)
@@ -561,7 +567,7 @@ void CGUIWindowSettingsCategory::CreateSettings()
 
 void CGUIWindowSettingsCategory::UpdateSettings()
 {
-  for (vector<BaseSettingControlPtr>::iterator it = m_settingControls.begin(); it != m_settingControls.end(); it++)
+  for (vector<BaseSettingControlPtr>::iterator it = m_settingControls.begin(); it != m_settingControls.end(); ++it)
   {
     BaseSettingControlPtr pSettingControl = *it;
     CSetting *pSetting = pSettingControl->GetSetting();
@@ -719,8 +725,6 @@ CGUIControl* CGUIWindowSettingsCategory::AddSettingControl(CGUIControl *pControl
 
 void CGUIWindowSettingsCategory::OnClick(BaseSettingControlPtr pSettingControl)
 {
-  std::string strSetting = pSettingControl->GetSetting()->GetId();
-
   // we need to first set the delayed setting and then execute OnClick()
   // because OnClick() triggers OnSettingChanged() and there we need to
   // know if the changed setting is delayed or not
@@ -754,7 +758,7 @@ CSettingSection* CGUIWindowSettingsCategory::GetSection(int windowID) const
 
 BaseSettingControlPtr CGUIWindowSettingsCategory::GetSettingControl(const std::string &strSetting)
 {
-  for (vector<BaseSettingControlPtr>::iterator control = m_settingControls.begin(); control != m_settingControls.end(); control++)
+  for (vector<BaseSettingControlPtr>::iterator control = m_settingControls.begin(); control != m_settingControls.end(); ++control)
   {
     if ((*control)->GetSetting() != NULL && (*control)->GetSetting()->GetId() == strSetting)
       return *control;
index e11575d..5cf8800 100644 (file)
@@ -49,6 +49,9 @@ using namespace std;
 CGUIWindowSettingsScreenCalibration::CGUIWindowSettingsScreenCalibration(void)
     : CGUIWindow(WINDOW_SCREEN_CALIBRATION, "SettingsScreenCalibration.xml")
 {
+  m_iCurRes = 0;
+  m_iControl = 0;
+  m_fPixelRatioBoxHeight = 0.0f;
   m_needsScaling = false;         // we handle all the scaling
 }
 
index 555e327..d9ab11d 100644 (file)
 CGUIWindowTestPattern::CGUIWindowTestPattern(void)
     : CGUIWindow(WINDOW_TEST_PATTERN, "")
 {
+  m_pattern = 0;
+  m_bounceX = 0;
+  m_bounceY = 0;
+  m_bounceDirectionX = 0;
+  m_bounceDirectionY = 0;
+  m_blinkFrame = 0;
   m_needsScaling = false;
 }