[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Add help text for extensions in svn 1.4

From: Karl Fogel <kfogel_at_google.com>
Date: 2006-07-31 23:20:01 CEST

"Lieven Govaerts" <lgo@mobsol.be> writes:
> Well that seems to be the current choice for 1.4, so if you want to put this
> up for further discussion, then better be quick :)

I'll let it slide; I don't have anything better to propose off the top
of my head :-).

> Attached patch has a slightly altered first sentence. Can you check if this
> is correct? (my English isn't good enough to cram that much information in
> such a small space).

I think most users don't know what "internal diff" means out of
context, but they might understand it this way:

   "If Subversion is invoking an external diff program, ARG is simply
    passed along to the program. But if Subversion is using its
    internal diff implementation (the default), ARG should be any of
    the following: ..."

?

>> Independently of the above, I'd recommend using parens
>> instead of square brackets, since the square brackets imply
>> an optional param, not an alternate name for an option.
>
> Hm, while I can agree with you here, why are the top-level options all
> specified as '-c [--change]', -x [--extensions] arg' etc.? I don't see the
> difference.

Oh. Well, okay, IMHO those should be changed to parens too, but
obviously not as part of this change.

-Karl

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Jul 31 23:21:24 2006

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.