TX post-process for CONFIRMED UL in no-reception case

Hasnain Virk · Hasnain Virk · commit 19e87ceab9f5 · 2018-09-21T15:55:30.000+03:00
The idea behind the method post_process_no_reception() is to post
process any outgoing TX but we shouldn't do that if a CONFIRMED message
is outgoing and there are still some retries left.
diff --git a/features/lorawan/LoRaWANStack.cpp b/features/lorawan/LoRaWANStack.cpp
@@ -647,7 +647,6 @@ void LoRaWANStack::post_process_tx_with_reception()
 void LoRaWANStack::post_process_tx_no_reception()
 {
     if (_loramac.get_mcps_confirmation()->req_type == MCPS_CONFIRMED) {
-        _loramac.post_process_mcps_req();
         if (_loramac.continue_sending_process()) {
             _ctrl_flags &= ~TX_DONE_FLAG;
             _ctrl_flags &= ~RETRY_EXHAUSTED_FLAG;

Original file line number	Diff line number	Diff line change
`@@ -647,7 +647,6 @@ void LoRaWANStack::post_process_tx_with_reception()`
`647`	`647`	`void LoRaWANStack::post_process_tx_no_reception()`
`648`	`648`	`{`
`649`	`649`	`if (_loramac.get_mcps_confirmation()->req_type == MCPS_CONFIRMED) {`
`650`		`- _loramac.post_process_mcps_req();`
`651`	`650`	`if (_loramac.continue_sending_process()) {`
`652`	`651`	`_ctrl_flags &= ~TX_DONE_FLAG;`
`653`	`652`	`_ctrl_flags &= ~RETRY_EXHAUSTED_FLAG;`