On Tue, Jul 11, 2017 at 10:26:03PM +0200, Sebastian Benoit wrote: > ok with one question. > > Florian Obser(flor...@openbsd.org) on 2017.07.11 19:29:44 +0000:
> > @@ -444,12 +443,11 @@ parse_iflist(struct if_msghdr ***ifmlist_p, char > > *buf, size_t bufsize) > > if (ifm->ifm_version == RTM_VERSION) > > (*ifmlist_p)[ifm->ifm_index] = ifm; > > } else { > > - log_warn("out of sync parsing NET_RT_IFLIST," > > + fatalx("out of sync parsing NET_RT_IFLIST," > > " expected %d, got %d, msglen = %d," > > " buf:%p, ifm:%p, lim:%p", > > RTM_IFINFO, ifm->ifm_type, ifm->ifm_msglen, > > buf, ifm, lim); > > - exit(1); > > } > > does it still look readable without the { } ? I tried, it's kinda maybe a bit too long, so I kept the {}, but it's close. -- I'm not entirely sure you are real.