On Wed, 4 Nov 2020 15:30:27 +0200 Ido Schimmel wrote: > *flags |= (nhc->nhc_flags & RTNH_F_ONLINK); > if (nhc->nhc_flags & RTNH_F_OFFLOAD) > *flags |= RTNH_F_OFFLOAD; > + if (nhc->nhc_flags & RTNH_F_TRAP) > + *flags |= RTNH_F_TRAP;
Out of curiosity - why use this if construct like OFFLOAD rather than the more concise mask like ONLINK does? In fact looks like the mask could just be extended there instead?
