rymanluk commented on a change in pull request #815:
URL: https://github.com/apache/mynewt-nimble/pull/815#discussion_r424777371



##########
File path: nimble/controller/src/ble_ll_conn.c
##########
@@ -3598,8 +3769,10 @@ int
 ble_ll_conn_rx_isr_end(uint8_t *rxbuf, struct ble_mbuf_hdr *rxhdr)
 {
     int rc;
-    int is_ctrl;
+    uint8_t is_ctrl;
+    uint8_t is_data;

Review comment:
       looks like `is_data` is used only when
   MYNEWT_VAL(BLE_LL_CFG_FEAT_CTRL_TO_HOST_FLOW_CONTROL  is set.




----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to