Subject: Re: CVS commit: src/sys/arch/i386/stand/genprom
To: Perry E. Metzger <perry@piermont.com>
From: Tom Spindler <dogcow@babymeat.com>
List: source-changes
Date: 12/26/2007 23:06:01
> Alan Barrett <apb@cequrux.com> writes:
> > 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.
>
> Compiles with high WARNS levels would now fail...
Have you tried compiling with a high WARNS level? No errors are
thrown, since <bsd.hostprog.mk> doesn't do anything with WARNS.
In any case: the only time a warning would be emitted is if you
(rather inexplicably) turned on -Wmissing-noreturn - which isn't
enabled by any other -W flags.
As far as fixing the build so that __dead doesn't produce compiler
warnings - please follow up to my query on tech-misc, message-id
<20071226105053.GB23538@babymeat.com>, which addresses precisely
this sort of issue. (Alternately, patch the code in whatever
fashion is acceptable.)