Re: svn commit: rev 3153

Karl Fogel <kfogel@newton.ch.collab.net> writes:

> But there's still a bit of an abstraction failure, in that the code in
> split_url.c "knows" that "" is the empty path, yet path.c wavers
> between using a #define for that and just hardcoding it?
>
> I feel like we could be more clean about this... If "" is the only
> possible empty path, then really, why do we need svn_path_is_empty()
> at all? We could just test path[0] directly every time.

Well, originally I was going to use "." as the empty path, but then I
found that the fs library would need to be changed, and it was simpler
(for me anyway) to change the wc and client libraries to use "". The
path library itself will work with either, hence the local #define. An
explicit "" is used in quite a few places in the code, most of which
used to be "." but I suppose a constant should be used instead.

-- 
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org