Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/lib/libc/time

To: source-changes-d%NetBSD.org@localhost
Subject: Re: CVS commit: src/lib/libc/time
From: David Holland <dholland-sourcechanges%netbsd.org@localhost>
Date: Mon, 14 Dec 2009 06:15:21 +0000

On Mon, Dec 14, 2009 at 05:51:57AM +0000, Matt Thomas wrote:
 > Suppress a warning if time_t is __int64_t

-                               } while ((sse * 10 <= TIME_MAX) &&
+                               } while (((uint64_t)(sse * 10) <= TIME_MAX) &&

Don't you want ((uint64_t)sse * 10 <= TIME_MAX), that is, cast before
multiplying?

-- 
David A. Holland
dholland%netbsd.org@localhost

Follow-Ups:
- Re: CVS commit: src/lib/libc/time
  - From: Joerg Sonnenberger

Prev by Date: Re: CVS commit: src/sys/external/bsd/drm/dist/bsd-core
Next by Date: Re: CVS commit: src/sys/kern
Previous by Thread: Re: CVS commit: src/common/lib/libprop
Next by Thread: Re: CVS commit: src/lib/libc/time
Indexes:

Home | Main Index | Thread Index | Old Index