rename getConnectionHandles() to getConnectedHandles() and its signature per review

Author: hathach

libraries/Bluefruit52Lib/src/bluefruit.cpp

@@ -607,19 +607,19 @@ bool AdafruitBluefruit::connected(uint16_t conn_hdl)
   return conn && conn->connected();
 }
 
-void AdafruitBluefruit::getConnectionHandles(uint16_t* connectionHandles, uint8_t maxConnectionHandleCount, uint8_t* actualConnectionHandleCount)
+uint8_t AdafruitBluefruit::getConnectedHandles(uint16_t* hdl_list, uint8_t max_count)
 {
   uint8_t count = 0;
-  for (uint16_t connectionHandle = 0; (connectionHandle < BLE_MAX_CONNECTION) && (count < maxConnectionHandleCount); ++connectionHandle)
+  for (uint16_t hdl = 0; (hdl < BLE_MAX_CONNECTION) && (count < max_count); ++hdl)
   {
-    if (this->connected(connectionHandle))
+    if (this->connected(hdl))
     {
-      connectionHandles[count] = connectionHandle;
+      hdl_list[count] = hdl;
       count++;
     }
   }
 
-  *actualConnectionHandleCount = count;
+  return count;
 }
 
 bool AdafruitBluefruit::disconnect(uint16_t conn_hdl)

libraries/Bluefruit52Lib/src/bluefruit.h

@@ -163,8 +163,7 @@ class AdafruitBluefruit
     uint8_t  connected         (void); // Number of connected
     bool     connected         (uint16_t conn_hdl);
 
-    void getConnectionHandles(uint16_t* connectionHandles, uint8_t maxConnectionHandleCount, uint8_t* actualConnectionHandleCount);
-
+    uint8_t  getConnectedHandles(uint16_t* hdl_list, uint8_t max_count);
     uint16_t connHandle        (void);
 
     // Alias to BLEConnection API()