Re: [PATCH] Issue 1074 (Take 3)

cmpilato@collab.net writes:

> > There were a couple of things I was unsure about. I went ahead and added
> > subcommand_verify to svnadmin, but it really calls subcommand_dump after
> > setting a couple of options.
>
> Make sense. Perhaps a cleaner way would just be to core the guts out
> of subcommand_dump into a helper function that is called by both
> subcommand_dump and subcommand_verify.

Ignore this. It's poppycock. subcommand_verify is about 5 lines of
code, so there's no point in trying to abstract anything out.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Aug 25 09:26:53 2003