You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by cm...@apache.org on 2011/01/19 20:59:55 UTC

svn commit: r1060945 - /subversion/branches/uris-as-urls/BRANCH-README

Author: cmpilato
Date: Wed Jan 19 19:59:55 2011
New Revision: 1060945

URL: http://svn.apache.org/viewvc?rev=1060945&view=rev
Log:
On the 'uris-as-urls' branch:

* BRANCH-README
  Add some TODO notes.

Modified:
    subversion/branches/uris-as-urls/BRANCH-README

Modified: subversion/branches/uris-as-urls/BRANCH-README
URL: http://svn.apache.org/viewvc/subversion/branches/uris-as-urls/BRANCH-README?rev=1060945&r1=1060944&r2=1060945&view=diff
==============================================================================
--- subversion/branches/uris-as-urls/BRANCH-README (original)
+++ subversion/branches/uris-as-urls/BRANCH-README Wed Jan 19 19:59:55 2011
@@ -13,4 +13,42 @@ This branch is *NOT* guaranteed to build
 are being intentionally used as cheap pointers to code that needs to
 be examined in light of the desired functionality.
 
+Here are the svn_uri_* functions which need to be reviewed/modified:
+
+   svn_uri_is_absolute [DONE]
+
+      Remove.  All URLs will be absolute by definition.
+
+   svn_uri_join
+
+      Remove.  Maybe add svn_uri_join_relpath() (though
+      svn_path_url_add_component2() should be sufficient)
+   
+   svn_uri_split
+   svn_uri_basename
+   svn_uri_dirname
+   svn_uri_is_canonical
+   svn_uri_get_longest_ancestor
+   svn_uri_is_child
+   svn_uri_is_ancestor
+   svn_uri_skip_ancestor
+   svn_uri_condense_targets
+
+      Remove code that accepts non-URL URIs (asserting that input is
+      valid).
+   
+   svn_uri_canonicalize
+
+      Assert that we have something vaguely URL-ish to even work with.
+   
+   svn_uri_get_dirent_from_file_url [DONE]
+   svn_uri_get_file_url_from_dirent [DONE]
+
+      Expect nothing will change here.
+   
+   svn_url_is_root
+
+      This says it's not to be used with URLs, so purge this function
+      and rework its current usage to use something else.
+
 [1] http://subversion.tigris.org/issues/show_bug.cgi?id=2028