Re: CVS commit: src/usr.bin/make

[christos%astron.com@localhost (Christos Zoulas)]

In article <20081213151929.AFC3F175D0%cvs.netbsd.org@localhost>,
David Laight  <dsl%netbsd.org@localhost> wrote:
>
>Module Name:   src
>Committed By:  dsl
>Date:          Sat Dec 13 15:19:29 UTC 2008
>
>Modified Files:
>       src/usr.bin/make: arch.c compat.c cond.c dir.c for.c hash.c job.c job.h
>           lst.h main.c make.c make.h parse.c suff.c targ.c util.c var.c
>       src/usr.bin/make/lst.lib: lstAppend.c lstConcat.c lstDatum.c
>           lstDeQueue.c lstDestroy.c lstDupl.c lstFind.c lstFindFrom.c
>           lstFirst.c lstForEachFrom.c lstInit.c lstInsert.c lstInt.h
>           lstIsEmpty.c lstLast.c lstMember.c lstNext.c lstOpen.c lstPrev.c
>           lstRemove.c lstReplace.c lstSucc.c
>
>Log Message:
>Use NULL instead of -1 cast to the relavant type (usually via NIL).
>This was a suggestion from christos - so blame him if there is a deep
>reason for using -1 :-)

-1 is more poisonous than 0 when it comes to core-dumping (old OS's
sometimes mapped 0@0). This was 20 years ago though.

christos