GATTHandler::disconnect(..): Expose full parametrized variant only, adding 'bool disconnectDevice'

disconnectDevice shall not always occure w/ GATTHandler::disconnect, i.e.:
- when issuing DBTDevice::disconnectGATT() as it is already called from DBTDevice::disconnect(..)
- via GATTHandler dtor

This avoid unnecessary recursions, which should fail anyways as first disconnect
shall make subsequent disconnect calls return early.

1 files changed, 9 insertions, 5 deletions

diff --git a/api/direct_bt/GATTHandler.hpp b/api/direct_bt/GATTHandler.hpp
index 336f0371..67c932fd 100644
--- a/api/direct_bt/GATTHandler.hpp
+++ b/api/direct_bt/GATTHandler.hpp
@@ -122,8 +122,6 @@ namespace direct_bt {
              */
             uint16_t exchangeMTU(const uint16_t clientMaxMTU=number(Defaults::MAX_ATT_MTU));
 
-            bool disconnect(const bool ioErrorCause);
-
         public:
             GATTHandler(const std::shared_ptr<DBTDevice> & device, const int replyTimeoutMS = number(Defaults::L2CAP_COMMAND_REPLY_TIMEOUT));
 
@@ -137,9 +135,15 @@ namespace direct_bt {
              * See getServerMTU() and getUsedMTU(), the latter is in use.
              */
             bool connect();
-            bool disconnect() {
-                return disconnect(false /* ioErrorCause */);
-            }
+
+            /**
+             * Disconnect this GATTHandler and optionally the associated device
+             * @param disconnectDevice if true, associated device will also be disconnected, otherwise not.
+             * @param ioErrorCause if true, reason for disconnection is an IO error
+             * @return true if successful, otherwise false
+             */
+            bool disconnect(const bool disconnectDevice, const bool ioErrorCause);
+
             bool isOpen() const { return L2CAPComm::State::Disconnected < state && l2cap.isOpen(); }
 
             uint16_t getServerMTU() const { return serverMTU; }