You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by rh...@apache.org on 2012/05/11 01:53:36 UTC

svn commit: r1336970 - /subversion/trunk/subversion/libsvn_client/copy.c

Author: rhuijben
Date: Thu May 10 23:53:35 2012
New Revision: 1336970

URL: http://svn.apache.org/viewvc?rev=1336970&view=rev
Log:
* subversion/libsvn_client/copy.c
  (wc_to_repos_copy): Remove a block of non functional code. The revision is
    not used by the harvester and all versioned vs unversioned errors were
    ignored.

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

Modified: subversion/trunk/subversion/libsvn_client/copy.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_client/copy.c?rev=1336970&r1=1336969&r2=1336970&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_client/copy.c (original)
+++ subversion/trunk/subversion/libsvn_client/copy.c Thu May 10 23:53:35 2012
@@ -1154,22 +1154,6 @@ wc_to_repos_copy(const apr_array_header_
 
   iterpool = svn_pool_create(pool);
 
-  /* Verify that all the source paths exist, are versioned, etc.
-     We'll do so by querying the base revisions of those things (which
-     we'll need to know later anyway).
-     ### Should we use the 'origin' revision instead of 'base'?
-    */
-  for (i = 0; i < copy_pairs->nelts; i++)
-    {
-      svn_client__copy_pair_t *pair = APR_ARRAY_IDX(copy_pairs, i,
-                                                    svn_client__copy_pair_t *);
-      svn_pool_clear(iterpool);
-
-      SVN_ERR(svn_wc__node_get_base(&pair->src_revnum, NULL, NULL, NULL,
-                                    ctx->wc_ctx, pair->src_abspath_or_url,
-                                    iterpool, iterpool));
-    }
-
   /* Determine the longest common ancestor for the destinations, and open an RA
      session to that location. */
   /* ### But why start by getting the _parent_ of the first one? */