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