From 1e5ca1852e059d640a7d2c080a2d7a79d53b37e8 Mon Sep 17 00:00:00 2001
From: Bjørn Lindeijer <bjorn@lindeijer.nl>
Date: Mon, 2 May 2005 11:48:26 +0000
Subject: Prettier errors when things go wrong.

---
 src/configuration.cpp | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

(limited to 'src/configuration.cpp')

diff --git a/src/configuration.cpp b/src/configuration.cpp
index 8cacd065..54fe3a1d 100644
--- a/src/configuration.cpp
+++ b/src/configuration.cpp
@@ -85,8 +85,17 @@ void Configuration::init(const std::string &filename)
     xmlFreeDoc(doc);
 }
 
-bool Configuration::write()
+void Configuration::write()
 {
+    // Do not attempt to write to file that cannot be opened for writing
+    FILE *testFile = fopen(configPath.c_str(), "w");
+    if (!testFile) {
+        return;
+    }
+    else {
+        fclose(testFile);
+    }
+
     xmlTextWriterPtr writer = xmlNewTextWriterFilename(configPath.c_str(), 0);
 
     if (writer)
@@ -112,11 +121,7 @@ bool Configuration::write()
 
         xmlTextWriterEndDocument(writer);
         xmlFreeTextWriter(writer);
-
-        return true;
     }
-
-    return false;
 }
 
 void Configuration::setValue(const std::string &key, std::string value)
-- 
cgit v1.2.3-70-g09d2