You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Arfrever Frehtes Taifersar Arahesis <ar...@gmail.com> on 2008/08/11 01:29:15 UTC

Re: svn commit: r32423 - in branches/windows_dirents/subversion: include libsvn_subr tests/libsvn_subr

2008-08-10 13:51:55 lgo@tigris.org napisaƂ(a):
> Author: lgo
> Date: Sun Aug 10 04:51:55 2008
> New Revision: 32423
> 
> Log:
> Partial implementation of functions handling directory entries 
> (ie. local paths), in order to correctly handling Windows specific path 
> syntax
> like X:foo and //srv/share.
> 
> Also see issues #1711 and #2028.
> 
> Note: most of this code is ressurected from r23906, as it was reverted in
> r23907.
> 
> * subversion/libsvn_subr/path.c
>   (svn_dirent_join, 
>    svn_dirent_join_many,
>    dirent_previous_segment,
>    svn_dirent_dirname,
>    svn_dirent_basename,
>    svn_dirent_split,
>    svn_dirent_is_absolute,
>    svn_dirent_get_longest_ancestor,
>    svn_dirent_canonicalize,
>    svn_dirent_is_canonical): New functions, basically behave the same as 
> their
>     svn_path_* counterparts but specifically support local paths, no URLs.
>   (get_path_ancestor_length): Add mode for URLs or dirents.
>   (svn_path_get_longest_ancestor): Add support for dirents.
> 
> * subversion/include/svn_path.h
>   (svn_dirent_join, 
>    svn_dirent_join_many,
>    svn_dirent_dirname,
>    svn_dirent_basename,
>    svn_dirent_split,
>    svn_dirent_is_absolute,
>    svn_dirent_get_longest_ancestor,
>    svn_dirent_canonicalize,
>    svn_dirent_is_canonical): New function declarations, comments to be added!
> 
> * subversion/tests/libsvn_subr/path-test.c
>   (test_dirent_is_absolute,
>    test_dirent_join,
>    test_dirent_basename,
>    test_dirent_dirname,
>    test_dirent_canonicalize,
>    test_dirent_split): New tests.
>   (test_path_join, test_path_basename, test_path_dirname,
>    test_path_canonicalize, test_path_is_canonical): Added _path_ in function
>     name to indicate these test the svn_path_* functions.
>   (test_funcs[]): Updated and added test function names.
> 
> Modified:
>    branches/windows_dirents/subversion/include/svn_path.h
>    branches/windows_dirents/subversion/libsvn_subr/path.c
>    branches/windows_dirents/subversion/tests/libsvn_subr/path-test.c
> 
> Modified: branches/windows_dirents/subversion/include/svn_path.h
> URL: http://svn.collab.net/viewvc/svn/branches/windows_dirents/subversion/include/svn_path.h?pathrev=32423&r1=32422&r2=32423
> ==============================================================================
> --- branches/windows_dirents/subversion/include/svn_path.h	Sun Aug 10 04:22:34 2008	(r32422)
> +++ branches/windows_dirents/subversion/include/svn_path.h	Sun Aug 10 04:51:55 2008	(r32423)
> @@ -63,6 +63,13 @@ const char *svn_path_internal_style(cons
>  /** Convert @a path from the canonical internal style to the local style. */
>  const char *svn_path_local_style(const char *path, apr_pool_t *pool);
>  
> +/** XXX 
> +  */
> +const char *svn_dirent_canonicalize(const char *path, apr_pool_t *pool);

Function types should be on own lines.

-- 
Arfrever Frehtes Taifersar Arahesis