From 27cb927d8fb870c360f5273549e2ee6a7639faa2 Mon Sep 17 00:00:00 2001
From: Dan Williams <dcbw@redhat.com>
Date: Thu, 22 May 2014 11:41:58 -0500
Subject: [PATCH] dispatcher: convert action_to_string to a table

Oddly, this increases the compiled+stripped size of the object file
by 24 bytes (8296 -> 8320), but I think it produces more readable code.
---
 src/nm-dispatcher.c | 43 ++++++++++++++++---------------------------
 1 file changed, 16 insertions(+), 27 deletions(-)

diff --git a/src/nm-dispatcher.c b/src/nm-dispatcher.c
index 6ebf427fd0..51b9a8a196 100644
--- a/src/nm-dispatcher.c
+++ b/src/nm-dispatcher.c
@@ -270,36 +270,25 @@ dispatcher_done_cb (DBusGProxy *proxy, DBusGProxyCall *call, gpointer user_data)
 	g_object_unref (proxy);
 }
 
+static const char *action_table[] = {
+	[DISPATCHER_ACTION_HOSTNAME]     = "hostname",
+	[DISPATCHER_ACTION_PRE_UP]       = "pre-up",
+	[DISPATCHER_ACTION_UP]           = "up",
+	[DISPATCHER_ACTION_PRE_DOWN]     = "pre-down",
+	[DISPATCHER_ACTION_DOWN]         = "down",
+	[DISPATCHER_ACTION_VPN_PRE_UP]   = "vpn-pre-up",
+	[DISPATCHER_ACTION_VPN_UP]       = "vpn-up",
+	[DISPATCHER_ACTION_VPN_PRE_DOWN] = "vpn-pre-down",
+	[DISPATCHER_ACTION_VPN_DOWN]     = "vpn-down",
+	[DISPATCHER_ACTION_DHCP4_CHANGE] = "dhcp4-change",
+	[DISPATCHER_ACTION_DHCP6_CHANGE] = "dhcp6-change",
+};
+
 static const char *
 action_to_string (DispatcherAction action)
 {
-	switch (action) {
-	case DISPATCHER_ACTION_HOSTNAME:
-		return "hostname";
-	case DISPATCHER_ACTION_PRE_UP:
-		return "pre-up";
-	case DISPATCHER_ACTION_UP:
-		return "up";
-	case DISPATCHER_ACTION_PRE_DOWN:
-		return "pre-down";
-	case DISPATCHER_ACTION_DOWN:
-		return "down";
-	case DISPATCHER_ACTION_VPN_PRE_UP:
-		return "vpn-pre-up";
-	case DISPATCHER_ACTION_VPN_UP:
-		return "vpn-up";
-	case DISPATCHER_ACTION_VPN_PRE_DOWN:
-		return "vpn-pre-down";
-	case DISPATCHER_ACTION_VPN_DOWN:
-		return "vpn-down";
-	case DISPATCHER_ACTION_DHCP4_CHANGE:
-		return "dhcp4-change";
-	case DISPATCHER_ACTION_DHCP6_CHANGE:
-		return "dhcp6-change";
-	default:
-		break;
-	}
-	g_assert_not_reached ();
+	g_assert (action >= 0 && action < G_N_ELEMENTS (action_table));
+	return action_table[action];
 }
 
 static gboolean