From: Denys Vlasenko <vda.linux@googlemail.com>
Date: Sat, 24 Jul 2010 21:26:58 +0000 (+0200)
Subject: ip: fix "ip r" breakage
X-Git-Tag: 1_18_0~392
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=e0a622093c6c1f3e3a1f58713118395994000fd9;p=oweals%2Fbusybox.git

ip: fix "ip r" breakage

function                                             old     new   delta
static.ip_func_ptrs                                    -      32     +32
ip_main                                              110      53     -57
------------------------------------------------------------------------------
(add/remove: 1/0 grow/shrink: 0/1 up/down: 32/-57)            Total: -25 bytes

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
---

diff --git a/networking/ip.c b/networking/ip.c
index 7a0f308f0..3a99fa320 100644
--- a/networking/ip.c
+++ b/networking/ip.c
@@ -1,14 +1,12 @@
 /* vi: set sw=4 ts=4: */
 /*
- * ip.c		"ip" utility frontend.
+ * "ip" utility frontend.
  *
  * Licensed under the GPL v2 or later, see the file LICENSE in this tarball.
  *
- * Authors:	Alexey Kuznetsov, <kuznet@ms2.inr.ac.ru>
- *
+ * Authors: Alexey Kuznetsov, <kuznet@ms2.inr.ac.ru>
  *
  * Changes:
- *
  * Rani Assaf <rani@magic.metawire.com> 980929:	resolve addresses
  * Bernhard Reutner-Fischer rewrote to use index_in_substr_array
  */
@@ -24,12 +22,14 @@
  || ENABLE_FEATURE_IP_TUNNEL \
  || ENABLE_FEATURE_IP_RULE
 
-static int NORETURN ip_print_help(char **argv UNUSED_PARAM)
+static int ip_print_help(char **argv UNUSED_PARAM)
 {
 	bb_show_usage();
 }
 
-static int ip_do(int (*ip_func)(char **argv), char **argv)
+typedef int (*ip_func_ptr_t)(char**);
+
+static int ip_do(ip_func_ptr_t ip_func, char **argv)
 {
 	argv = ip_parse_common_args(argv + 1);
 	return ip_func(argv);
@@ -78,45 +78,29 @@ int ip_main(int argc UNUSED_PARAM, char **argv)
 	static const char keywords[] ALIGN1 =
 		IF_FEATURE_IP_ADDRESS("address\0")
 		IF_FEATURE_IP_ROUTE("route\0")
+		IF_FEATURE_IP_ROUTE("r\0")
 		IF_FEATURE_IP_LINK("link\0")
-		IF_FEATURE_IP_TUNNEL("tunnel\0" "tunl\0")
+		IF_FEATURE_IP_TUNNEL("tunnel\0")
+		IF_FEATURE_IP_TUNNEL("tunl\0")
 		IF_FEATURE_IP_RULE("rule\0")
 		;
-	enum {
-		IF_FEATURE_IP_ADDRESS(IP_addr,)
-		IF_FEATURE_IP_ROUTE(IP_route,)
-		IF_FEATURE_IP_LINK(IP_link,)
-		IF_FEATURE_IP_TUNNEL(IP_tunnel, IP_tunl,)
-		IF_FEATURE_IP_RULE(IP_rule,)
-		IP_none
+	static const ip_func_ptr_t ip_func_ptrs[] = {
+		ip_print_help,
+		IF_FEATURE_IP_ADDRESS(do_ipaddr,)
+		IF_FEATURE_IP_ROUTE(do_iproute,)
+		IF_FEATURE_IP_ROUTE(do_iproute,)
+		IF_FEATURE_IP_LINK(do_iplink,)
+		IF_FEATURE_IP_TUNNEL(do_iptunnel,)
+		IF_FEATURE_IP_TUNNEL(do_iptunnel,)
+		IF_FEATURE_IP_RULE(do_iprule,)
 	};
-	int (*ip_func)(char**) = ip_print_help;
+	ip_func_ptr_t ip_func;
+	int key;
 
 	argv = ip_parse_common_args(argv + 1);
-	if (*argv) {
-		int key = index_in_substrings(keywords, *argv);
-		argv++;
-#if ENABLE_FEATURE_IP_ADDRESS
-		if (key == IP_addr)
-			ip_func = do_ipaddr;
-#endif
-#if ENABLE_FEATURE_IP_ROUTE
-		if (key == IP_route)
-			ip_func = do_iproute;
-#endif
-#if ENABLE_FEATURE_IP_LINK
-		if (key == IP_link)
-			ip_func = do_iplink;
-#endif
-#if ENABLE_FEATURE_IP_TUNNEL
-		if (key == IP_tunnel || key == IP_tunl)
-			ip_func = do_iptunnel;
-#endif
-#if ENABLE_FEATURE_IP_RULE
-		if (key == IP_rule)
-			ip_func = do_iprule;
-#endif
-	}
+	key = *argv ? index_in_substrings(keywords, *argv++) : -1;
+	ip_func = ip_func_ptrs[key + 1];
+
 	return ip_func(argv);
 }