Merge pull request #7195 from jarvte/fixing_easycellular_release_version

adbridge · web-flow · commit 3e2e6a714a3b · 2018-06-15T11:13:34.000+01:00
Cellular: Semaphore wasn't released in easycellular release build.
diff --git a/features/cellular/easy_cellular/EasyCellularConnection.cpp b/features/cellular/easy_cellular/EasyCellularConnection.cpp
@@ -42,7 +42,7 @@ bool EasyCellularConnection::cellular_status(int state, int next_state)
 
     if (_target_state == state) {
         tr_info("Target state reached: %s", _cellularConnectionFSM->get_state_string(_target_state));
-        MBED_ASSERT(_cellularSemaphore.release() == osOK);
+        (void)_cellularSemaphore.release();
         return false; // return false -> state machine is halted
     }
     return true;

Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,7 @@ bool EasyCellularConnection::cellular_status(int state, int next_state)`
`42`	`42`
`43`	`43`	`if (_target_state == state) {`
`44`	`44`	`tr_info("Target state reached: %s", _cellularConnectionFSM->get_state_string(_target_state));`
`45`		`- MBED_ASSERT(_cellularSemaphore.release() == osOK);`
	`45`	`+ (void)_cellularSemaphore.release();`
`46`	`46`	`return false; // return false -> state machine is halted`
`47`	`47`	`}`
`48`	`48`	`return true;`