On 1/30/18 11:09 AM, Serhey Popovych wrote: > Improve print_linkinfo_brief() and it's callers: > > 1) Get rid of custom @struct filter pointer @pfilter: it is NULL in > all callers anyway and global @filter is used. >
Looks like I somehow dropped a vrf change that was going to use that. Send a standalone patch to revert 63891c70137f (straight revert does not work so some fixup is needed) and then a follow on for the second part of this change. > 2) Simplify calling code in ipaddr_list_flush_or_save() by > introducing intermediate variable of @struct nlmsghdr, drop > duplicated code: print_linkinfo_brief() never returns values other > than <= 0 so we can move print_selected_addrinfo() outside of each > block. > > Signed-off-by: Serhey Popovych <serhe.popov...@gmail.com> > --- > ip/ip_common.h | 3 +-- > ip/ipaddress.c | 60 > ++++++++++++++++++++++++-------------------------------- > ip/iplink.c | 2 +- > 3 files changed, 28 insertions(+), 37 deletions(-) >