Re: [PATCH] svn_path_canonicalize

On Wed, Sep 19, 2001 at 09:08:43PM +0200, Branko Cibej wrote:
 
> Conceptionally, the svn_path_*_style constants select a style, not a
> path separator. At this level, an URL is definitely a "path style"
> (although note that we implicitly support only file:// and http(s)://
> schemas in this function). Therefore the decision about whether to merge
> consecutive directory separators, whether ".." means "one level up", etc
> ... is a property of the style, not the function name.

ok, you've got me convinced ;-)

i'll work up another patch, then if people feel like debating the
point they can just chose between them..

(you know if i had just seen this side of the argument before
implemnting the other side, it would have saved me a long debugging
session... of course in that case i wouldn't know nearly as much
about how the client works as i do now, so i guess we're even ;-)

-garrett

-- 
garrett rooney                     Unix was not designed to stop you from 
rooneg@electricjellyfish.net       doing stupid things, because that would  
http://electricjellyfish.net/      stop you from doing clever things.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org