From 700953f2950e1c4eb8340c30f17de7b81332a27e Mon Sep 17 00:00:00 2001
From: hiker <henrichsjoerg@gmail.com>
Date: Thu, 20 Feb 2014 22:37:27 +1100
Subject: [PATCH] Removed unnecessary save function.

---
 src/achievements/achievements_manager.cpp | 27 ++---------------------
 src/achievements/achievements_manager.hpp |  3 +--
 2 files changed, 3 insertions(+), 27 deletions(-)

diff --git a/src/achievements/achievements_manager.cpp b/src/achievements/achievements_manager.cpp
index 6bf8a76c0..cb19c2e82 100644
--- a/src/achievements/achievements_manager.cpp
+++ b/src/achievements/achievements_manager.cpp
@@ -45,13 +45,12 @@ AchievementsManager::AchievementsManager()
 // ----------------------------------------------------------------------------
 AchievementsManager::~AchievementsManager()
 {
-    save();
     std::map<uint32_t, AchievementInfo *>::iterator it;
     for ( it = m_achievements_info.begin(); it != m_achievements_info.end(); ++it ) {
         delete it->second;
     }
     m_achievements_info.clear();
-}
+}   // ~AchievementsManager
 
 // ----------------------------------------------------------------------------
 /** Parses the data/achievements.xml file and stores the information about
@@ -103,6 +102,7 @@ AchievementsStatus*
 {
     AchievementsStatus *status = new AchievementsStatus();
 
+    // First add all achievements, before restoring the saved data.
     std::map<uint32_t, AchievementInfo *>::const_iterator it;
     for (it  = m_achievements_info.begin(); 
          it != m_achievements_info.end(); ++it)
@@ -126,29 +126,6 @@ AchievementsStatus*
     return status;
 }   // createAchievementStatus
 
-// ----------------------------------------------------------------------------
-void AchievementsManager::save()
-{
-    std::string filename = file_manager->getUserConfigFile("achievements.xml");
-
-    std::ofstream achievements_file(filename.c_str(), std::ios::out);
-
-    if (!achievements_file.is_open())
-    {
-        Log::warn("AchievementsManager::save",
-                  "Failed to open '%s' for writing, achievements won't be saved\n",
-                  filename.c_str());
-        return;
-    }
-
-    achievements_file << "<?xml version=\"1.0\"?>\n";
-    achievements_file << "<achievements>\n";
-
-    achievements_file << "</achievements>\n\n";
-    achievements_file.close();
-}
-
-
 // ----------------------------------------------------------------------------
 void AchievementsManager::onRaceEnd()
 {
diff --git a/src/achievements/achievements_manager.hpp b/src/achievements/achievements_manager.hpp
index 9428e8c31..ec91597b6 100644
--- a/src/achievements/achievements_manager.hpp
+++ b/src/achievements/achievements_manager.hpp
@@ -69,9 +69,8 @@ public:
         delete m_achievements_manager;
         m_achievements_manager = NULL;
     }   // destroy
-    // ------------------------------------------------------------------------
+    // ========================================================================
 
-    void save();
     void onRaceEnd();
     AchievementInfo* getAchievementInfo(uint32_t id) const;
     AchievementsStatus* createAchievementsStatus(const XMLNode *node=NULL);