Subject: Re: CVS commit: src/sys/kern
To: None <source-changes@NetBSD.org, fredb@NetBSD.org, jhawk@NetBSD.org>
From: Frederick Bruckman <fredb@immanent.net>
List: source-changes
Date: 06/19/2004 16:33:17
In article <20040619200238.265212DA1D@cvs.netbsd.org>,
	fredb@netbsd.org (Frederick Bruckman) writes:
> 
> Modified Files:
> 	src/sys/kern: kern_microtime.c
> 
> Log Message:
> Using the same variable to mean two different things is false economy.
> With this change, the compiled object is 72 bytes smaller on i386, and
> the code is easier to understand, to boot.

jhawk has pointed out to me that that message is rather cryptic without
resort to the patch. I've changed it in the cvs repository to read...

 Introduce a variable "cc" to hold the cpu counter delta, rather than
 overloading "usec". The counter isn't counting micro-seconds, and using
 the same variable to mean two different things is false economy: with
 this change, the compiled object is 72 bytes smaller on i386, and the
 code is easier to understand, to boot.

> To generate a diff of this commit:
> cvs rdiff -r1.6 -r1.7 src/sys/kern/kern_microtime.c
> 
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.

-- 
Frederick