diff options
| author | Donald Sharp <sharpd@nvidia.com> | 2024-01-20 18:50:49 -0500 |
|---|---|---|
| committer | Donald Sharp <sharpd@nvidia.com> | 2024-01-20 20:08:05 -0500 |
| commit | 63816f7579b2a8a8be55b842913df06277637bb3 (patch) | |
| tree | 7dee1b6d772a089d103fa06a6a9d162c0814c744 /zebra/zebra_nhg.c | |
| parent | af6499d0524065761c78e0fa70fba6dc84dc5fe1 (diff) | |
zebra: use break instead of goto
There is a goto statement that would be better served
with a break statement. Let's try to minimize this
in the code.
Signed-off-by: Donald Sharp <sharpd@nvidia.com>
Diffstat (limited to 'zebra/zebra_nhg.c')
| -rw-r--r-- | zebra/zebra_nhg.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/zebra/zebra_nhg.c b/zebra/zebra_nhg.c index 93758cca20..9e3aa8bc5c 100644 --- a/zebra/zebra_nhg.c +++ b/zebra/zebra_nhg.c @@ -1082,11 +1082,10 @@ void zebra_nhg_check_valid(struct nhg_hash_entry *nhe) frr_each(nhg_connected_tree, &nhe->nhg_depends, rb_node_dep) { if (CHECK_FLAG(rb_node_dep->nhe->flags, NEXTHOP_GROUP_VALID)) { valid = true; - goto done; + break; } } -done: if (valid) zebra_nhg_set_valid(nhe); else |