You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mynewt.apache.org by an...@apache.org on 2021/10/04 10:35:51 UTC

[mynewt-nimble] 02/02: nimble/ll: Add clarification comment

This is an automated email from the ASF dual-hosted git repository.

andk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mynewt-nimble.git

commit c0ef83b072542da7a47f53dc6af20bd108843281
Author: Andrzej Kaczmarek <an...@codecoup.pl>
AuthorDate: Thu Sep 30 23:14:14 2021 +0200

    nimble/ll: Add clarification comment
    
    Just clarify why we should return here instead of 'break'.
---
 nimble/controller/src/ble_ll_scan.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/nimble/controller/src/ble_ll_scan.c b/nimble/controller/src/ble_ll_scan.c
index cf264b0..399ce22 100644
--- a/nimble/controller/src/ble_ll_scan.c
+++ b/nimble/controller/src/ble_ll_scan.c
@@ -2133,6 +2133,9 @@ ble_ll_scan_rx_isr_end(struct os_mbuf *rxpdu, uint8_t crcok)
             rxinfo->flags |= BLE_MBUF_HDR_F_IGNORED;
         }
         ble_ll_state_set(BLE_LL_STATE_STANDBY);
+        /* Return here, we do not want any further processing since it's all
+         * handled in scan_aux.
+         */
         return -1;
 #endif
     default: