- work on dbus interface

diff --git a/server/Client.cc b/server/Client.cc
index 481882d22cd4232c9b489a26d30a577231578872..0ee339edb27408c25c335384ebcb312e93affc1e 100644 (file)
--- a/server/Client.cc
+++ b/server/Client.cc
@@ -391,7 +391,7 @@ Client::signal_config_deleted(DBus::Connection& conn, const string& config_name)
 
 void
 Client::signal_snapshot_created(DBus::Connection& conn, const string& config_name,
-                                 unsigned int num)
+                               unsigned int num)
 {
     DBus::MessageSignal msg(PATH, INTERFACE, "SnapshotCreated");
 
@@ -404,7 +404,7 @@ Client::signal_snapshot_created(DBus::Connection& conn, const string& config_nam
 
 void
 Client::signal_snapshots_deleted(DBus::Connection& conn, const string& config_name,
-                                  list<dbus_uint32_t> nums)
+                                const list<dbus_uint32_t>& nums)
 {
     DBus::MessageSignal msg(PATH, INTERFACE, "SnapshotsDeleted");
 

diff --git a/server/Client.h b/server/Client.h
index c906984fcfdc853e7e403e1911a07e5f44d73426..9715b22ac75522d60bdf6d49d9fab605f2d451f0 100644 (file)
--- a/server/Client.h
+++ b/server/Client.h
@@ -77,7 +77,7 @@ public:
     void signal_snapshot_created(DBus::Connection& conn, const string& config_name,
                                 unsigned int num);
     void signal_snapshots_deleted(DBus::Connection& conn, const string& config_name,
-                                 list<dbus_uint32_t> nums);
+                                 const list<dbus_uint32_t>& nums);
 
     void list_configs(DBus::Connection& conn, DBus::Message& msg);
     void get_config(DBus::Connection& conn, DBus::Message& msg);

diff --git a/server/snapperd.cc b/server/snapperd.cc
index b1ab6eb941eaf6775cedc4af55eb9df1751223f4..a8ce250682743e709c4284dfd582fbc8e10bf42f 100644 (file)
--- a/server/snapperd.cc
+++ b/server/snapperd.cc
@@ -35,6 +35,10 @@
 using namespace std;
 
 
+const int idle_time = 60;
+const int snapper_cleanup_time = 30;
+
+
 class MyMainLoop : public DBus::MainLoop
 {
 public:
@@ -141,11 +145,11 @@ MyMainLoop::periodic()
     clients.remove_zombies();
 
     if (clients.empty())
-       set_idle_timeout(30);
+       set_idle_timeout(idle_time);
 
     for (MetaSnappers::iterator it = meta_snappers.begin(); it != meta_snappers.end(); ++it)
     {
-       if (it->is_loaded() && it->unused_for() > 10)
+       if (it->is_loaded() && it->unused_for() > snapper_cleanup_time)
            it->unload();
     }
 }
@@ -197,7 +201,7 @@ main(int argc, char** argv)
 
     MyMainLoop mainloop(DBUS_BUS_SYSTEM);
 
-    mainloop.set_idle_timeout(30);
+    mainloop.set_idle_timeout(idle_time);
 
     y2mil("Requesting DBus name");