Fixed/Improved signal/slot registrations.

Author: 5cript

README.md

@@ -518,15 +518,15 @@ using namespace std::chrono_literals;
 class IAccounts
 {
 public:
-	virtual ~IAccounts() = default;
+    virtual ~IAccounts() = default;
 
 public: // Methods
-	virtual auto CreateUser(std::string const& name, std::string const& fullname, int32_t accountType) -> object_path = 0;
-	virtual auto DeleteUser(int64_t id, bool removeFiles) -> void = 0;
+    virtual auto CreateUser(std::string const& name, std::string const& fullname, int32_t accountType) -> object_path = 0;
+    virtual auto DeleteUser(int64_t id, bool removeFiles) -> void = 0;
 public: // Properties
 public: // signals
-	DBusGlue::signal <void(object_path)> UserAdded;
-	DBusGlue::signal <void(object_path)> UserDeleted;
+    DBusGlue::signal <void(object_path)> UserAdded;
+    DBusGlue::signal <void(object_path)> UserDeleted;
 };
 
 //----------------------------------------------------------------------------------------
@@ -545,70 +545,67 @@ DBUS_DECLARE
 
 int main()
 {
-	auto bus = open_system_bus();
-
-	try
-	{
-		bus.install_event_loop(std::unique_ptr <event_loop> (new busy_loop(&bus, 50ms)));
-
-		// wrapped interface for creating / deleting accounts.
-		auto accountControl = create_interface <IAccounts>(
-		    bus,
-		    "org.freedesktop.Accounts",
-		    "/org/freedesktop/Accounts",
-		    "org.freedesktop.Accounts"
-		);
-
-		accountControl.UserAdded.listen(
-		    [](object_path const& p) {
-			    // success callback
-			    std::cout << "callback - create: " << p << std::endl;
-		    },
-		    [](message&, std::string const& str) {
-			    // failure callback
-			    std::cerr << "oh no something gone wrong: " << str << "\n";
-		    }
-		);
-
-		// WARNING! Passing "release_slot" forces you to manage the slots lifetime yourself!
-		// You can use this variation to manage lifetimes of your observed signals. With a unique_ptr for example.
-		auto* slot = accountControl.UserDeleted.listen(
-		    [&](object_path const& p) {
-		        // this is called from the dbus system.
-		        std::cout << "callback - delete: " << p << std::endl;
-
-		        // create a user from here.
-		        auto path = accountControl.CreateUser("tempus", "tempus", 0);
-	        },
-		    [](message&, std::string const& str) {
-			    // this is called when an error got signaled into our callback.
-			    std::cerr << "oh no something gone wrong: " << str << "\n";
-		    },
-		    DBusGlue::release_slot
-			);
-
-			// try to delete a user with id 1001. WARNING, DONT JUST DELETE SOME USER ON YOUR SYSTEM. obviously...
-			try {
-				// commented out in case you just run this example
-				// you should get the id from the name first.
-				//accountControl.DeleteUser(1001, false);
-			} catch (std::exception const& exc) {
-				// Create the user if he doesn't exist
-				accountControl.CreateUser("tempus", "tempus", 0);
-				std::cout << exc.what() << std::endl;
-			}
-
-			// just wait here so we dont exit directly
-			std::cin.get();
-
-		// cleanup. IF!!! you passed "release_slot"
-		delete slot;
-	}
-	catch (std::exception const& exc)
-	{
-		std::cout << exc.what() << "\n";
-	}
-
-	return 0;
+    auto bus = open_system_bus();
+
+    try
+    {
+        bus.install_event_loop(std::unique_ptr <event_loop> (new busy_loop(&bus, 50ms)));
+
+        // wrapped interface for creating / deleting accounts.
+        auto accountControl = create_interface <IAccounts>(
+            bus,
+            "org.freedesktop.Accounts",
+            "/org/freedesktop/Accounts",
+            "org.freedesktop.Accounts"
+        );
+
+        accountControl.UserAdded.listen(
+            [](object_path const& p) {
+                // success callback
+                std::cout << "callback - create: " << p << std::endl;
+            },
+            [](message&, std::string const& str) {
+                // failure callback
+                std::cerr << "oh no something gone wrong: " << str << "\n";
+            }
+        );
+
+        // WARNING! Passing "release_slot" forces you to manage the slots lifetime yourself!
+        // You can use this variation to manage lifetimes of your observed signals.
+        std::unique_ptr<void, void(*)(void*)> slot = accountControl.UserDeleted.listen(
+            [&](object_path const& p) {
+                // this is called from the dbus system.
+                std::cout << "callback - delete: " << p << std::endl;
+
+                // create a user from here.
+                auto path = accountControl.CreateUser("tempus", "tempus", 0);
+            },
+            [](message&, std::string const& str) {
+                // this is called when an error got signaled into our callback.
+                std::cerr << "oh no something gone wrong: " << str << "\n";
+            },
+            DBusGlue::release_slot
+        );
+
+        // try to delete a user with id 1001. WARNING, DONT JUST DELETE SOME USER ON YOUR SYSTEM. obviously...
+        try {
+            // commented out in case you just run this example
+            // you should get the id from the name first.
+            //accountControl.DeleteUser(1001, false);
+        } catch (std::exception const& exc) {
+            // Create the user if he doesn't exist
+            accountControl.CreateUser("tempus", "tempus", 0);
+            std::cout << exc.what() << std::endl;
+        }
+
+        // just wait here so we dont exit directly
+        std::cin.get();
+    }
+    catch (std::exception const& exc)
+    {
+        std::cout << exc.what() << "\n";
+    }
+
+    return 0;
 }
 ```