Merge pull request #950 from aschnell/master

- move more functions to snapper namespace

client/cleanup.cc

@@ -37,6 +37,9 @@
 #include "cleanup.h"
 
 
+namespace snapper
+{
+
 using namespace std;
 
 
@@ -887,3 +890,5 @@ do_cleanup_empty_pre_post(ProxySnapper* snapper, bool verbose, std::function<boo
     EmptyPrePostCleaner cleaner(snapper, verbose, parameters);
     cleaner.cleanup(condition, report);
 }
+
+}

client/cleanup.h

@@ -26,6 +26,9 @@
 #include "proxy/proxy.h"
 
 
+namespace snapper
+{
+
 /*
  * The following three functions do the cleanup based on the conditionals defined in the
  * config, that are hard limit, quota and free space.
@@ -57,3 +60,5 @@ do_cleanup_timeline(ProxySnapper* snapper, bool verbose, std::function<bool()> c
 void
 do_cleanup_empty_pre_post(ProxySnapper* snapper, bool verbose, std::function<bool()> condition,
 			  Plugins::Report& report);
+
+}

client/misc.cc

@@ -36,6 +36,9 @@
 #include "misc.h"
 
 
+namespace snapper
+{
+
 unsigned int
 read_num(const string& str)
 {
@@ -213,3 +216,5 @@ Differ::run(const string& f1, const string& f2) const
     for (const string& line : cmd.get_stderr())
 	cerr << line << endl;
 }
+
+}

client/misc.h

@@ -33,9 +33,8 @@
 #include "proxy/proxy.h"
 
 
-using namespace snapper;
-using namespace std;
-
+namespace snapper
+{
 
 unsigned int
 read_num(const string& str);
@@ -67,9 +66,6 @@ struct Differ
 };
 
 
-namespace snapper
-{
-
     /**
      * Return a string listing the possible enum values. E.g. "Use auto, classic or
      * transactional." for possible_enum_values<Ambit>().

client/proxy/commands.cc

@@ -30,14 +30,18 @@
 #include "dbus/DBusPipe.h"
 #include "snapper/AppUtil.h"
 
-using namespace std;
-
 
 #define SERVICE "org.opensuse.Snapper"
 #define OBJECT "/org/opensuse/Snapper"
 #define INTERFACE "org.opensuse.Snapper"
 
 
+namespace snapper
+{
+
+using namespace std;
+
+
 vector<XConfigInfo>
 command_list_xconfigs(DBus::Connection& conn)
 {
@@ -744,3 +748,5 @@ command_debug(DBus::Connection& conn)
 
     return lines;
 }
+
+}

client/proxy/commands.h

@@ -21,17 +21,12 @@
  */
 
 
-#include <string>
-#include <vector>
-#include <map>
-
-using std::string;
-using std::vector;
-using std::map;
-
 #include "types.h"
 
 
+namespace snapper
+{
+
 vector<XConfigInfo>
 command_list_xconfigs(DBus::Connection& conn);
 
@@ -163,3 +158,5 @@ command_get_plugins_report(DBus::Connection& conn);
 
 vector<string>
 command_debug(DBus::Connection& conn);
+
+}

client/proxy/proxy-dbus.cc

@@ -25,6 +25,9 @@
 #include "../utils/text.h"
 
 
+namespace snapper
+{
+
 using namespace std;
 
 
@@ -487,3 +490,5 @@ ProxySnappers::createDbus()
 {
     return ProxySnappers(new ProxySnappersDbus());
 }
+
+}

client/proxy/proxy-dbus.h

@@ -30,6 +30,9 @@
 #include "proxy.h"
 
 
+namespace snapper
+{
+
 class ProxySnapshotDbus;
 class ProxySnapshotsDbus;
 class ProxySnapperDbus;
@@ -229,5 +232,7 @@ class ProxyComparisonDbus : public ProxyComparison::Impl
 
 };
 
+}
+
 
 #endif

client/proxy/proxy-lib.cc

@@ -23,6 +23,9 @@
 #include "proxy-lib.h"
 
 
+namespace snapper
+{
+
 using namespace std;
 
 
@@ -228,3 +231,5 @@ to_lib(const ProxySnapshot& proxy_snapshot)
 {
     return dynamic_cast<const ProxySnapshotLib&>(proxy_snapshot.get_impl());
 }
+
+}

client/proxy/proxy-lib.h

@@ -30,6 +30,9 @@
 #include <snapper/Comparison.h>
 
 
+namespace snapper
+{
+
 class ProxySnapshotLib : public ProxySnapshot::Impl
 {
 
@@ -203,5 +206,6 @@ class ProxyComparisonLib : public ProxyComparison::Impl
 const ProxySnapshotLib&
 to_lib(const ProxySnapshot& proxy_snapshot);
 
+}
 
 #endif

client/proxy/proxy.cc

@@ -31,6 +31,9 @@
 #include "proxy.h"
 
 
+namespace snapper
+{
+
 using namespace std;
 
 
@@ -207,3 +210,5 @@ ProxySnapshots::findPost(const_iterator pre) const
 
     return end();
 }
+
+}

client/proxy/proxy.h

@@ -34,8 +34,8 @@
 #include <snapper/File.h>
 
 
-using namespace snapper;
-
+namespace snapper
+{
 
 /**
  * The proxy classes here allow clients, so far only the snapper command line
@@ -360,5 +360,7 @@ class ProxyComparison
 
 };
 
+}
+
 
 #endif

client/proxy/types.h

@@ -25,18 +25,20 @@
 #include <vector>
 #include <map>
 
-using std::string;
-using std::vector;
-using std::map;
-
 #include "dbus/DBusMessage.h"
 #include "dbus/DBusConnection.h"
 #include "snapper/Snapshot.h"
 #include "snapper/File.h"
 #include "snapper/SnapperTmpl.h"
 #include "snapper/Snapper.h"
 
-using namespace snapper;
+
+namespace snapper
+{
+
+using std::string;
+using std::vector;
+using std::map;
 
 
 struct XConfigInfo
@@ -102,6 +104,8 @@ struct XReport
     int exit_status;
 };
 
+}
+
 
 namespace DBus
 {

zypp-plugin/snapper-zypp-plugin.cc

@@ -31,7 +31,7 @@
 #include <boost/algorithm/string.hpp>
 
 #include <json.h>
-// a collision with client/errors.h
+// a collision with client/proxy/errors.h
 #ifdef error_description
 #undef error_description
 #endif