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>
This commit is contained in:
Denys Vlasenko 2010-07-24 23:26:58 +02:00
parent b78ac5a20e
commit e0a622093c

View File

@ -1,14 +1,12 @@
/* vi: set sw=4 ts=4: */ /* 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. * 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: * Changes:
*
* Rani Assaf <rani@magic.metawire.com> 980929: resolve addresses * Rani Assaf <rani@magic.metawire.com> 980929: resolve addresses
* Bernhard Reutner-Fischer rewrote to use index_in_substr_array * Bernhard Reutner-Fischer rewrote to use index_in_substr_array
*/ */
@ -24,12 +22,14 @@
|| ENABLE_FEATURE_IP_TUNNEL \ || ENABLE_FEATURE_IP_TUNNEL \
|| ENABLE_FEATURE_IP_RULE || 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(); 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); argv = ip_parse_common_args(argv + 1);
return ip_func(argv); return ip_func(argv);
@ -78,45 +78,29 @@ int ip_main(int argc UNUSED_PARAM, char **argv)
static const char keywords[] ALIGN1 = static const char keywords[] ALIGN1 =
IF_FEATURE_IP_ADDRESS("address\0") IF_FEATURE_IP_ADDRESS("address\0")
IF_FEATURE_IP_ROUTE("route\0") IF_FEATURE_IP_ROUTE("route\0")
IF_FEATURE_IP_ROUTE("r\0")
IF_FEATURE_IP_LINK("link\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") IF_FEATURE_IP_RULE("rule\0")
; ;
enum { static const ip_func_ptr_t ip_func_ptrs[] = {
IF_FEATURE_IP_ADDRESS(IP_addr,) ip_print_help,
IF_FEATURE_IP_ROUTE(IP_route,) IF_FEATURE_IP_ADDRESS(do_ipaddr,)
IF_FEATURE_IP_LINK(IP_link,) IF_FEATURE_IP_ROUTE(do_iproute,)
IF_FEATURE_IP_TUNNEL(IP_tunnel, IP_tunl,) IF_FEATURE_IP_ROUTE(do_iproute,)
IF_FEATURE_IP_RULE(IP_rule,) IF_FEATURE_IP_LINK(do_iplink,)
IP_none 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); argv = ip_parse_common_args(argv + 1);
if (*argv) { key = *argv ? index_in_substrings(keywords, *argv++) : -1;
int key = index_in_substrings(keywords, *argv); ip_func = ip_func_ptrs[key + 1];
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
}
return ip_func(argv); return ip_func(argv);
} }