You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by ju...@apache.org on 2010/11/30 15:18:08 UTC

svn commit: r1040535 - /subversion/trunk/subversion/libsvn_client/checkout.c

Author: julianfoad
Date: Tue Nov 30 14:18:08 2010
New Revision: 1040535

URL: http://svn.apache.org/viewvc?rev=1040535&view=rev
Log:
* subversion/libsvn_client/checkout.c
  (svn_client__checkout_internal): Don't canonicalize the input URL here in
    the library, just check it is canonical already, like everywhere else.

Modified:
    subversion/trunk/subversion/libsvn_client/checkout.c

Modified: subversion/trunk/subversion/libsvn_client/checkout.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_client/checkout.c?rev=1040535&r1=1040534&r2=1040535&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_client/checkout.c (original)
+++ subversion/trunk/subversion/libsvn_client/checkout.c Tue Nov 30 14:18:08 2010
@@ -92,7 +92,7 @@ svn_client__checkout_internal(svn_revnum
 
   /* Sanity check.  Without these, the checkout is meaningless. */
   SVN_ERR_ASSERT(local_abspath != NULL);
-  SVN_ERR_ASSERT(url != NULL);
+  SVN_ERR_ASSERT(svn_uri_is_canonical(url, pool));
   SVN_ERR_ASSERT(svn_dirent_is_absolute(local_abspath));
 
   /* Fulfill the docstring promise of svn_client_checkout: */
@@ -101,9 +101,6 @@ svn_client__checkout_internal(svn_revnum
       && (revision->kind != svn_opt_revision_head))
     return svn_error_create(SVN_ERR_CLIENT_BAD_REVISION, NULL, NULL);
 
-  /* Canonicalize the URL. */
-  url = svn_uri_canonicalize(url, pool);
-
   {
     svn_boolean_t have_repos_root_url;
     svn_boolean_t have_repos_uuid;