Philip Martin <philip@codematters.co.uk> writes:
> Strange how things turn out. I have made the changes to the path
> library, the path regression tests now pass using either "." or "" as
> the empty path (and switching is easy). However when we come to the
> full regression test "." fails abysmally, whereas "" works. I've
> looked at a few of the "." failures and it appears that the FS library
> also uses the path library, and it doesn't handle "." as an empty dir.
>
> So, despite my earlier emails, it looks like I'm going to make "" the
> canonical empty path. Hey, we all knew that was the really the right
> solution, didn't we ;-)
No libsvn_fs path is allowed to have a '.' component, but that
doesn't mean the filesystem, as a user of the path utility functions,
can't just learn to expect a '.' return value from, say,
svn_path_split() to mean "the path passed in had but one component".
You just have to teach the filesystem code the new semantics, and let
it modify its behavior accordingly.
Also, I'm wondering something. If '' is the canonical empty dir, then
would:
svn_path_canonicalize ('./foo')
return 'foo' (which feels, for unexplained reasons, like an
information loss) or '/foo' (which is plain wrong) ?
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Aug 29 16:14:40 2002