Blair Zajac wrote:
> If you pass a path to svn_opt_parse_path with no trailing @, then the
> revision is svn_opt_revision_unspecified:
>
> /* Didn't find an @-sign. */
> *truepath = svn_path_canonicalize(path, pool);
> rev->kind = svn_opt_revision_unspecified;
>
> but if you have path with a trailing @ just to protect another @ earlier
> in the path, then it's set to either svn_opt_revision_head or
> svn_opt_revision_base based if the path is an URL or not:
>
> if (path[i + 1] == '\0') /* looking at empty peg revision */
> {
> ret = 0;
> start_revision.kind = is_url ? svn_opt_revision_head
> : svn_opt_revision_base;
> }
>
> ...
> ...
>
> rev->kind = start_revision.kind;
> rev->value = start_revision.value;
>
> Shouldn't the @ protection just use svn_opt_revision_unspecified?
Seems so to me. That looks like a real bug.
--
C. Michael Pilato <cmpilato@collab.net>
CollabNet <> www.collab.net <> Distributed Development On Demand
Received on Wed Oct 17 01:18:15 2007