[PATCH iproute2-next v4 3/3] bridge: refactor bridge mcast querier function

[Fabian Pfitzner]

Make code more readable and consistent with other functions.

Signed-off-by: Fabian Pfitzner <f.pfitz...@pengutronix.de>
---
 lib/bridge.c | 72 +++++++++++++++++++++++++---------------------------
 1 file changed, 34 insertions(+), 38 deletions(-)

diff --git a/lib/bridge.c b/lib/bridge.c
index 35cc409a..5386aa01 100644
--- a/lib/bridge.c
+++ b/lib/bridge.c
@@ -49,59 +49,55 @@ void bridge_print_vlan_stats(const struct 
bridge_vlan_xstats *vstats)
 void bridge_print_mcast_querier_state(const struct rtattr *vtb)
 {
        struct rtattr *bqtb[BRIDGE_QUERIER_MAX + 1];
+       const char *querier_ip;
        SPRINT_BUF(other_time);
+       __u64 tval;

        parse_rtattr_nested(bqtb, BRIDGE_QUERIER_MAX, vtb);
        memset(other_time, 0, sizeof(other_time));

        open_json_object("mcast_querier_state_ipv4");
        if (bqtb[BRIDGE_QUERIER_IP_ADDRESS]) {
-               print_string(PRINT_FP,
-                       NULL,
-                       "%s ",
-                       "mcast_querier_ipv4_addr");
-               print_color_string(PRINT_ANY,
-                       COLOR_INET,
-                       "mcast_querier_ipv4_addr",
-                       "%s ",
-                       format_host_rta(AF_INET, 
bqtb[BRIDGE_QUERIER_IP_ADDRESS]));
+               querier_ip = format_host_rta(AF_INET,
+                                            bqtb[BRIDGE_QUERIER_IP_ADDRESS]);
+               print_string(PRINT_FP, NULL, "%s ",
+                            "mcast_querier_ipv4_addr");
+               print_color_string(PRINT_ANY, COLOR_INET,
+                                  "mcast_querier_ipv4_addr", "%s ",
+                                  querier_ip);
        }
        if (bqtb[BRIDGE_QUERIER_IP_PORT])
-               print_uint(PRINT_ANY,
-                       "mcast_querier_ipv4_port",
-                       "mcast_querier_ipv4_port %u ",
-                       rta_getattr_u32(bqtb[BRIDGE_QUERIER_IP_PORT]));
-       if (bqtb[BRIDGE_QUERIER_IP_OTHER_TIMER])
+               print_uint(PRINT_ANY, "mcast_querier_ipv4_port",
+                          "mcast_querier_ipv4_port %u ",
+                          rta_getattr_u32(bqtb[BRIDGE_QUERIER_IP_PORT]));
+       if (bqtb[BRIDGE_QUERIER_IP_OTHER_TIMER]) {
+               tval = rta_getattr_u64(bqtb[BRIDGE_QUERIER_IP_OTHER_TIMER]);
                print_string(PRINT_ANY,
-                       "mcast_querier_ipv4_other_timer",
-                       "mcast_querier_ipv4_other_timer %s ",
-                       sprint_time64(
-                               
rta_getattr_u64(bqtb[BRIDGE_QUERIER_IP_OTHER_TIMER]),
-                                                               other_time));
+                            "mcast_querier_ipv4_other_timer",
+                            "mcast_querier_ipv4_other_timer %s ",
+                            sprint_time64(tval, other_time));
+       }
        close_json_object();
        open_json_object("mcast_querier_state_ipv6");
        if (bqtb[BRIDGE_QUERIER_IPV6_ADDRESS]) {
-               print_string(PRINT_FP,
-                       NULL,
-                       "%s ",
-                       "mcast_querier_ipv6_addr");
-               print_color_string(PRINT_ANY,
-                       COLOR_INET6,
-                       "mcast_querier_ipv6_addr",
-                       "%s ",
-                       format_host_rta(AF_INET6, 
bqtb[BRIDGE_QUERIER_IPV6_ADDRESS]));
+               querier_ip = format_host_rta(AF_INET6,
+                                            bqtb[BRIDGE_QUERIER_IPV6_ADDRESS]);
+               print_string(PRINT_FP, NULL, "%s ",
+                            "mcast_querier_ipv6_addr");
+               print_color_string(PRINT_ANY, COLOR_INET6,
+                                  "mcast_querier_ipv6_addr", "%s ",
+                                  querier_ip);
        }
        if (bqtb[BRIDGE_QUERIER_IPV6_PORT])
-               print_uint(PRINT_ANY,
-                       "mcast_querier_ipv6_port",
-                       "mcast_querier_ipv6_port %u ",
-                       rta_getattr_u32(bqtb[BRIDGE_QUERIER_IPV6_PORT]));
-       if (bqtb[BRIDGE_QUERIER_IPV6_OTHER_TIMER])
+               print_uint(PRINT_ANY, "mcast_querier_ipv6_port",
+                          "mcast_querier_ipv6_port %u ",
+                          rta_getattr_u32(bqtb[BRIDGE_QUERIER_IPV6_PORT]));
+       if (bqtb[BRIDGE_QUERIER_IPV6_OTHER_TIMER]) {
+               tval = rta_getattr_u64(bqtb[BRIDGE_QUERIER_IPV6_OTHER_TIMER]);
                print_string(PRINT_ANY,
-                       "mcast_querier_ipv6_other_timer",
-                       "mcast_querier_ipv6_other_timer %s ",
-                       sprint_time64(
-                               
rta_getattr_u64(bqtb[BRIDGE_QUERIER_IPV6_OTHER_TIMER]),
-                                                               other_time));
+                            "mcast_querier_ipv6_other_timer",
+                            "mcast_querier_ipv6_other_timer %s ",
+                            sprint_time64(tval, other_time));
+       }
        close_json_object();
 }
--
2.39.5