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.

Reply via email to