Change in osmo-bsc[master]: cosmetic: gscon: drop odd use of OSMO_STRINGIFY

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
Fri Jun 8 16:16:48 UTC 2018


Harald Welte has submitted this change and it was merged. ( https://gerrit.osmocom.org/9292 )

Change subject: cosmetic: gscon: drop odd use of OSMO_STRINGIFY
......................................................................

cosmetic: gscon: drop odd use of OSMO_STRINGIFY

Do not invoke OSMO_STRINGIFY() with arbitrary names, just quote instead.

The idea was that OSMO_STRINGIFY() avoids typos by ensuring well-defined names
are stringified, but this highlights that OSMO_STRINGIFY() is in fact usable
with completely arbitrary arguments and actually lacks the validation part :/

Change-Id: I458cd2cd0d6ddb0e6db3bb8d546a20336ae8c5f1
---
M src/osmo-bsc/bsc_subscr_conn_fsm.c
1 file changed, 14 insertions(+), 14 deletions(-)

Approvals:
  Neels Hofmeyr: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/osmo-bsc/bsc_subscr_conn_fsm.c b/src/osmo-bsc/bsc_subscr_conn_fsm.c
index 2ae4b33..d2d2095 100644
--- a/src/osmo-bsc/bsc_subscr_conn_fsm.c
+++ b/src/osmo-bsc/bsc_subscr_conn_fsm.c
@@ -820,19 +820,19 @@
 
 static const struct osmo_fsm_state gscon_fsm_states[] = {
 	[ST_INIT] = {
-		.name = OSMO_STRINGIFY(INIT),
+		.name = "INIT",
 		.in_event_mask = S(GSCON_EV_A_CONN_REQ) | S(GSCON_EV_A_CONN_IND),
 		.out_state_mask = S(ST_WAIT_CC),
 		.action = gscon_fsm_init,
 	 },
 	[ST_WAIT_CC] = {
-		.name = OSMO_STRINGIFY(WAIT_CC),
+		.name = "WAIT_CC",
 		.in_event_mask = S(GSCON_EV_A_CONN_CFM),
 		.out_state_mask = S(ST_ACTIVE),
 		.action = gscon_fsm_wait_cc,
 	},
 	[ST_ACTIVE] = {
-		.name = OSMO_STRINGIFY(ACTIVE),
+		.name = "ACTIVE",
 		.in_event_mask = EV_TRANSPARENT_SCCP | S(GSCON_EV_A_ASSIGNMENT_CMD) |
 				 S(GSCON_EV_A_HO_REQ) | S(GSCON_EV_HO_START),
 		.out_state_mask = S(ST_CLEARING) | S(ST_WAIT_CRCX_BTS) | S(ST_WAIT_ASS_CMPL) |
@@ -840,31 +840,31 @@
 		.action = gscon_fsm_active,
 	},
 	[ST_WAIT_CRCX_BTS] = {
-		.name = OSMO_STRINGIFY(WAIT_CRCX_BTS),
+		.name = "WAIT_CRCX_BTS",
 		.in_event_mask = EV_TRANSPARENT_SCCP | S(GSCON_EV_MGW_CRCX_RESP_BTS),
 		.out_state_mask = S(ST_ACTIVE) | S(ST_WAIT_ASS_CMPL),
 		.action = gscon_fsm_wait_crcx_bts,
 	},
 	[ST_WAIT_ASS_CMPL] = {
-		.name = OSMO_STRINGIFY(WAIT_ASS_CMPL),
+		.name = "WAIT_ASS_CMPL",
 		.in_event_mask = EV_TRANSPARENT_SCCP | S(GSCON_EV_RR_ASS_COMPL) | S(GSCON_EV_RR_ASS_FAIL),
 		.out_state_mask = S(ST_ACTIVE) | S(ST_WAIT_MDCX_BTS),
 		.action = gscon_fsm_wait_ass_cmpl,
 	},
 	[ST_WAIT_MDCX_BTS] = {
-		.name = OSMO_STRINGIFY(WAIT_MDCX_BTS),
+		.name = "WAIT_MDCX_BTS",
 		.in_event_mask = EV_TRANSPARENT_SCCP | S(GSCON_EV_MGW_MDCX_RESP_BTS),
 		.out_state_mask = S(ST_ACTIVE) | S(ST_WAIT_CRCX_MSC),
 		.action = gscon_fsm_wait_mdcx_bts,
 	},
 	[ST_WAIT_CRCX_MSC] = {
-		.name = OSMO_STRINGIFY(WAIT_CRCX_MSC),
+		.name = "WAIT_CRCX_MSC",
 		.in_event_mask = EV_TRANSPARENT_SCCP | S(GSCON_EV_MGW_CRCX_RESP_MSC),
 		.out_state_mask = S(ST_ACTIVE),
 		.action = gscon_fsm_wait_crcx_msc,
 	},
 	[ST_CLEARING] = {
-		.name = OSMO_STRINGIFY(CLEARING),
+		.name = "CLEARING",
 		.in_event_mask = S(GSCON_EV_RSL_CLEAR_COMPL),
 		.action = gscon_fsm_clearing,
 	 },
@@ -873,27 +873,27 @@
 	 * program flow in handover_logic.c a bit and handle some of the logic
 	 * here? */
 	[ST_WAIT_MT_HO_ACC] = {
-		.name = OSMO_STRINGIFY(WAIT_MT_HO_ACC),
+		.name = "WAIT_MT_HO_ACC",
 	},
 	[ST_WAIT_MT_HO_COMPL] = {
-		 .name = OSMO_STRINGIFY(WAIT_MT_HO_COMPL),
+		 .name = "WAIT_MT_HO_COMPL",
 	},
 	[ST_WAIT_MO_HO_CMD] = {
-		.name = OSMO_STRINGIFY(WAIT_MO_HO_CMD),
+		.name = "WAIT_MO_HO_CMD",
 	},
 	[ST_MO_HO_PROCEEDING] = {
-		 .name = OSMO_STRINGIFY(MO_HO_PROCEEDING),
+		 .name = "MO_HO_PROCEEDING",
 	},
 
 	/* Internal handover */
 	[ST_WAIT_HO_COMPL] = {
-		.name = OSMO_STRINGIFY(WAIT_HO_COMPL),
+		.name = "WAIT_HO_COMPL",
 		.in_event_mask = S(GSCON_EV_HO_COMPL) | S(GSCON_EV_HO_FAIL) | S(GSCON_EV_HO_TIMEOUT),
 		.out_state_mask = S(ST_ACTIVE) | S(ST_WAIT_MDCX_BTS_HO) | S(ST_CLEARING),
 		.action = gscon_fsm_wait_ho_compl,
 	},
 	[ST_WAIT_MDCX_BTS_HO] = {
-		.name = OSMO_STRINGIFY(WAIT_MDCX_BTS_HO),
+		.name = "WAIT_MDCX_BTS_HO",
 		.in_event_mask = EV_TRANSPARENT_SCCP | S(GSCON_EV_MGW_MDCX_RESP_BTS),
 		.action = gscon_fsm_wait_mdcx_bts_ho,
 		.out_state_mask = S(ST_ACTIVE),

-- 
To view, visit https://gerrit.osmocom.org/9292
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I458cd2cd0d6ddb0e6db3bb8d546a20336ae8c5f1
Gerrit-Change-Number: 9292
Gerrit-PatchSet: 3
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20180608/3ebc2a78/attachment.htm>


More information about the gerrit-log mailing list