Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 22 Dec 2002 11:30:06 +1100 (EST)
From:      Bruce Evans <bde@zeta.org.au>
To:        Sam Leffler <sam@errno.com>
Cc:        Nate Lawson <nate@root.org>, <cvs-all@FreeBSD.org>, <cvs-committers@FreeBSD.org>
Subject:   Re: cvs commit: src/sys/sys time.h src/sys/kern kern_time.c
Message-ID:  <20021222110710.Y7634-100000@gamplex.bde.org>
In-Reply-To: <129901c2a927$7767df70$52557f42@errno.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, 21 Dec 2002, Sam Leffler wrote:

> The version in sys/time.h is #ifndef _KERNEL and marked as specifically for
> compatibility for netbsd.  Rather than change the include file and pollute a
> namespace I added a private definition.  If it's ok to do add this pollution
> then sure, just remove the private definition.

Not OK.  Use the 4.xBSD timeval*() interfaces for manipulating timevals,
not the NetBSD timer*() ones.

Better still, don't use timevals for new interfaces; use timespecs and
timespec*().  Rate limiting doesn't need either nsec or usec resolution
so it may as well use a non-legacy type for times.

Bruce


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?20021222110710.Y7634-100000>