diff options
| author | Quentin Young <qlyoung@cumulusnetworks.com> | 2018-04-16 20:37:30 +0200 |
|---|---|---|
| committer | Quentin Young <qlyoung@cumulusnetworks.com> | 2018-04-17 19:58:24 +0200 |
| commit | 72c038017e1bd3e0df21fced2e4ba113bba5e74c (patch) | |
| tree | ec963149153b2364738b125f50b7c60f89e46243 /ospfd | |
| parent | zebra: fix error-prone array iterator (diff) | |
| download | frr-72c038017e1bd3e0df21fced2e4ba113bba5e74c.tar.xz frr-72c038017e1bd3e0df21fced2e4ba113bba5e74c.zip | |
ospfd: fix stylistic issue
Macro that expands to be wrapped in parentheses was being used as a raw
condition for an if statement, leading to some very weird and confusing
formatting...
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
Diffstat (limited to 'ospfd')
| -rw-r--r-- | ospfd/ospf_te.c | 17 |
1 files changed, 7 insertions, 10 deletions
diff --git a/ospfd/ospf_te.c b/ospfd/ospf_te.c index 05adc5aa4..13857fd0b 100644 --- a/ospfd/ospf_te.c +++ b/ospfd/ospf_te.c @@ -1161,16 +1161,13 @@ static struct ospf_lsa *ospf_mpls_te_lsa_new(struct ospf *ospf, /* Set opaque-LSA header fields depending of the type of RFC */ if (IS_INTER_AS(lp->type)) { - if - IS_FLOOD_AS(lp->type) - { - options |= OSPF_OPTION_E; /* Enable AS external - as we flood - Inter-AS with - Opaque Type 11 */ - lsa_type = OSPF_OPAQUE_AS_LSA; - } - else { + if (IS_FLOOD_AS(lp->type)) { + /* Enable AS external as we flood Inter-AS with Opaque + * Type 11 + */ + options |= OSPF_OPTION_E; + lsa_type = OSPF_OPAQUE_AS_LSA; + } else { options |= LSA_OPTIONS_GET( area); /* Get area default option */ options |= LSA_OPTIONS_NSSA_GET(area); |