Subject: Re: CVS commit: src/sys/arch/i386/stand/genprom
To: Tom Spindler <dogcow@NetBSD.org>
From: Alan Barrett <apb@cequrux.com>
List: source-changes
Date: 12/26/2007 20:09:30
On Wed, 26 Dec 2007, Tom Spindler wrote:
> Modified Files:
> 	src/sys/arch/i386/stand/genprom: genprom.c
> 
> Log Message:
> ding dong, the witch is __dead. Not sure why it was even bothered with here.

A log message should make sense without having to be viewed alongside
the diff, should explain what you changed, and explain why you changed
it.  In this case, I think you meant "Remove __dead marker from bail()
function beacsue __dead is not defined in my crosscompile environment."

Since bail() doesn't return, it seems to me that the __dead marker was
correct.  The proper fix is not to remove the __dead, but to change the
build so that __dead is defined.

--apb (Alan Barrett)