*: clang-SA friendly switch-enum-return-string

clang-19's SA complains about unused initializers for this kind of
"switch (enum) { return string }" kind of code.  Use direct string
return values to avoid the issue.

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

bgpd/bgp_fsm.c

@@ -2776,99 +2776,72 @@ int bgp_gr_update_all(struct bgp *bgp, enum global_gr_command global_gr_cmd)
 
 const char *print_peer_gr_mode(enum peer_mode pr_mode)
 {
-	const char *peer_gr_mode = NULL;
-
 	switch (pr_mode) {
 	case PEER_HELPER:
-		peer_gr_mode = "PEER_HELPER";
+		return "PEER_HELPER";
-		break;
 	case PEER_GR:
-		peer_gr_mode = "PEER_GR";
+		return "PEER_GR";
-		break;
 	case PEER_DISABLE:
-		peer_gr_mode = "PEER_DISABLE";
+		return "PEER_DISABLE";
-		break;
 	case PEER_INVALID:
-		peer_gr_mode = "PEER_INVALID";
+		return "PEER_INVALID";
-		break;
 	case PEER_GLOBAL_INHERIT:
-		peer_gr_mode = "PEER_GLOBAL_INHERIT";
+		return "PEER_GLOBAL_INHERIT";
-		break;
 	}
 
-	return peer_gr_mode;
+	return NULL;
 }
 
 const char *print_peer_gr_cmd(enum peer_gr_command pr_gr_cmd)
 {
-	const char *peer_gr_cmd = NULL;
-
 	switch (pr_gr_cmd) {
 	case PEER_GR_CMD:
-		peer_gr_cmd = "PEER_GR_CMD";
+		return "PEER_GR_CMD";
-		break;
 	case NO_PEER_GR_CMD:
-		peer_gr_cmd = "NO_PEER_GR_CMD";
+		return "NO_PEER_GR_CMD";
-		break;
 	case PEER_DISABLE_CMD:
-		peer_gr_cmd = "PEER_DISABLE_GR_CMD";
+		return "PEER_DISABLE_GR_CMD";
-		break;
 	case NO_PEER_DISABLE_CMD:
-		peer_gr_cmd = "NO_PEER_DISABLE_GR_CMD";
+		return "NO_PEER_DISABLE_GR_CMD";
-		break;
 	case PEER_HELPER_CMD:
-		peer_gr_cmd = "PEER_HELPER_CMD";
+		return "PEER_HELPER_CMD";
-		break;
 	case NO_PEER_HELPER_CMD:
-		peer_gr_cmd = "NO_PEER_HELPER_CMD";
+		return "NO_PEER_HELPER_CMD";
-		break;
 	}
 
-	return peer_gr_cmd;
+	return NULL;
 }
 
 const char *print_global_gr_mode(enum global_mode gl_mode)
 {
-	const char *global_gr_mode = "???";
-
 	switch (gl_mode) {
 	case GLOBAL_HELPER:
-		global_gr_mode = "GLOBAL_HELPER";
+		return "GLOBAL_HELPER";
-		break;
 	case GLOBAL_GR:
-		global_gr_mode = "GLOBAL_GR";
+		return "GLOBAL_GR";
-		break;
 	case GLOBAL_DISABLE:
-		global_gr_mode = "GLOBAL_DISABLE";
+		return "GLOBAL_DISABLE";
-		break;
 	case GLOBAL_INVALID:
-		global_gr_mode = "GLOBAL_INVALID";
+		return "GLOBAL_INVALID";
-		break;
 	}
 
-	return global_gr_mode;
+	return "???";
 }
 
 const char *print_global_gr_cmd(enum global_gr_command gl_gr_cmd)
 {
-	const char *global_gr_cmd = NULL;
-
 	switch (gl_gr_cmd) {
 	case GLOBAL_GR_CMD:
-		global_gr_cmd = "GLOBAL_GR_CMD";
+		return "GLOBAL_GR_CMD";
-		break;
 	case NO_GLOBAL_GR_CMD:
-		global_gr_cmd = "NO_GLOBAL_GR_CMD";
+		return "NO_GLOBAL_GR_CMD";
-		break;
 	case GLOBAL_DISABLE_CMD:
-		global_gr_cmd = "GLOBAL_DISABLE_CMD";
+		return "GLOBAL_DISABLE_CMD";
-		break;
 	case NO_GLOBAL_DISABLE_CMD:
-		global_gr_cmd = "NO_GLOBAL_DISABLE_CMD";
+		return "NO_GLOBAL_DISABLE_CMD";
-		break;
 	}
 
-	return global_gr_cmd;
+	return NULL;
 }
 
 enum global_mode bgp_global_gr_mode_get(struct bgp *bgp)

lib/zclient.h

@@ -251,21 +251,16 @@ enum zebra_error_types {
 
 static inline const char *zebra_error_type2str(enum zebra_error_types type)
 {
-	const char *ret = "UNKNOWN";
-
 	switch (type) {
 	case ZEBRA_UNKNOWN_ERROR:
-		ret = "ZEBRA_UNKNOWN_ERROR";
+		return "ZEBRA_UNKNOWN_ERROR";
-		break;
 	case ZEBRA_NO_VRF:
-		ret = "ZEBRA_NO_VRF";
+		return "ZEBRA_NO_VRF";
-		break;
 	case ZEBRA_INVALID_MSG_TYPE:
-		ret = "ZEBRA_INVALID_MSG_TYPE";
+		return "ZEBRA_INVALID_MSG_TYPE";
-		break;
 	}
 
-	return ret;
+	return "UNKNOWN";
 }
 
 struct redist_proto {
@@ -780,69 +775,51 @@ enum zclient_send_status {
 static inline const char *
 zapi_nhg_notify_owner2str(enum zapi_nhg_notify_owner note)
 {
-	const char *ret = "UNKNOWN";
-
 	switch (note) {
 	case ZAPI_NHG_FAIL_INSTALL:
-		ret = "ZAPI_NHG_FAIL_INSTALL";
+		return "ZAPI_NHG_FAIL_INSTALL";
-		break;
 	case ZAPI_NHG_INSTALLED:
-		ret = "ZAPI_NHG_INSTALLED";
+		return "ZAPI_NHG_INSTALLED";
-		break;
 	case ZAPI_NHG_REMOVE_FAIL:
-		ret = "ZAPI_NHG_REMOVE_FAIL";
+		return "ZAPI_NHG_REMOVE_FAIL";
-		break;
 	case ZAPI_NHG_REMOVED:
-		ret = "ZAPI_NHG_REMOVED";
+		return "ZAPI_NHG_REMOVED";
-		break;
 	}
 
-	return ret;
+	return "UNKNOWN";
 }
 
 static inline const char *
 zapi_rule_notify_owner2str(enum zapi_rule_notify_owner note)
 {
-	const char *ret = "UNKNOWN";
-
 	switch (note) {
 	case ZAPI_RULE_FAIL_INSTALL:
-		ret = "ZAPI_RULE_FAIL_INSTALL";
+		return "ZAPI_RULE_FAIL_INSTALL";
-		break;
 	case ZAPI_RULE_INSTALLED:
-		ret = "ZAPI_RULE_INSTALLED";
+		return "ZAPI_RULE_INSTALLED";
-		break;
 	case ZAPI_RULE_FAIL_REMOVE:
-		ret = "ZAPI_RULE_FAIL_REMOVE";
+		return "ZAPI_RULE_FAIL_REMOVE";
-		break;
 	case ZAPI_RULE_REMOVED:
-		ret = "ZAPI_RULE_REMOVED";
+		return "ZAPI_RULE_REMOVED";
-		break;
 	}
 
-	return ret;
+	return "UNKNOWN";
 }
 
 static inline const char *zapi_srv6_sid_notify2str(enum zapi_srv6_sid_notify note)
 {
-	const char *ret = "UNKNOWN";
-
 	switch (note) {
 	case ZAPI_SRV6_SID_FAIL_ALLOC:
-		ret = "ZAPI_SRV6_SID_FAIL_ALLOC";
+		return "ZAPI_SRV6_SID_FAIL_ALLOC";
-		break;
 	case ZAPI_SRV6_SID_ALLOCATED:
-		ret = "ZAPI_SRV6_SID_ALLOCATED";
+		return "ZAPI_SRV6_SID_ALLOCATED";
-		break;
 	case ZAPI_SRV6_SID_FAIL_RELEASE:
-		ret = "ZAPI_SRV6_SID_FAIL_RELEASE";
+		return "ZAPI_SRV6_SID_FAIL_RELEASE";
-		break;
 	case ZAPI_SRV6_SID_RELEASED:
-		ret = "ZAPI_SRV6_SID_RELEASED";
+		return "ZAPI_SRV6_SID_RELEASED";
-		break;
 	}
 
-	return ret;
+	return "UNKNOWN";
 }
 
 /* Zebra MAC types */

zebra/zebra_dplane.c

@@ -1046,144 +1046,102 @@ enum dplane_op_e dplane_ctx_get_op(const struct zebra_dplane_ctx *ctx)
 
 const char *dplane_op2str(enum dplane_op_e op)
 {
-	const char *ret = "UNKNOWN";
-
 	switch (op) {
 	case DPLANE_OP_NONE:
-		ret = "NONE";
+		return "NONE";
-		break;
 
 	/* Route update */
 	case DPLANE_OP_ROUTE_INSTALL:
-		ret = "ROUTE_INSTALL";
+		return "ROUTE_INSTALL";
-		break;
 	case DPLANE_OP_ROUTE_UPDATE:
-		ret = "ROUTE_UPDATE";
+		return "ROUTE_UPDATE";
-		break;
 	case DPLANE_OP_ROUTE_DELETE:
-		ret = "ROUTE_DELETE";
+		return "ROUTE_DELETE";
-		break;
 	case DPLANE_OP_ROUTE_NOTIFY:
-		ret = "ROUTE_NOTIFY";
+		return "ROUTE_NOTIFY";
-		break;
 
 	/* Nexthop update */
 	case DPLANE_OP_NH_INSTALL:
-		ret = "NH_INSTALL";
+		return "NH_INSTALL";
-		break;
 	case DPLANE_OP_NH_UPDATE:
-		ret = "NH_UPDATE";
+		return "NH_UPDATE";
-		break;
 	case DPLANE_OP_NH_DELETE:
-		ret = "NH_DELETE";
+		return "NH_DELETE";
-		break;
 
 	case DPLANE_OP_LSP_INSTALL:
-		ret = "LSP_INSTALL";
+		return "LSP_INSTALL";
-		break;
 	case DPLANE_OP_LSP_UPDATE:
-		ret = "LSP_UPDATE";
+		return "LSP_UPDATE";
-		break;
 	case DPLANE_OP_LSP_DELETE:
-		ret = "LSP_DELETE";
+		return "LSP_DELETE";
-		break;
 	case DPLANE_OP_LSP_NOTIFY:
-		ret = "LSP_NOTIFY";
+		return "LSP_NOTIFY";
-		break;
 
 	case DPLANE_OP_PW_INSTALL:
-		ret = "PW_INSTALL";
+		return "PW_INSTALL";
-		break;
 	case DPLANE_OP_PW_UNINSTALL:
-		ret = "PW_UNINSTALL";
+		return "PW_UNINSTALL";
-		break;
 
 	case DPLANE_OP_SYS_ROUTE_ADD:
-		ret = "SYS_ROUTE_ADD";
+		return "SYS_ROUTE_ADD";
-		break;
 	case DPLANE_OP_SYS_ROUTE_DELETE:
-		ret = "SYS_ROUTE_DEL";
+		return "SYS_ROUTE_DEL";
-		break;
 
 	case DPLANE_OP_BR_PORT_UPDATE:
-		ret = "BR_PORT_UPDATE";
+		return "BR_PORT_UPDATE";
-		break;
 
 	case DPLANE_OP_ADDR_INSTALL:
-		ret = "ADDR_INSTALL";
+		return "ADDR_INSTALL";
-		break;
 	case DPLANE_OP_ADDR_UNINSTALL:
-		ret = "ADDR_UNINSTALL";
+		return "ADDR_UNINSTALL";
-		break;
 
 	case DPLANE_OP_MAC_INSTALL:
-		ret = "MAC_INSTALL";
+		return "MAC_INSTALL";
-		break;
 	case DPLANE_OP_MAC_DELETE:
-		ret = "MAC_DELETE";
+		return "MAC_DELETE";
-		break;
 
 	case DPLANE_OP_NEIGH_INSTALL:
-		ret = "NEIGH_INSTALL";
+		return "NEIGH_INSTALL";
-		break;
 	case DPLANE_OP_NEIGH_UPDATE:
-		ret = "NEIGH_UPDATE";
+		return "NEIGH_UPDATE";
-		break;
 	case DPLANE_OP_NEIGH_DELETE:
-		ret = "NEIGH_DELETE";
+		return "NEIGH_DELETE";
-		break;
 	case DPLANE_OP_VTEP_ADD:
-		ret = "VTEP_ADD";
+		return "VTEP_ADD";
-		break;
 	case DPLANE_OP_VTEP_DELETE:
-		ret = "VTEP_DELETE";
+		return "VTEP_DELETE";
-		break;
 
 	case DPLANE_OP_RULE_ADD:
-		ret = "RULE_ADD";
+		return "RULE_ADD";
-		break;
 	case DPLANE_OP_RULE_DELETE:
-		ret = "RULE_DELETE";
+		return "RULE_DELETE";
-		break;
 	case DPLANE_OP_RULE_UPDATE:
-		ret = "RULE_UPDATE";
+		return "RULE_UPDATE";
-		break;
 
 	case DPLANE_OP_NEIGH_DISCOVER:
-		ret = "NEIGH_DISCOVER";
+		return "NEIGH_DISCOVER";
-		break;
 
 	case DPLANE_OP_IPTABLE_ADD:
-		ret = "IPTABLE_ADD";
+		return "IPTABLE_ADD";
-		break;
 	case DPLANE_OP_IPTABLE_DELETE:
-		ret = "IPTABLE_DELETE";
+		return "IPTABLE_DELETE";
-		break;
 	case DPLANE_OP_IPSET_ADD:
-		ret = "IPSET_ADD";
+		return "IPSET_ADD";
-		break;
 	case DPLANE_OP_IPSET_DELETE:
-		ret = "IPSET_DELETE";
+		return "IPSET_DELETE";
-		break;
 	case DPLANE_OP_IPSET_ENTRY_ADD:
-		ret = "IPSET_ENTRY_ADD";
+		return "IPSET_ENTRY_ADD";
-		break;
 	case DPLANE_OP_IPSET_ENTRY_DELETE:
-		ret = "IPSET_ENTRY_DELETE";
+		return "IPSET_ENTRY_DELETE";
-		break;
 	case DPLANE_OP_NEIGH_IP_INSTALL:
-		ret = "NEIGH_IP_INSTALL";
+		return "NEIGH_IP_INSTALL";
-		break;
 	case DPLANE_OP_NEIGH_IP_DELETE:
-		ret = "NEIGH_IP_DELETE";
+		return "NEIGH_IP_DELETE";
-		break;
 	case DPLANE_OP_NEIGH_TABLE_UPDATE:
-		ret = "NEIGH_TABLE_UPDATE";
+		return "NEIGH_TABLE_UPDATE";
-		break;
 
 	case DPLANE_OP_GRE_SET:
-		ret = "GRE_SET";
+		return "GRE_SET";
-		break;
 
 	case DPLANE_OP_INTF_ADDR_ADD:
 		return "INTF_ADDR_ADD";
@@ -1195,72 +1153,53 @@ const char *dplane_op2str(enum dplane_op_e op)
 		return "INTF_NETCONFIG";
 
 	case DPLANE_OP_INTF_INSTALL:
-		ret = "INTF_INSTALL";
+		return "INTF_INSTALL";
-		break;
 	case DPLANE_OP_INTF_UPDATE:
-		ret = "INTF_UPDATE";
+		return "INTF_UPDATE";
-		break;
 	case DPLANE_OP_INTF_DELETE:
-		ret = "INTF_DELETE";
+		return "INTF_DELETE";
-		break;
 
 	case DPLANE_OP_TC_QDISC_INSTALL:
-		ret = "TC_QDISC_INSTALL";
+		return "TC_QDISC_INSTALL";
-		break;
 	case DPLANE_OP_TC_QDISC_UNINSTALL:
-		ret = "TC_QDISC_UNINSTALL";
+		return "TC_QDISC_UNINSTALL";
-		break;
 	case DPLANE_OP_TC_CLASS_ADD:
-		ret = "TC_CLASS_ADD";
+		return "TC_CLASS_ADD";
-		break;
 	case DPLANE_OP_TC_CLASS_DELETE:
-		ret = "TC_CLASS_DELETE";
+		return "TC_CLASS_DELETE";
-		break;
 	case DPLANE_OP_TC_CLASS_UPDATE:
-		ret = "TC_CLASS_UPDATE";
+		return "TC_CLASS_UPDATE";
-		break;
 	case DPLANE_OP_TC_FILTER_ADD:
-		ret = "TC_FILTER_ADD";
+		return "TC_FILTER_ADD";
-		break;
 	case DPLANE_OP_TC_FILTER_DELETE:
-		ret = "TC_FILTER_DELETE";
+		return "TC_FILTER_DELETE";
-		break;
 	case DPLANE_OP_TC_FILTER_UPDATE:
-		ret = "TC__FILTER_UPDATE";
+		return "TC__FILTER_UPDATE";
-		break;
 	case DPLANE_OP_STARTUP_STAGE:
-		ret = "STARTUP_STAGE";
+		return "STARTUP_STAGE";
-		break;
 
 	case DPLANE_OP_SRV6_ENCAP_SRCADDR_SET:
-		ret = "SRV6_ENCAP_SRCADDR_SET";
+		return "SRV6_ENCAP_SRCADDR_SET";
-		break;
 
 	case DPLANE_OP_VLAN_INSTALL:
-		ret = "NEW_VLAN";
+		return "NEW_VLAN";
-		break;
 	}
 
-	return ret;
+	return "UNKNOWN";
 }
 
 const char *dplane_res2str(enum zebra_dplane_result res)
 {
-	const char *ret = "<Unknown>";
-
 	switch (res) {
 	case ZEBRA_DPLANE_REQUEST_FAILURE:
-		ret = "FAILURE";
+		return "FAILURE";
-		break;
 	case ZEBRA_DPLANE_REQUEST_QUEUED:
-		ret = "QUEUED";
+		return "QUEUED";
-		break;
 	case ZEBRA_DPLANE_REQUEST_SUCCESS:
-		ret = "SUCCESS";
+		return "SUCCESS";
-		break;
 	}
 
-	return ret;
+	return "<Unknown>";
 }
 
 void dplane_ctx_set_dest(struct zebra_dplane_ctx *ctx,