Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 28 Nov 2000 18:14:37 -0800
From:      Alfred Perlstein <bright@wintelcom.net>
To:        "Jeroen C. van Gelderen" <jeroen@vangelderen.org>
Cc:        Warner Losh <imp@village.org>, "Brian F. Feldman" <green@FreeBSD.ORG>, obrien@FreeBSD.ORG, cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG
Subject:   Re: cvs commit: src/usr.sbin/inetd builtins.c
Message-ID:  <20001128181437.S8051@fw.wintelcom.net>
In-Reply-To: <3A245F2C.9842890E@vangelderen.org>; from jeroen@vangelderen.org on Tue, Nov 28, 2000 at 09:43:08PM -0400
References:  <20001126182240.A8051@fw.wintelcom.net> <bright@wintelcom.net> <200011262140.eAQLe2576200@green.dyndns.org> <20001126182240.A8051@fw.wintelcom.net> <200011280524.WAA07413@harmony.village.org> <20001127221535.Y8051@fw.wintelcom.net> <3A245F2C.9842890E@vangelderen.org>

next in thread | previous in thread | raw e-mail | index | archive | help
* Jeroen C. van Gelderen <jeroen@vangelderen.org> [001128 17:43] wrote:
> Alfred Perlstein wrote:
> > 
> > * Warner Losh <imp@village.org> [001127 21:24] wrote:
> > >
> > > Shouldn't this be
> > >       if (to.tv_usec >= 1000000) {
> > >               to.tv_sec += to.tv_usec / 1000000;
> > >               to.tv_usec %= 1000000;
> > >       }
> > 
> > Yes, but it's overkill as afaik = 2000000
> > shouldn't happen.
> 
> 
> So, shouldn't it be:
> 
>    ASSERT(to.tv_usec < 2000000);
>    to.tv_usec += tv.tv_usec;
>    if (to.tv_usec >= 1000000) {
>        to.tv_usec -= 1000000;
>        to.tv_sec++;
>    }
> 

No it should not, but it could.

-- 
-Alfred Perlstein - [bright@wintelcom.net|alfred@freebsd.org]
"I have the heart of a child; I keep it in a jar on my desk."


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20001128181437.S8051>