svn_path_*_style: More eyes, please ...

I'm looking for the multi-file commit bug on Windows, and noticed that
the WC sometimes uses svn_path_repos_style to modify the "ancestor" URL.
Could someone else check if this patch is O.K.? Thanks.

[brane@silmaril libsvn_wc]$ svn diff
Index: .\entries.c
===================================================================
--- .\SVN\text-base\entries.c Fri Aug 31 23:32:26 2001
+++ .\entries.c Mon Sep 10 22:19:41 2001
@@ -395,7 +395,7 @@
                                          APR_HASH_KEY_STRING);
       dst->ancestor = svn_stringbuf_dup (src->ancestor, pool);
       svn_path_add_component (dst->ancestor, name,
- svn_path_repos_style);
+ svn_path_url_style);
     }
 }

@@ -917,7 +917,7 @@
             {
               this_path = svn_stringbuf_dup (this_dir->ancestor, pool);
               svn_path_add_component_nts (this_path, this_entry_name,
- svn_path_repos_style);
+ svn_path_url_style);
               if (svn_stringbuf_compare (this_path, this_entry->ancestor))
                 apr_hash_set (this_entry->attributes,
                               SVN_WC_ENTRY_ATTR_ANCESTOR,
Index: .\adm_ops.c
===================================================================
--- .\SVN\text-base\adm_ops.c Sat Sep 8 14:13:17 2001
+++ .\adm_ops.c Mon Sep 10 22:18:48 2001
@@ -598,7 +598,7 @@

   /* Derive the ancestor path for our new addition here. */
   ancestor_path = svn_stringbuf_dup (entry->ancestor, pool);
- svn_path_add_component (ancestor_path, basename, svn_path_repos_style);
+ svn_path_add_component (ancestor_path, basename, svn_path_url_style);

   /* Make sure this new directory has an admistrative subdirectory
      created inside of it */

-- 
Brane �ibej   <brane_at_xbc.nu>            http://www.xbc.nu/brane/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org