diff options
| author | Donald Sharp <sharpd@cumulusnetworks.com> | 2017-01-25 09:43:11 -0500 |
|---|---|---|
| committer | Donald Sharp <sharpd@cumulusnetworks.com> | 2017-01-25 09:43:11 -0500 |
| commit | c493f2d83a990b5fec0ea927915c46c47d7ed636 (patch) | |
| tree | 847802dd027fab2e777a78ed0367b97b8209ebc2 | |
| parent | 7eeee51e02e815ae79d6ed78f490fb45ea2ebae7 (diff) | |
bgpd: Remove redundant warning message
When bgp_vty_find_and_parse_afi_safi_vrf detects
a invalidly named vrf, it warns the user. There
is no need for the calling function to warn
again.
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
| -rw-r--r-- | bgpd/bgp_route.c | 42 |
1 files changed, 13 insertions, 29 deletions
diff --git a/bgpd/bgp_route.c b/bgpd/bgp_route.c index 92a2843ba9..02ba731f04 100644 --- a/bgpd/bgp_route.c +++ b/bgpd/bgp_route.c @@ -7877,10 +7877,8 @@ DEFUN (show_ip_bgp, idx = bgp_vty_find_and_parse_afi_safi_vrf (vty, argv, argc, idx, &afi, &safi, &vrf); if (!idx) - { - vty_out (vty, "View/Vrf Specified: %s is unknown", argv[5]->arg); - return CMD_WARNING; - } + return CMD_WARNING; + int uj = use_json (argc, argv); if (uj) argc--; @@ -7986,10 +7984,7 @@ DEFUN (show_ip_bgp_route, idx = bgp_vty_find_and_parse_afi_safi_vrf (vty, argv, argc, idx, &afi, &safi, &vrf); if (!idx) - { - vty_out (vty, "View/Vrf Specified: %s is unknown", argv[5]->arg); - return CMD_WARNING; - } + return CMD_WARNING; if (vrf != VRF_ALL) { @@ -8056,10 +8051,7 @@ DEFUN (show_ip_bgp_regexp, int idx = 0; idx = bgp_vty_find_and_parse_afi_safi_vrf (vty, argv, argc, idx, &afi, &safi, &vrf); if (!idx) - { - vty_out (vty, "View/Vrf Specified: %s is unknown", argv[5]->arg); - return CMD_WARNING; - } + return CMD_WARNING; // get index of regex argv_find (argv, argc, "regexp", &idx); @@ -8093,10 +8085,8 @@ DEFUN (show_ip_bgp_instance_all, int idx = 0; idx = bgp_vty_find_and_parse_afi_safi_vrf (vty, argv, argc, idx, &afi, &safi, &vrf); if (!idx) - { - vty_out (vty, "View/Vrf Specified: %s is unknown", argv[5]->arg); - return CMD_WARNING; - } + return CMD_WARNING; + int uj = use_json (argc, argv); if (uj) argc--; @@ -8786,10 +8776,8 @@ DEFUN (show_ip_bgp_instance_neighbor_prefix_counts, idx = bgp_vty_find_and_parse_afi_safi_vrf (vty, argv, argc, idx, &afi, &safi, &vrf); if (!idx) - { - vty_out (vty, "View/Vrf Specified: %s is unknown", argv[5]->arg); - return CMD_WARNING; - } + return CMD_WARNING; + int uj = use_json (argc, argv); if (uj) argc--; @@ -9146,10 +9134,8 @@ DEFUN (show_ip_bgp_instance_neighbor_advertised_route, idx = bgp_vty_find_and_parse_afi_safi_vrf (vty, argv, argc, idx, &afi, &safi, &vrf); if (!idx) - { - vty_out (vty, "View/Vrf Specified: %s is unknown", argv[5]->arg); - return CMD_WARNING; - } + return CMD_WARNING; + int uj = use_json (argc, argv); if (uj) argc--; @@ -9334,12 +9320,10 @@ DEFUN (show_ip_bgp_neighbor_routes, int idx = 0; - idx = bgp_vty_find_and_parse_afi_safi_vrf (vty, argv, argc, idx, &afi, &safi, &vrf); + idx = bgp_vty_find_and_parse_afi_safi_vrf (vty, argv, argc, idx, &afi, &safi, &vrf); if (!idx) - { - vty_out (vty, "View/Vrf Specified: %s is unknown", argv[5]->arg); - return CMD_WARNING; - } + return CMD_WARNING; + int uj = use_json (argc, argv); if (uj) argc--; |
