Improved example code.

Author: Tim Ebbeke

README.md

@@ -36,3 +36,8 @@ Everything will be contained within "bluetoot_project".
 - cd build
 - cmake ..
 - cmake --build . -j4
+
+## Troubleshooting
+### Unit dbus-org.bluez.service not found
+Start the bluetooth service that exposes the dbus interfaces: `sudo systemctl start bluetooth`.
+### could not give myself a name: Permission denied

bluetooth/bluez/adapter.cpp

@@ -148,20 +148,28 @@ namespace Bluetooth
         return devices;
     }
 //---------------------------------------------------------------------------------------------------------------------
-    Device Adapter::getPairedByName(std::string const& name)
+    std::optional<Device> Adapter::getPairedByName(std::string const& name)
     {
+        bool anyFound = false;
         DBusGlue::object_path foundDev;
         auto devices = pairedDevices();
         for (auto const& device : devices)
         {
-            std::cout << device << "\n";
             Device dev{impl_->bus, device};
 
             if (dev.api().Name == name)
+            {
                 foundDev = device;
+                anyFound = true;
+                break;
+            }
+        }
+        if (!anyFound)
+        {
+            return std::nullopt;
         }
 
-        return {impl_->bus, foundDev};
+        return Device{impl_->bus, foundDev};
     }
 //#####################################################################################################################
 }

bluetooth/bluez/adapter.hpp

@@ -5,6 +5,7 @@
 
 #include <memory>
 #include <string>
+#include <optional>
 
 namespace Bluetooth
 {
@@ -98,7 +99,7 @@ namespace Bluetooth
          * @param name
          * @return
          */
-        Device getPairedByName(std::string const& name);
+        std::optional<Device> getPairedByName(std::string const& name);
 
     private:
         std::unique_ptr <Implementation> impl_;

bluetooth/bluez/device.cpp

@@ -36,6 +36,10 @@ namespace Bluetooth
     }
 //---------------------------------------------------------------------------------------------------------------------
     Device::~Device() = default;
+//---------------------------------------------------------------------------------------------------------------------
+    Device::Device(Device&&) = default;
+//---------------------------------------------------------------------------------------------------------------------
+    Device& Device::operator=(Device&&) = default;
 //---------------------------------------------------------------------------------------------------------------------
     Device::device_type& Device::api()
     {

bluetooth/bluez/device.hpp

@@ -15,6 +15,10 @@ namespace Bluetooth
 
         Device(DBusGlue::dbus* bus, DBusGlue::object_path const& path);
         ~Device();
+        Device(Device&&);
+        Device(Device const&) = delete;
+        Device& operator=(Device&&);
+        Device& operator=(Device const&) = delete;
 
         device_type& api();
 

dbus-glue-system

@@ -18,8 +18,9 @@ int main()
         1,
         "/bla/bluetooth",
         "SerialServer",
-        true,
-        "de.blabla.cpp"s
+        true
+        // requires modified rights as documented in README under 3)
+        // "de.blabla.cpp"s
     };
 
     Server server{&bus, info};
@@ -36,8 +37,10 @@ int main()
     */
 
     auto phone = driver.getPairedByName("HUAWEI P20");
-
-    phone.api().Trusted = true;
+    if (phone)
+    {
+        phone->api().Trusted = true;
+    }
 
     server.start(
         // on connect: