svn commit: r1239749 - /subversion/branches/1.7.x-issue4059/subversion/libsvn_client/commit_util.c

[st...@apache.org]

Author: stsp
Date: Thu Feb  2 18:39:40 2012
New Revision: 1239749

URL: http://svn.apache.org/viewvc?rev=1239749&view=rev
Log:
On the 1.7.x-issue4059 branch:

* subversion/libsvn_client/commit_util.c
  (harvest_commitables): The code on trunk can rely on the is_explicit_target
   parameter to prevent unnecessarily checking danglers which aren't in the
   commit target list. On this branch, we can simulate the same by setting
   the danglers parameter to NULL during recursion. Prevents a performance
   regression (wc.db queries issued for explicit targets vs. every committable).

Suggested by: rhuijben

Modified:
    subversion/branches/1.7.x-issue4059/subversion/libsvn_client/commit_util.c

Modified: subversion/branches/1.7.x-issue4059/subversion/libsvn_client/commit_util.c
URL: http://svn.apache.org/viewvc/subversion/branches/1.7.x-issue4059/subversion/libsvn_client/commit_util.c?rev=1239749&r1=1239748&r2=1239749&view=diff
==============================================================================
--- subversion/branches/1.7.x-issue4059/subversion/libsvn_client/commit_util.c (original)
+++ subversion/branches/1.7.x-issue4059/subversion/libsvn_client/commit_util.c Thu Feb  2 18:39:40 2012
@@ -885,7 +885,7 @@ harvest_committables(svn_wc_context_t *w
                                        changelists,
                                        (depth < svn_depth_files),
                                        (depth < svn_depth_immediates),
-                                       danglers,
+                                       NULL, /* danglers */
                                        check_url_func, check_url_baton,
                                        cancel_func, cancel_baton,
                                        notify_func, notify_baton,