Merge lp:~tintou/wingpanel-indicator-bluetooth/no-disconnect into lp:~wingpanel-devs/wingpanel-indicator-bluetooth/trunk

[Corentin Noël]

Don't disconnect if this is not required

Don't disconnect at startup

[Adam Bieńkowski (donadigo)]

Perhaps put the check outside the thread function and do a "if (!state)"?

[Corentin Noël (tintou)]

if (!state) and if (state == false) are the same condition, it's just that the "state" name isn't obvious so I prefer to explicitly keep the == false

I actually need the device.disconnect (); to be done on a separate thread so that it doesn't block the UI

=== modified file 'src/Services/Manager.vala'
--- src/Services/Manager.vala	2017-01-22 16:33:44 +0000
+++ src/Services/Manager.vala	2017-02-01 15:57:24 +0000
@@ -187,13 +187,15 @@
 
     public void set_global_state (bool state) {
         new Thread<void*> (null, () => {
-            lock (devices) {
-                foreach (var device in devices.values) {
-                    if (device.connected) {
-                        try {
-                            device.disconnect ();
-                        } catch (Error e) {
-                            critical (e.message);
+            if (state == false) {
+                lock (devices) {
+                    foreach (var device in devices.values) {
+                        if (device.connected) {
+                            try {
+                                device.disconnect ();
+                            } catch (Error e) {
+                                critical (e.message);
+                            }
                         }
                     }
                 }