proxmox-mirrors/mirror_frr
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_frr synced 2025-04-30 02:22:45 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

lib: Make nexthop_next* use const for nexthop

Browse Source 
Make nexthop_next() and nexthop_next_active_resolved() use
const for the nexthop argument. They should not be modifying so
it makes sense here.

Signed-off-by: Stephen Worley <sworley@cumulusnetworks.com>
This commit is contained in:
Stephen Worley 2019-10-23 14:37:20 -04:00
parent 0b4dadb385
commit 17c25e0368
2 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/nexthop.c
View File

@ -349,7 +349,7 @@ const char *nexthop2str(const struct nexthop *nexthop, char *str, int size)
* left branch is 'resolved' and right branch is 'next': * left branch is 'resolved' and right branch is 'next':
* https://en.wikipedia.org/wiki/Tree_traversal#/media/File:Sorted_binary_tree_preorder.svg * https://en.wikipedia.org/wiki/Tree_traversal#/media/File:Sorted_binary_tree_preorder.svg
*/ */
struct nexthop *nexthop_next(struct nexthop *nexthop) struct nexthop *nexthop_next(const struct nexthop *nexthop)
{ {
if (CHECK_FLAG(nexthop->flags, NEXTHOP_FLAG_RECURSIVE)) if (CHECK_FLAG(nexthop->flags, NEXTHOP_FLAG_RECURSIVE))
return nexthop->resolved; return nexthop->resolved;
@ -365,7 +365,7 @@ struct nexthop *nexthop_next(struct nexthop *nexthop)
} }
/* Return the next nexthop in the tree that is resolved and active */ /* Return the next nexthop in the tree that is resolved and active */
struct nexthop *nexthop_next_active_resolved(struct nexthop *nexthop) struct nexthop *nexthop_next_active_resolved(const struct nexthop *nexthop)
{ {
struct nexthop *next = nexthop_next(nexthop); struct nexthop *next = nexthop_next(nexthop);

5
lib/nexthop.h
View File

@ -161,8 +161,9 @@ extern int nexthop_same_firsthop(struct nexthop *next1, struct nexthop *next2);
extern const char *nexthop2str(const struct nexthop *nexthop, extern const char *nexthop2str(const struct nexthop *nexthop,
char *str, int size); char *str, int size);
extern struct nexthop *nexthop_next(struct nexthop *nexthop); extern struct nexthop *nexthop_next(const struct nexthop *nexthop);
extern struct nexthop *nexthop_next_active_resolved(struct nexthop *nexthop); extern struct nexthop *
nexthop_next_active_resolved(const struct nexthop *nexthop);
extern unsigned int nexthop_level(struct nexthop *nexthop); extern unsigned int nexthop_level(struct nexthop *nexthop);
/* Copies to an already allocated nexthop struct */ /* Copies to an already allocated nexthop struct */
extern void nexthop_copy(struct nexthop *copy, const struct nexthop *nexthop, extern void nexthop_copy(struct nexthop *copy, const struct nexthop *nexthop,