DBTAdapter::startDiscoveryBackground() check isEnabled(); DBTAdapter.stopDiscovery() shall always call DBTAdapter::stopDiscovery()

DBTAdapter::startDiscoveryBackground() check isEnabled() same as startDiscovery(), adapter needs to be powered on.

Added note in stopDiscovery(), as we don't check isEnabled.
Here we shall pass through to have at least the discovery state updated and listeners notified.

+++

DBTAdapter.stopDiscovery() shall always call DBTAdapter::stopDiscovery(),
same as DBTAdapter.startDiscovery().

Just in case 'isDiscovery' state differs from native implementation,
it shall be corrected here. (Robustness)

1 files changed, 2 insertions, 4 deletions

diff --git a/java/direct_bt/tinyb/DBTAdapter.java b/java/direct_bt/tinyb/DBTAdapter.java
index 549de9bf..d3e42fab 100644
--- a/java/direct_bt/tinyb/DBTAdapter.java
+++ b/java/direct_bt/tinyb/DBTAdapter.java
@@ -342,10 +342,8 @@ public class DBTAdapter extends DBTObject implements BluetoothAdapter
     @Override
     public boolean stopDiscovery() throws BluetoothException {
         synchronized( discoveryLock ) {
-            if( isDiscovering.get() ) {
-                return stopDiscoveryImpl();  // event callbacks will be generated by implementation
-            }
-            return true;
+            // Ignoring 'isDiscovering', be consistent with startDiscovery
+            return stopDiscoveryImpl();  // event callbacks will be generated by implementation
         }
     }
     private native boolean stopDiscoveryImpl() throws BluetoothException;