diff options
| author | David Lamparter <equinox@opensourcerouting.org> | 2022-04-04 14:00:59 +0200 |
|---|---|---|
| committer | David Lamparter <equinox@opensourcerouting.org> | 2022-04-04 14:11:29 +0200 |
| commit | b6fcc0b7a65a3e5bff771b0579d4b7eb2997709c (patch) | |
| tree | 8e455b59c8e80eee58b63654aa406131ebd5ec71 /pimd/pim_vty.c | |
| parent | 96322c148bfc862a60cbd11bf03a0afab5da4b75 (diff) | |
pimd: remove useless PIM_IF_* macros
The only function these macros have is to make the code confusing.
"PIM_IF_DO_PIM" sounds like it triggers some action, but it doesn't.
Replace with "bool" fields in struct pim_interface.
(Note: PIM_IF_*_IGMP_LISTEN_ALLROUTERS was always set, without any way
to unset it. It is completely removed now and always enabled.)
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
Diffstat (limited to 'pimd/pim_vty.c')
| -rw-r--r-- | pimd/pim_vty.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/pimd/pim_vty.c b/pimd/pim_vty.c index a0dea63b79..b360181f43 100644 --- a/pimd/pim_vty.c +++ b/pimd/pim_vty.c @@ -289,7 +289,7 @@ static int pim_igmp_config_write(struct vty *vty, int writes, struct pim_interface *pim_ifp) { /* IF ip igmp */ - if (PIM_IF_TEST_IGMP(pim_ifp->options)) { + if (pim_ifp->igmp_enable) { vty_out(vty, " ip igmp\n"); ++writes; } @@ -361,7 +361,7 @@ int pim_config_write(struct vty *vty, int writes, struct interface *ifp, { struct pim_interface *pim_ifp = ifp->info; - if (PIM_IF_TEST_PIM(pim_ifp->options)) { + if (pim_ifp->pim_enable) { vty_out(vty, " " PIM_AF_NAME " pim\n"); ++writes; } |