bgpd, lib, zebra: Remove unnecessary return

The route_map_walk_update_list callback function
never uses the return code, so just remove it.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

diff --git a/bgpd/bgp_routemap.c b/bgpd/bgp_routemap.c
index 14c1c33a318eb89e87bd6fe1ad9d24991300d374..9f3bec81d358e6dc990f0c53d0dd8a61a37fa984 100644 (file)
--- a/bgpd/bgp_routemap.c
+++ b/bgpd/bgp_routemap.c
@@ -3112,7 +3112,7 @@ static void bgp_route_map_process_update(struct bgp *bgp, const char *rmap_name,
        }
 }
 
-static int bgp_route_map_process_update_cb(char *rmap_name)
+static void bgp_route_map_process_update_cb(char *rmap_name)
 {
        struct listnode *node, *nnode;
        struct bgp *bgp;
@@ -3127,8 +3127,6 @@ static int bgp_route_map_process_update_cb(char *rmap_name)
        }
 
        vpn_policy_routemap_event(rmap_name);
-
-       return 0;
 }
 
 int bgp_route_map_update_timer(struct thread *thread)

diff --git a/lib/routemap.c b/lib/routemap.c
index b2c19295076d4cceeb630f883802bea2ea23cf53..056c7934540d744d50e17fbada8c88338780f56c 100644 (file)
--- a/lib/routemap.c
+++ b/lib/routemap.c
@@ -807,7 +807,7 @@ static struct route_map *route_map_get(const char *name)
        return map;
 }
 
-void route_map_walk_update_list(int (*route_map_update_fn)(char *name))
+void route_map_walk_update_list(void (*route_map_update_fn)(char *name))
 {
        struct route_map *node;
        struct route_map *nnode = NULL;

diff --git a/lib/routemap.h b/lib/routemap.h
index e653a8a78872bd85ef5f24319356d2148aa64033..0aeba7e1f66d4e5627c9b1bb2641f0e289af002e 100644 (file)
--- a/lib/routemap.h
+++ b/lib/routemap.h
@@ -216,7 +216,7 @@ extern void route_map_add_hook(void (*func)(const char *));
 extern void route_map_delete_hook(void (*func)(const char *));
 extern void route_map_event_hook(void (*func)(route_map_event_t, const char *));
 extern int route_map_mark_updated(const char *name);
-extern void route_map_walk_update_list(int (*update_fn)(char *name));
+extern void route_map_walk_update_list(void (*update_fn)(char *name));
 extern void route_map_upd8_dependency(route_map_event_t type, const char *arg,
                                      const char *rmap_name);
 extern void route_map_notify_dependencies(const char *affected_name,

diff --git a/zebra/zebra_routemap.c b/zebra/zebra_routemap.c
index 8e368c257eb317ffe4ea4953c1fdd9d7c3d0ff65..6d3c2b2265a3df5c12e8f66e3eaebc2349d6d5c5 100644 (file)
--- a/zebra/zebra_routemap.c
+++ b/zebra/zebra_routemap.c
@@ -1282,13 +1282,11 @@ static struct route_map_rule_cmd route_set_src_cmd = {
        "src", route_set_src, route_set_src_compile, route_set_src_free,
 };
 
-static int zebra_route_map_process_update_cb(char *rmap_name)
+static void zebra_route_map_process_update_cb(char *rmap_name)
 {
        if (IS_ZEBRA_DEBUG_EVENT)
                zlog_debug("Event handler for route-map: %s",
                           rmap_name);
-
-       return 0;
 }
 
 static int zebra_route_map_update_timer(struct thread *thread)