svn commit: r1038577 - /subversion/trunk/subversion/include/svn_dirent_uri.h

[ju...@apache.org]

Author: julianfoad
Date: Wed Nov 24 12:08:38 2010
New Revision: 1038577

URL: http://svn.apache.org/viewvc?rev=1038577&view=rev
Log:
* subversion/include/svn_dirent_uri.h
  (svn_dirent_skip_ancestor, svn_relpath_skip_ancestor,
   svn_uri_skip_ancestor): Add notes about poor semantics.

Modified:
    subversion/trunk/subversion/include/svn_dirent_uri.h

Modified: subversion/trunk/subversion/include/svn_dirent_uri.h
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/include/svn_dirent_uri.h?rev=1038577&r1=1038576&r2=1038577&view=diff
==============================================================================
--- subversion/trunk/subversion/include/svn_dirent_uri.h (original)
+++ subversion/trunk/subversion/include/svn_dirent_uri.h Wed Nov 24 12:08:38 2010
@@ -653,6 +653,9 @@ svn_uri_is_ancestor(const char *parent_u
  * This function assumes @a parent_dirent and @a child_dirent are both
  * absolute or relative in the same way.
  *
+ * ### Returning the child in the no-match case is a bad idea when the
+ *     paths are relative; can be useful when they are absolute.
+ *
  * @since New in 1.7.
  */
 const char *
@@ -664,6 +667,8 @@ svn_dirent_skip_ancestor(const char *par
  * @a child_relpath. If @a child_relpath is not below @a parent_relpath,
  * return @a child_relpath.
  *
+ * ### Returning the child in the no-match case is a bad idea.
+ *
  * @since New in 1.7.
  */
 const char *
@@ -677,6 +682,9 @@ svn_relpath_skip_ancestor(const char *pa
  * This function assumes @a parent_uri and @a child_uri are both absolute or
  * relative in the same way.
  *
+ * ### Returning the child in the no-match case is a bad idea when the
+ *     paths are relative; can be useful when they are absolute.
+ *
  * @since New in 1.7.
  */
 const char *