svn commit: r212770 - head/usr.bin/script

Bruce Cran bruce at cran.org.uk
Mon Dec 20 15:34:20 UTC 2010


On Mon, 20 Dec 2010 18:30:36 +0300
Anonymous <swell.k at gmail.com> wrote:

> "David E. O'Brien" <obrien at FreeBSD.org> writes:
> > +	if (av[0])
> > +		for (k = 0 ; av[k] ; ++k)
> > +			fprintf(fscript, "%s%s", k ? " " : "",
> > av[k]);
> > +		fprintf(fscript, "\r\n");
> > +
> 
> An indent bug? 2nd fprintf() call is not under if(av[0]).

But the 2nd printf() call isn't in the scope of the if statement.
Personally I prefer braces if a statement is longer than 1 line, even
if not strictly required.

-- 
Bruce Cran


More information about the svn-src-head mailing list