diff options
| author | Stephen Worley <sworley@cumulusnetworks.com> | 2020-09-23 14:17:15 -0400 |
|---|---|---|
| committer | Stephen Worley <sworley@cumulusnetworks.com> | 2020-09-23 14:17:15 -0400 |
| commit | 5d06c5d5ef50b66f7ffc2fee70c20f49d064aa87 (patch) | |
| tree | 4b7477ec79aa225e58b74489cac722ceadedff77 /pbrd/pbr_zebra.c | |
| parent | fe870621b577e3a486604b95cedd4fde86d36ccd (diff) | |
pbrd: use bool for pbr_send_pbr_map() return val
Use a bool as the return val for pbr_send_pbr_map() to make
the code a bit more readable. Dont expect there to be need
for values other than true or false anyway.
Signed-off-by: Stephen Worley <sworley@cumulusnetworks.com>
Diffstat (limited to 'pbrd/pbr_zebra.c')
| -rw-r--r-- | pbrd/pbr_zebra.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/pbrd/pbr_zebra.c b/pbrd/pbr_zebra.c index 077cb9a691..8ef675186f 100644 --- a/pbrd/pbr_zebra.c +++ b/pbrd/pbr_zebra.c @@ -549,8 +549,8 @@ static void pbr_encode_pbr_map_sequence(struct stream *s, stream_put(s, ifp->name, INTERFACE_NAMSIZ); } -int pbr_send_pbr_map(struct pbr_map_sequence *pbrms, - struct pbr_map_interface *pmi, bool install, bool changed) +bool pbr_send_pbr_map(struct pbr_map_sequence *pbrms, + struct pbr_map_interface *pmi, bool install, bool changed) { struct pbr_map *pbrm = pbrms->parent; struct stream *s; @@ -569,10 +569,10 @@ int pbr_send_pbr_map(struct pbr_map_sequence *pbrms, * to delete just return. */ if (install && is_installed && !changed) - return 1; + return false; if (!install && !is_installed) - return 1; + return false; s = zclient->obuf; stream_reset(s); @@ -596,5 +596,5 @@ int pbr_send_pbr_map(struct pbr_map_sequence *pbrms, zclient_send_message(zclient); - return 0; + return true; } |