Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 27 Dec 2005 17:24:14 +0100
From:      Stefan Farfeleder <stefan@fafoe.narf.at>
To:        John Baldwin <jhb@freebsd.org>
Cc:        cvs-src@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org
Subject:   Re: cvs commit: src/sys/netatm atm_sys.h
Message-ID:  <20051227162414.GK63814@wombat.fafoe.narf.at>
In-Reply-To: <200512271105.17467.jhb@freebsd.org>
References:  <200512271233.jBRCXIgp017054@repoman.freebsd.org> <200512271105.17467.jhb@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Dec 27, 2005 at 11:05:15AM -0500, John Baldwin wrote:
> On Tuesday 27 December 2005 07:33 am, Stefan Farfeleder wrote:
> > stefanf     2005-12-27 12:33:18 UTC
> >
> >   FreeBSD src repository
> >
> >   Modified files:
> >     sys/netatm           atm_sys.h
> >   Log:
> >   Try to unbreak the tinderbox: Cast a time_t argument to long before
> > printing it with %ld.
> 
> You should probably use intmax_t and %j instead.  On an i386 with a 64-bit 
> time_t, time_t doesn't fit into a long.  If time_t were 64-bit on powerpc 
> you'd have the same problem there as well.

It's the difference between two time_ts.  I pondered using intmax_t but
thought it's fair to assume it fits into 32 bits.  Feel free to change
it though.

Stefan



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