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

Re: [PATCH] Repository root relative url support in svn CLI -- REDUX

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Thu, 06 Mar 2008 20:32:36 -0500

Julian Foad wrote:
> The fact that a "^/" prefix was the way to recognise an incoming
> relative URL on the command line doesn't mean it has to be the way it is
> communicated internally later on. We could use a different way to
> distinguish the type of target, such as attaching an enumeration {
> TARGET_TYPE_URL, TARGET_TYPE_REL_URL, TARGET_TYPE_PATH }.

In the past I've wanted to make the svn_opt target-parsing stuff get
transformed to return not arrays of 'const char *' but arrays of
svn_opt_target_t (or some new structure for holding parsed target-y stuff).
  At the time I wanted to do this because I wanted the following fields:

    const char *path;
    svn_opt_revision_t peg_revision;

So if you go this route of return structures with paths and path-type enums,
consider also plopping the peg_revisions in there, too.

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand

Received on 2008-03-07 02:33:18 CET

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