Subject: Re: CVS commit: src/usr.sbin/installboot
To: Havard Eidnes <he@netbsd.org>
From: David Laight <david@l8s.co.uk>
List: source-changes
Date: 10/30/2006 07:46:56
On Mon, Oct 30, 2006 at 07:03:34AM +0000, Havard Eidnes wrote:
>
> Module Name: src
> Committed By: he
> Date: Mon Oct 30 07:03:34 UTC 2006
>
> Modified Files:
> src/usr.sbin/installboot: ffs.c
>
> Log Message:
> In order to avoid warnings (which is converted to error with -Werror)
> about empty bodies in an if-statement, we need to make add some proprocessor
> conditionals around these if statements. FFS byte-swapping is sometimes
> compiled out of install-media versions of this program.
Having to do that is just plain stupid....
I've also deliberatly used:
#define x(y) if (y); else {...}
To avoid having to wrap the expansion inside do ... while (0);
David
--
David Laight: david@l8s.co.uk