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

Re: Parameter /startrev vs /revstart in tortoiseproc

From: Stefan Küng <tortoisesvn_at_gmail.com>
Date: 2007-03-23 18:16:14 CET

Tobias Schäfer wrote:

> I've just noticed that there are parameters /startrev and /revstart (same
> applies to /endrev vs /revend). This is confusing and I would like to name
> them the same everywhere.
>
> /startrev:
> * cmdDiff
> * cmdBlame
>
> /revstart:
> * cmdLog
>
> So which do we chose and should we continue to evaluate the old parameters
> for the duration of a major release?
>
> I personally prefer /startrev and will replace /revstart with /startrev in
> TortoiseProc, the calling functions and in the documentation. At the same
> time I'll rename /revend to /endrev and /revpeg to /pegrev. I don't think
> we need evaluate the old parameters. The change will be documented in
> Changelog.txt and the parameters can be corrected quickly if needed.

Not sure, but maybe we should support *both*, at least for some time.
There are other apps/plugins out there which call TortoiseProc directly,
and they would start to fail if we change the param names. (that's why I
never 'fixed' this inconsistency).

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.net
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org
Received on Fri Mar 23 18:16:24 2007

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