[prev in list] [next in list] [prev in thread] [next in thread] 

List:       subversion-issues
Subject:    =?UTF-8?B?W0lzc3VlIDIwMjhdIE5ldyAtIFNlcGFyYXRlIHBhdGggZnVuY3Rpbw==?=
From:       ghudson () tigris ! org
Date:       2004-08-29 14:40:13
Message-ID: 20040829144013.28324.qmail () tigris ! org
[Download RAW message or body]

http://subversion.tigris.org/issues/show_bug.cgi?id=2028
                  Issue #:|2028
                  Summary:|Separate path functions from URL functions
                Component:|subversion
                  Version:|current
                 Platform:|All
                      URL:|
               OS/Version:|All
                   Status:|NEW
        Status whiteboard:|
                 Keywords:|
               Resolution:|
               Issue type:|TASK
                 Priority:|P4
             Subcomponent:|libsvn_subr
              Assigned to:|issues@subversion
              Reported by:|ghudson






------- Additional comments from ghudson@tigris.org Sun Aug 29 07:40:13 -0700 2004 -------
A number of svn_path functions are documented as accepting URLs in particular
positions.  This practice can lead to trouble if a caller assumes without 
checking that what it has is a path; for instance, if you
svn_path_canonicalize() a URL and then pass it to svn_path_dirname(), what
should happen if you reach the "http://hostname" part of the URL?

In 2.0, we should have separate functions for operating on paths and URLs, and a
third set for operating on "targets" which might be either (e.g.
svn_target_is_url to distinguish between path targets and URL targets).  Then if
you canonicalize a URL as a path, what you will get back is a canonicalized path
(e.g. "http:/hostname/path"), not a URL, and the behavior of svn_path_dirname
and friends are well-specified.

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

[prev in list] [next in list] [prev in thread] [next in thread]