Skip to content

Commit

Permalink
zebra: changes for code maintainability
Browse files Browse the repository at this point in the history
these changes are for improving the code maintainability and readability

Signed-off-by: sri-mohan1 <[email protected]>
  • Loading branch information
sri-mohan1 committed Mar 15, 2024
1 parent 19a0211 commit efbf981
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion zebra/zebra_dplane.c
Original file line number Diff line number Diff line change
Expand Up @@ -6351,7 +6351,7 @@ dplane_provider_dequeue_out_ctx(struct zebra_dplane_provider *prov)
*/
bool dplane_provider_is_threaded(const struct zebra_dplane_provider *prov)
{
return (prov->dp_flags & DPLANE_PROV_FLAG_THREADED);
return CHECK_FLAG(prov->dp_flags, DPLANE_PROV_FLAG_THREADED);
}

#ifdef HAVE_NETLINK
Expand Down
10 changes: 5 additions & 5 deletions zebra/zebra_mpls.c
Original file line number Diff line number Diff line change
Expand Up @@ -329,7 +329,7 @@ static void fec_evaluate(struct zebra_vrf *zvrf)

/* Skip configured FECs and those without a label index.
*/
if (fec->flags & FEC_FLAG_CONFIGURED
if (CHECK_FLAG(fec->flags, FEC_FLAG_CONFIGURED)
|| fec->label_index == MPLS_INVALID_LABEL_INDEX)
continue;

Expand Down Expand Up @@ -2420,7 +2420,7 @@ static int zebra_mpls_cleanup_fecs_for_client(struct zserv *client)
if (fec_client == client) {
listnode_delete(fec->client_list,
fec_client);
if (!(fec->flags & FEC_FLAG_CONFIGURED)
if (!CHECK_FLAG(fec->flags, FEC_FLAG_CONFIGURED)
&& list_isempty(fec->client_list))
fec_del(fec);
break;
Expand Down Expand Up @@ -2538,7 +2538,7 @@ int zebra_mpls_static_fec_add(struct zebra_vrf *zvrf, struct prefix *p,
if (IS_ZEBRA_DEBUG_MPLS)
zlog_debug("Add fec %pFX label %u", p, in_label);
} else {
fec->flags |= FEC_FLAG_CONFIGURED;
SET_FLAG(fec->flags, FEC_FLAG_CONFIGURED);
if (fec->label == in_label)
/* Duplicate config */
return 0;
Expand Down Expand Up @@ -2587,7 +2587,7 @@ int zebra_mpls_static_fec_del(struct zebra_vrf *zvrf, struct prefix *p)
}

old_label = fec->label;
fec->flags &= ~FEC_FLAG_CONFIGURED;
UNSET_FLAG(fec->flags, FEC_FLAG_CONFIGURED);
fec->label = MPLS_INVALID_LABEL;

/* If no client exists, just delete the FEC. */
Expand Down Expand Up @@ -2630,7 +2630,7 @@ int zebra_mpls_write_fec_config(struct vty *vty, struct zebra_vrf *zvrf)
char lstr[BUFSIZ];
fec = rn->info;

if (!(fec->flags & FEC_FLAG_CONFIGURED))
if (!CHECK_FLAG(fec->flags, FEC_FLAG_CONFIGURED))
continue;

write = 1;
Expand Down

0 comments on commit efbf981

Please sign in to comment.