Java DBTAdapter + DBTDevice close @ JVM shutdown: Avoid all interaction, native dtor (deleteImpl) cleans up.

Shortcut of 'close' take-down @ JVM shutdown, avoiding any potential deadlocks.

However, all device's and adapter's deleteImpl() gets issued,
allowing the native adapter's destructor to be called for orderly device cleanup.

1 files changed, 23 insertions, 17 deletions

diff --git a/java/direct_bt/tinyb/DBTDevice.java b/java/direct_bt/tinyb/DBTDevice.java
index a98636bb..1f9ec3be 100644
--- a/java/direct_bt/tinyb/DBTDevice.java
+++ b/java/direct_bt/tinyb/DBTDevice.java
@@ -233,30 +233,36 @@ public class DBTDevice extends DBTObject implements BluetoothDevice
 
     @Override
     public synchronized void close() {
+        close(false);
+    }
+
+    /* pp */ synchronized void close(final boolean isShutdown) {
         if( !isValid() ) {
             return;
         }
-        disconnect();
+        if( !isShutdown ) { // avoid all interaction @ JVM shutdown, native dtor (deleteImpl) cleans up.
+            disconnect();
 
-        disableConnectedNotifications();
-        disableRSSINotifications();
-        disableManufacturerDataNotifications();
-        disableServicesResolvedNotifications();
+            disableConnectedNotifications();
+            disableRSSINotifications();
+            disableManufacturerDataNotifications();
+            disableServicesResolvedNotifications();
 
-        disableBlockedNotifications();
-        disableBlockedNotificationsImpl();
-        disablePairedNotifications();
-        disablePairedNotificationsImpl();
-        disableServiceDataNotifications();
-        disableTrustedNotifications();
-        // FIXME disableTrustedNotificationsImpl();
+            disableBlockedNotifications();
+            disableBlockedNotificationsImpl();
+            disablePairedNotifications();
+            disablePairedNotificationsImpl();
+            disableServiceDataNotifications();
+            disableTrustedNotifications();
+            // FIXME disableTrustedNotificationsImpl();
 
-        clearServiceCache();
+            clearServiceCache();
 
-        final DBTAdapter a = getAdapter();
-        if( null != a ) {
-            a.removeStatusListener(statusListener);
-            a.removeDiscoveredDevice(this);
+            final DBTAdapter a = getAdapter();
+            if( null != a ) {
+                a.removeStatusListener(statusListener);
+                a.removeDiscoveredDevice(this);
+            }
         }
         super.close();
     }