*: remove useless return variables

Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
This commit is contained in:
Quentin Young 2019-02-25 18:59:55 +00:00
parent b08047f82d
commit 9f2d035447
6 changed files with 9 additions and 22 deletions

View File

@ -577,8 +577,6 @@ static uint16_t eigrp_next_sequence_encode(struct stream *s)
static uint16_t eigrp_hello_parameter_encode(struct eigrp_interface *ei, static uint16_t eigrp_hello_parameter_encode(struct eigrp_interface *ei,
struct stream *s, uint8_t flags) struct stream *s, uint8_t flags)
{ {
uint16_t length = EIGRP_TLV_PARAMETER_LEN;
// add in the parameters TLV // add in the parameters TLV
stream_putw(s, EIGRP_TLV_PARAMETER); stream_putw(s, EIGRP_TLV_PARAMETER);
stream_putw(s, EIGRP_TLV_PARAMETER_LEN); stream_putw(s, EIGRP_TLV_PARAMETER_LEN);
@ -605,7 +603,7 @@ static uint16_t eigrp_hello_parameter_encode(struct eigrp_interface *ei,
// and set hold time value.. // and set hold time value..
stream_putw(s, ei->params.v_wait); stream_putw(s, ei->params.v_wait);
return length; return EIGRP_TLV_PARAMETER_LEN;
} }
/** /**

View File

@ -1499,8 +1499,6 @@ static int eigrp_config_write(struct vty *vty)
{ {
struct eigrp *eigrp; struct eigrp *eigrp;
int write = 0;
eigrp = eigrp_lookup(); eigrp = eigrp_lookup();
if (eigrp != NULL) { if (eigrp != NULL) {
/* Writes 'router eigrp' section to config */ /* Writes 'router eigrp' section to config */
@ -1525,7 +1523,7 @@ static int eigrp_config_write(struct vty *vty)
// config_write_eigrp_distance (vty, eigrp) // config_write_eigrp_distance (vty, eigrp)
} }
return write; return 0;
} }
void eigrp_vty_show_init(void) void eigrp_vty_show_init(void)

View File

@ -2164,7 +2164,6 @@ int send_csnp(struct isis_circuit *circuit, int level)
int send_l1_csnp(struct thread *thread) int send_l1_csnp(struct thread *thread)
{ {
struct isis_circuit *circuit; struct isis_circuit *circuit;
int retval = ISIS_OK;
circuit = THREAD_ARG(thread); circuit = THREAD_ARG(thread);
assert(circuit); assert(circuit);
@ -2181,13 +2180,12 @@ int send_l1_csnp(struct thread *thread)
isis_jitter(circuit->csnp_interval[0], CSNP_JITTER), isis_jitter(circuit->csnp_interval[0], CSNP_JITTER),
&circuit->t_send_csnp[0]); &circuit->t_send_csnp[0]);
return retval; return ISIS_OK;
} }
int send_l2_csnp(struct thread *thread) int send_l2_csnp(struct thread *thread)
{ {
struct isis_circuit *circuit; struct isis_circuit *circuit;
int retval = ISIS_OK;
circuit = THREAD_ARG(thread); circuit = THREAD_ARG(thread);
assert(circuit); assert(circuit);
@ -2204,7 +2202,7 @@ int send_l2_csnp(struct thread *thread)
isis_jitter(circuit->csnp_interval[1], CSNP_JITTER), isis_jitter(circuit->csnp_interval[1], CSNP_JITTER),
&circuit->t_send_csnp[1]); &circuit->t_send_csnp[1]);
return retval; return ISIS_OK;
} }
/* /*
@ -2329,7 +2327,6 @@ int send_l1_psnp(struct thread *thread)
{ {
struct isis_circuit *circuit; struct isis_circuit *circuit;
int retval = ISIS_OK;
circuit = THREAD_ARG(thread); circuit = THREAD_ARG(thread);
assert(circuit); assert(circuit);
@ -2342,7 +2339,7 @@ int send_l1_psnp(struct thread *thread)
isis_jitter(circuit->psnp_interval[0], PSNP_JITTER), isis_jitter(circuit->psnp_interval[0], PSNP_JITTER),
&circuit->t_send_psnp[0]); &circuit->t_send_psnp[0]);
return retval; return ISIS_OK;
} }
/* /*
@ -2352,7 +2349,6 @@ int send_l1_psnp(struct thread *thread)
int send_l2_psnp(struct thread *thread) int send_l2_psnp(struct thread *thread)
{ {
struct isis_circuit *circuit; struct isis_circuit *circuit;
int retval = ISIS_OK;
circuit = THREAD_ARG(thread); circuit = THREAD_ARG(thread);
assert(circuit); assert(circuit);
@ -2366,7 +2362,7 @@ int send_l2_psnp(struct thread *thread)
isis_jitter(circuit->psnp_interval[1], PSNP_JITTER), isis_jitter(circuit->psnp_interval[1], PSNP_JITTER),
&circuit->t_send_psnp[1]); &circuit->t_send_psnp[1]);
return retval; return ISIS_OK;
} }
/* /*

View File

@ -380,12 +380,10 @@ static int ism_interface_up(struct ospf_interface *oi)
static int ism_loop_ind(struct ospf_interface *oi) static int ism_loop_ind(struct ospf_interface *oi)
{ {
int ret = 0;
/* call ism_interface_down. */ /* call ism_interface_down. */
/* ret = ism_interface_down (oi); */ /* ret = ism_interface_down (oi); */
return ret; return 0;
} }
/* Interface down event handler. */ /* Interface down event handler. */

View File

@ -1039,8 +1039,6 @@ static int dplane_ctx_pw_init(struct zebra_dplane_ctx *ctx,
enum dplane_op_e op, enum dplane_op_e op,
struct zebra_pw *pw) struct zebra_pw *pw)
{ {
int ret = AOK;
if (IS_ZEBRA_DEBUG_DPLANE_DETAIL) if (IS_ZEBRA_DEBUG_DPLANE_DETAIL)
zlog_debug("init dplane ctx %s: pw '%s', loc %u, rem %u", zlog_debug("init dplane ctx %s: pw '%s', loc %u, rem %u",
dplane_op2str(op), pw->ifname, pw->local_label, dplane_op2str(op), pw->ifname, pw->local_label,
@ -1070,7 +1068,7 @@ static int dplane_ctx_pw_init(struct zebra_dplane_ctx *ctx,
ctx->u.pw.fields = pw->data; ctx->u.pw.fields = pw->data;
return ret; return AOK;
} }
/* /*

View File

@ -3236,7 +3236,6 @@ void rib_close_table(struct route_table *table)
*/ */
static int handle_pw_result(struct zebra_dplane_ctx *ctx) static int handle_pw_result(struct zebra_dplane_ctx *ctx)
{ {
int ret = 0;
struct zebra_pw *pw; struct zebra_pw *pw;
struct zebra_vrf *vrf; struct zebra_vrf *vrf;
@ -3255,7 +3254,7 @@ static int handle_pw_result(struct zebra_dplane_ctx *ctx)
done: done:
return ret; return 0;
} }