On Tue, Jul 11, 2017 at 10:26:03PM +0200, Sebastian Benoit wrote:
> ok with one question.
>
> Florian Obser([email protected]) 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.