]> git.puffer.fish Git - matthieu/frr.git/commitdiff
bgpd: change conditional operators for bgp/ospf json changes
authorDon Slice <dslice@cumulusnetworks.com>
Fri, 31 Aug 2018 12:50:02 +0000 (12:50 +0000)
committerDon Slice <dslice@cumulusnetworks.com>
Fri, 31 Aug 2018 12:50:02 +0000 (12:50 +0000)
Signed-off-by: Don Slice <dslice@cumulusnetworks.com>
bgpd/bgp_vty.c

index 77c6167e249365be4484caec9002bb9e52806b0b..ef89981dcd7681bb3e6d4618b3ac8301c255bdd6 100644 (file)
@@ -321,11 +321,11 @@ int bgp_vty_find_and_parse_afi_safi_bgp(struct vty *vty,
                else {
                        *bgp = bgp_lookup_by_name(vrf_name);
                        if (!*bgp) {
-                               use_json
-                                       ? vty_out(vty, "{}\n")
-                                       : vty_out(vty,
-                                                 "View/Vrf specified is unknown: %s\n",
-                                                 vrf_name);
+                               vty_out(vty,
+                                       use_json
+                                               ? "{}\n"
+                                               : "View/Vrf specified is unknown: %s\n",
+                                       vrf_name);
                                *idx = 0;
                                return 0;
                        }
@@ -333,10 +333,10 @@ int bgp_vty_find_and_parse_afi_safi_bgp(struct vty *vty,
        } else {
                *bgp = bgp_get_default();
                if (!*bgp) {
-                       use_json
-                               ? vty_out(vty, "{}\n")
-                               : vty_out(vty,
-                                         "Unable to find default BGP instance\n");
+                       vty_out(vty,
+                               use_json
+                                       ? "{}\n"
+                                       : "Unable to find default BGP instance\n");
                        *idx = 0;
                        return 0;
                }
@@ -8118,8 +8118,7 @@ static void bgp_show_summary_afi_safi(struct vty *vty, struct bgp *bgp, int afi,
        if (use_json && is_wildcard)
                vty_out(vty, "}\n");
        else if (!nbr_output)
-               use_json ? vty_out(vty, "{}\n")
-                        : vty_out(vty, "%% No BGP neighbors found\n");
+               vty_out(vty, use_json ? "{}\n" : "%% No BGP neighbors found\n");
 }
 
 static void bgp_show_all_instances_summary_vty(struct vty *vty, afi_t afi,
@@ -8177,10 +8176,10 @@ int bgp_show_summary_vty(struct vty *vty, const char *name, afi_t afi,
                        bgp = bgp_lookup_by_name(name);
 
                        if (!bgp) {
-                               use_json
-                                       ? vty_out(vty, "{}\n")
-                                       : vty_out(vty,
-                                                 "%% BGP instance not found\n");
+                               vty_out(vty,
+                                       use_json
+                                               ? "{}\n"
+                                               : "%% BGP instance not found\n");
                                return CMD_WARNING;
                        }
 
@@ -8195,8 +8194,8 @@ int bgp_show_summary_vty(struct vty *vty, const char *name, afi_t afi,
        if (bgp)
                bgp_show_summary_afi_safi(vty, bgp, afi, safi, use_json, NULL);
        else {
-               use_json ? vty_out(vty, "{}\n")
-                        : vty_out(vty, "%% No such BGP instance exist\n");
+               vty_out(vty,
+                       use_json ? "{}\n" : "%% No such BGP instance exist\n");
                return CMD_WARNING;
        }
 
@@ -10880,10 +10879,8 @@ static int bgp_show_neighbor_vty(struct vty *vty, const char *name,
                                          json);
                }
                json_object_free(json);
-       } else {
-               use_json ? vty_out(vty, "{}\n")
-                        : vty_out(vty, "%% BGP instance not found\n");
-       }
+       } else
+               vty_out(vty, use_json ? "{}\n" : "%% BGP instance not found\n");
 
        return CMD_SUCCESS;
 }