espressif/esp-nimble
1
0
Fork 0
mirror of https://github.com/espressif/esp-nimble.git synced 2026-08-31 22:30:04 +00:00
Code Issues Packages Projects Releases Wiki Activity

nimble/ll: Remove unnecessary code

Browse Source 
While reviewing code noticed that there was no need to check for
not receiving a terminate indication as that case is already
checked and could thus never be true. Removed the code that was
not needed.
This commit is contained in:
Will San Filippo
2019-10-03 17:41:27 -07:00
parent 125974613b
commit d6710d6a03
1 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
nimble/controller/src/ble_ll_conn.c
+2 -3
View File
@@ -811,7 +811,7 @@ ble_ll_conn_tx_data_pdu(struct ble_ll_conn_sm *connsm)
if (connsm->csmflags.cfbit.terminate_ind_rxd) {
/* We just received terminate indication.
* Just sent empty packet as an ACK
* Just send empty packet as an ACK
*/
CONN_F_EMPTY_PDU_TXD(connsm) = 1;
goto conn_tx_pdu;
@@ -918,8 +918,7 @@ ble_ll_conn_tx_data_pdu(struct ble_ll_conn_sm *connsm)
* Set the more data data flag if we have more data to send and we
* have not been asked to terminate
*/
if ((nextpkthdr || ((cur_offset + cur_txlen) < pktlen)) &&
!connsm->csmflags.cfbit.terminate_ind_rxd) {
if (nextpkthdr || ((cur_offset + cur_txlen) < pktlen)) {
/* Get next event time */
next_event_time = ble_ll_conn_get_next_sched_time(connsm);