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 2011/04/11 15:59:15 UTC

svn commit: r1091077 - /subversion/trunk/subversion/libsvn_client/add.c

Author: rhuijben
Date: Mon Apr 11 13:59:15 2011
New Revision: 1091077

URL: http://svn.apache.org/viewvc?rev=1091077&view=rev
Log:
* subversion/libsvn_client/add.c
  (mkdir_urls): Update comment to explain what is going on.

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

Modified: subversion/trunk/subversion/libsvn_client/add.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_client/add.c?rev=1091077&r1=1091076&r2=1091077&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_client/add.c (original)
+++ subversion/trunk/subversion/libsvn_client/add.c Mon Apr 11 13:59:15 2011
@@ -742,9 +742,7 @@ mkdir_urls(const apr_array_header_t *url
   SVN_ERR(svn_uri_condense_targets(&common, &targets, urls, FALSE,
                                    pool, pool));
 
-  /* ### BH: This looks unnecessary, because the hash is not used and
-         you can't rely on hash ordering but it fails our tests if I
-         remove it. The qsort() below determines the final ordering. */
+  /*Remove duplicate targets introduced by make_parents with more targets. */
   SVN_ERR(svn_hash_from_cstring_keys(&targets_hash, targets, pool));
   SVN_ERR(svn_hash_keys(&targets, targets_hash, pool));