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

Hasnain Virk · Hasnain Virk · commit 9e4487c85099 · 2018-10-16T12:23:21.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
@@ -646,7 +646,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
`@@ -646,7 +646,6 @@ void LoRaWANStack::post_process_tx_with_reception()`
`646`	`646`	`void LoRaWANStack::post_process_tx_no_reception()`
`647`	`647`	`{`
`648`	`648`	`if (_loramac.get_mcps_confirmation()->req_type == MCPS_CONFIRMED) {`
`649`		`- _loramac.post_process_mcps_req();`
`650`	`649`	`if (_loramac.continue_sending_process()) {`
`651`	`650`	`_ctrl_flags &= ~TX_DONE_FLAG;`
`652`	`651`	`_ctrl_flags &= ~RETRY_EXHAUSTED_FLAG;`