svn commit: r1414554 - /subversion/branches/in-repo-authz/subversion/libsvn_repos/authz.c

[br...@apache.org]

Author: breser
Date: Wed Nov 28 07:04:22 2012
New Revision: 1414554

URL: http://svn.apache.org/viewvc?rev=1414554&view=rev
Log:
* subversion/libsvn_repos/authz.c
  (authz_retrieve_config): Fix a comment that had a sentence fragment in it and
    a concern regarding Windows.  There is no issue on Windows, repos_root is
    always internal style and path is a url (or repos relative URL which is
    a URL fragment) and as such uses / as the path seperator.


Modified:
    subversion/branches/in-repo-authz/subversion/libsvn_repos/authz.c

Modified: subversion/branches/in-repo-authz/subversion/libsvn_repos/authz.c
URL: http://svn.apache.org/viewvc/subversion/branches/in-repo-authz/subversion/libsvn_repos/authz.c?rev=1414554&r1=1414553&r2=1414554&view=diff
==============================================================================
--- subversion/branches/in-repo-authz/subversion/libsvn_repos/authz.c (original)
+++ subversion/branches/in-repo-authz/subversion/libsvn_repos/authz.c Wed Nov 28 07:04:22 2012
@@ -864,12 +864,7 @@ authz_retrieve_config(svn_config_t **cfg
       svn_error_t *err;
       apr_pool_t *scratch_pool = svn_pool_create(pool);
 
-      /* Convert the repos_root to a file schema URL first, so we can
-       * work out 
-       * ### Is path or repos_root guaranteed to be in internal format
-       * ### already on Windows?  I don't think so, need to test on Windows.
-       * ### If not this isn't going to work right since it'll construct a
-       * ### URL with blackslahes instead of forward slashes.  */
+      /* Convert the repos_root to a file schema URL first. */
       err = svn_uri_get_file_url_from_dirent(&repos_root_url, repos_root,
                                              scratch_pool);