Re: CVS update: subversion/subversion/libsvn_fs tree.c

[Greg Stein <gs...@lyra.org>]

On Thu, Feb 15, 2001 at 06:33:20PM -0000, jimb@tigris.org wrote:
>...
>   +/* Return a null-terminated copy of the first component of PATH,
>   +   allocated in POOL.  PATH must not begin with a slash, and must not
>   +   be the empty string; return zero if PATH is malformed.

Maybe this is unrelated to the svn_fs.h interface, but how is the API-user
supposed to refer to the root node?

I think mod_dav_svn is using "" right now (empty string).

Can this be clarified in the svn_fs.h comment about paths?

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/

Re: CVS update: subversion/subversion/libsvn_fs tree.c

[Jim Blandy <ji...@zwingli.cygnus.com>]

Greg Stein <gs...@lyra.org> writes:
> On Thu, Feb 15, 2001 at 06:33:20PM -0000, jimb@tigris.org wrote:
> >...
> >   +/* Return a null-terminated copy of the first component of PATH,
> >   +   allocated in POOL.  PATH must not begin with a slash, and must not
> >   +   be the empty string; return zero if PATH is malformed.
> 
> Maybe this is unrelated to the svn_fs.h interface, but how is the API-user
> supposed to refer to the root node?
> 
> I think mod_dav_svn is using "" right now (empty string).
> 
> Can this be clarified in the svn_fs.h comment about paths?

The 'no leading slash' rule is a holdover from the svn_fs_node_t days,
where it made more sense.  Under the new interface, "/" clearly must
be a valid filename.  I'll fix the docs.

Thanks for pointing this out!