sjanc commented on a change in pull request #744: [WIP] Add support for 
Enhanced LE CoC as per BT 5.2 
URL: https://github.com/apache/mynewt-nimble/pull/744#discussion_r377674175
 
 

 ##########
 File path: nimble/host/src/ble_hs_conn.c
 ##########
 @@ -88,6 +88,24 @@ ble_hs_conn_chan_find_by_dcid(struct ble_hs_conn *conn, 
uint16_t cid)
     return NULL;
 }
 
+bool
+ble_hs_conn_chan_exist(struct ble_hs_conn *conn, struct ble_l2cap_chan *chan)
+{
+#if !NIMBLE_BLE_CONNECT
+    return NULL;
 
 Review comment:
   false
   
   also please follow
   
    #ifdef foo
      foo()
    #else 
     return NULL
     #endif
   
   
   otherwise static analysis tools gives false positives

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to