Synchronize venti/copy with Plan 9 version; add nwf's indented verbose mode

[russcox_gmail.com, 2009-08-21 16:32:22]

Thanks.  A few small comments.
Please fix and reupload with 
codereview -i 110062

http://codereview.appspot.com/110062/diff/1/3
File src/cmd/venti/copy.c (right):

http://codereview.appspot.com/110062/diff/1/3#newcode95
Line 95: for (i = 0; i < depth; i++)
the indentation style here is different from
the rest of the code; please use tabs to indent

http://codereview.appspot.com/110062/diff/1/3#newcode180
Line 180: if ((type !=0 || base !=0) && verbose) {
same indentation comment

[russcox_gmail.com, 2009-08-21 17:31:18]

a few more nits, sorry

http://codereview.appspot.com/110062/diff/7/9
File src/cmd/venti/copy.c (right):

http://codereview.appspot.com/110062/diff/7/9#newcode94
Line 94: if (verbose) {
all the if and for statements in this file
drop the spaces around the parens:

if(verbose){

not

if (verbose) {

please do the same for consistency.
right now it is obvious that the new code
is new code; there should be no such
typographical hints.

http://codereview.appspot.com/110062/diff/7/9#newcode96
Line 96: fprint(2, " ");
this looks like tab tab space
instead of tab tab tab
but i bet it is really tab tab tab
and rietveld is not displaying it
correct.  just double-check, please.

http://codereview.appspot.com/110062/diff/7/9#newcode185
Line 185: fprint(2, " ");
same comment about tab tab tab

[russcox_gmail.com, 2009-08-21 18:24:51]

sorry, for loops too.

http://codereview.appspot.com/110062/diff/12/1004
File src/cmd/venti/copy.c (right):

http://codereview.appspot.com/110062/diff/12/1004#newcode95
Line 95: for (i = 0; i < depth; i++)
same here

http://codereview.appspot.com/110062/diff/12/1004#newcode184
Line 184: for (i = 0; i < depth; i++)
same here

[russcox_gmail.com, 2009-08-21 19:54:54]

thanks!

codebot: apply
venti/copy: synchronize with Plan 9; indent in verbose mode

[old.codebot, 2009-08-21 19:56:10]

# codebot: apply

applied.