[MERGED] osmo-bsc[master]: Cosmetic: fix missing semicolon after osmo-assert

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Harald Welte gerrit-no-reply at lists.osmocom.org
Thu Mar 22 12:55:51 UTC 2018


Harald Welte has submitted this change and it was merged.

Change subject: Cosmetic: fix missing semicolon after osmo-assert
......................................................................


Cosmetic: fix missing semicolon after osmo-assert

- add the missing semicolong,
- correct wrong indentation caused by the missing semicolon

Change-Id: I9a1144b70ab82156175c6cdee32a6d22b4b53539
Related: OS#2823
---
M src/libbsc/bsc_subscr_conn_fsm.c
1 file changed, 3 insertions(+), 3 deletions(-)

Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/libbsc/bsc_subscr_conn_fsm.c b/src/libbsc/bsc_subscr_conn_fsm.c
index 5154fb2..dc088a1 100644
--- a/src/libbsc/bsc_subscr_conn_fsm.c
+++ b/src/libbsc/bsc_subscr_conn_fsm.c
@@ -866,9 +866,9 @@
 		/* Note: An MGW connection die per definition at any time.
 		 * However, if it dies during the assignment we must return
 		 * with an assignment failure */
-		OSMO_ASSERT(fi->state != ST_INIT && fi->state != ST_WAIT_CC)
-		    if (fi->state == ST_WAIT_CRCX_BTS || fi->state == ST_WAIT_ASS_CMPL || fi->state == ST_WAIT_MDCX_BTS
-			|| fi->state == ST_WAIT_CRCX_MSC) {
+		OSMO_ASSERT(fi->state != ST_INIT && fi->state != ST_WAIT_CC);
+		if (fi->state == ST_WAIT_CRCX_BTS || fi->state == ST_WAIT_ASS_CMPL || fi->state == ST_WAIT_MDCX_BTS
+		    || fi->state == ST_WAIT_CRCX_MSC) {
 			resp = gsm0808_create_assignment_failure(GSM0808_CAUSE_EQUIPMENT_FAILURE, NULL);
 			sigtran_send(conn, resp, fi);
 			osmo_fsm_inst_state_chg(fi, ST_ACTIVE, 0, 0);

-- 
To view, visit https://gerrit.osmocom.org/7425
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I9a1144b70ab82156175c6cdee32a6d22b4b53539
Gerrit-PatchSet: 2
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Owner: dexter <pmaier at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder



More information about the gerrit-log mailing list