Source-Changes archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: CVS commit: src/usr.bin/make
On Fri, Feb 15, 2008 at 10:12:39PM +0100, David Holland wrote:
> On Fri, Feb 15, 2008 at 08:08:12PM +0000, Christos Zoulas wrote:
> > Module Name: src
> > Committed By: christos
> > Date: Fri Feb 15 20:08:12 UTC 2008
> >
> > Modified Files:
> > src/usr.bin/make: job.c
> >
> > Log Message:
> > Improve dh's fix:
> > - make sure i is initialized
> > - don't subtract 1 in comparison, add 1 to the other side
>
> Does that really fix it? If max is 0, then job->curPos and nr must
> also both be 0, in which case gotNL must be false and fbuf cannot be
> set to true, and the comparison you fixed can't be reached. (And the
> value of i isn't used.)
>
> If the above reasoning is correct, and this change nonetheless alters
> the behavior, then probably job->curPos is coming up invalid and we
> haven't really fixed the problem but only masked it...
>
> (Also, while I'm kibitzing, the default-case value for i should be
> job->curPos, not job->curPos-1, because it's going to overwrite that
> position with a \0 and it thus shouldn't be a slot that's already
> used. The buffer is sized so there's space.)
Just as a datapoint: make -j5 on my old P4 w/ Hyperthreading enabled
called from build.sh ... distribution still gives
compile compat/rec_search.lo
compile compat/rec_seq.lo
compile compat/rec_utils.lo
build compat/libnbcompat.a
install ===> tools/compat
dependall ===> tools/binstall
compile binstall/xinstall.lo
compile binstall/getid.lo
link binstall/xinstall
xinstall.lo: In stall/xinstall
nbmake: Child (11938) status 0 not in table?
with a runaway make that has to be killed manualy...
Kurt
Home |
Main Index |
Thread Index |
Old Index