cvs commit: src/usr.sbin/moused moused.c

[Yar Tikhiy]

On Thu, Mar 6, 2008 at 5:43 AM, Jung-uk Kim <jkim at freebsd.org> wrote:
> On Wednesday 05 March 2008 09:31 pm, Max Laier wrote:
>  > Am Do, 6.03.2008, 03:14, schrieb Jung-uk Kim:
>  > > jkim        2008-03-06 02:14:45 UTC
>  > >
>  > >   FreeBSD src repository
>  > >
>  > >   Modified files:
>  > >     usr.sbin/moused      moused.c
>  > >   Log:
>  > >   Cast time_t to long to mute tinderbox build failure.
>  >
>  > doesn't this defeat the purpose of having time_t in the first
>  > place?  I'd rather cast to intmax_t and print with %j  ... though
>  > it probably doesn't matter.
>
>  Now the timestamp is uptime and the breakage was just debugging
>  output, it shouldn't really matter at all but I will do that soon.

Yes it matters because it has to do with everyday coding practice. It
is one of small yet important advances in C99 that we can now printf
non-basic integral types _portably_[1] and _reliably_ through %j[du]
and a cast to [u]intmax_t.

[1] Well, almost portably as we need to guess the signedness of the type.

-- 
Yar