summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorQuentin Young <qlyoung@cumulusnetworks.com>2018-04-16 14:50:30 -0400
committerQuentin Young <qlyoung@cumulusnetworks.com>2018-04-17 13:58:24 -0400
commita8fd6fdeea4b7212a730a2fa6d205171d92550fb (patch)
tree06a802ca87b31711744748f97c907f49309660d3
parent1aa37b48107de774f3a0f3ceec7a020951483a91 (diff)
bgpd: fix stylistic issue
Again, use of a macro as a conditional without wrapping in parens... Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
-rw-r--r--bgpd/rfapi/rfapi_import.c23
1 files changed, 10 insertions, 13 deletions
diff --git a/bgpd/rfapi/rfapi_import.c b/bgpd/rfapi/rfapi_import.c
index 392c878675..c1af269d3f 100644
--- a/bgpd/rfapi/rfapi_import.c
+++ b/bgpd/rfapi/rfapi_import.c
@@ -2802,19 +2802,16 @@ rfapiBiStartWithdrawTimer(struct rfapi_import_table *import_table,
uint32_t lifetime;
struct rfapi_withdraw *wcb;
- if
- CHECK_FLAG(bi->flags, BGP_INFO_REMOVED)
- {
- /*
- * Already on the path to being withdrawn,
- * should already have a timer set up to
- * delete it.
- */
- vnc_zlog_debug_verbose(
- "%s: already being withdrawn, do nothing",
- __func__);
- return;
- }
+ if (CHECK_FLAG(bi->flags, BGP_INFO_REMOVED)) {
+ /*
+ * Already on the path to being withdrawn,
+ * should already have a timer set up to
+ * delete it.
+ */
+ vnc_zlog_debug_verbose(
+ "%s: already being withdrawn, do nothing", __func__);
+ return;
+ }
rfapiGetVncLifetime(bi->attr, &lifetime);
vnc_zlog_debug_verbose("%s: VNC lifetime is %u", __func__, lifetime);