Date: Mon, 4 Mar 2002 16:50:27 -0800 From: "David O'Brien" <obrien@FreeBSD.org> To: Mark Murray <markm@FreeBSD.org> Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/sys random.h src/sys/dev/random harvest.c hash.c hash.h randomdev.c randomdev.h yarrow.c Message-ID: <20020304165027.A38109@dragon.nuxi.com> In-Reply-To: <200203031944.g23JiMj39565@freefall.freebsd.org>; from markm@FreeBSD.org on Sun, Mar 03, 2002 at 11:44:22AM -0800 References: <200203031944.g23JiMj39565@freefall.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, Mar 03, 2002 at 11:44:22AM -0800, Mark Murray wrote:
> markm 2002/03/03 11:44:22 PST
>
> Modified files:
> sys/sys random.h
> sys/dev/random harvest.c hash.c hash.h randomdev.c
> randomdev.h yarrow.c
> Log:
> Massive lint-inspired cleanup.
> Remove unneeded includes.
> Deal with unused function arguments.
> Resolve a boatload of signed/unsigned imcompatabilities.
- for (i = 0; i < count; i+= sizeof(u_long)) {
+ for (i = 0; i < count; i+= (size_t)sizeof(u_long)) {
Why is this needed?? sizeof already has a type of "size_t".
This looks like more obfuscation just to quiet Lint.
ISO-C99 says:
6.5.3.4 The _sizeof_ operator
4 The value of the result is implementation-defined, and its type (an
unsigned integer type) is `size_t', defined in `<stddef.h>' (and other
headers).
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?20020304165027.A38109>