Subject: Re: dtc: Trivial formatting fixes

From: David Gibson <[email-masked]>
Date: Thu, 27 Mar 2008 09:30:38 +1100

On Wed, Mar 26, 2008 at 08:15:35AM -0500, Jon Loeliger wrote:
> > This patch fixes some trivial indentation and brace/bracket style
> > problems.
> 
> 
> > @@ -179,9 +179,8 @@
> >  		arg = argv[optind];
> >  
> >  	/* minsize and padsize are mutually exclusive */
> > -	if ((minsize) && (padsize)) {
> > +	if (minsize && padsize)
> >  		die("Can't set both -p and -S\n");
> > -	}
> 
> 
> I do not consider extra braces a "problem", and will
> not be applying those changes.  The other indentation
> fixes will be applied, of course.

Meh, whatever.  Usual kernel style - which is what I originally
adopted for dtc - says no braces for one line blocks.

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson