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/06/03 20:25:46 UTC

svn commit: r1345734 - /subversion/trunk/subversion/libsvn_wc/conflicts.c

Author: rhuijben
Date: Sun Jun  3 18:25:46 2012
New Revision: 1345734

URL: http://svn.apache.org/viewvc?rev=1345734&view=rev
Log:
Remove an unneeded wc_db call in the conflict resolver code. This was just used
to calculate a parent path, that was not really used.

* subversion/libsvn_wc/conflicts.c
  (resolve_conflict_on_node): Don't calculate the conflict dir, as we no longer
    need that. Joining an absolute temp path to this directory just leaves the
    absolute temp path.

Modified:
    subversion/trunk/subversion/libsvn_wc/conflicts.c

Modified: subversion/trunk/subversion/libsvn_wc/conflicts.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_wc/conflicts.c?rev=1345734&r1=1345733&r2=1345734&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_wc/conflicts.c (original)
+++ subversion/trunk/subversion/libsvn_wc/conflicts.c Sun Jun  3 18:25:46 2012
@@ -146,16 +146,13 @@ resolve_conflict_on_node(svn_boolean_t *
   const char *conflict_new = NULL;
   const char *conflict_working = NULL;
   const char *prop_reject_file = NULL;
-  svn_kind_t kind;
   int i;
   const apr_array_header_t *conflicts;
-  const char *conflict_dir_abspath;
   svn_skel_t *work_items = NULL;
   svn_skel_t *work_item;
 
   *did_resolve = FALSE;
 
-  SVN_ERR(svn_wc__db_read_kind(&kind, db, local_abspath, TRUE, pool));
   SVN_ERR(svn_wc__db_read_conflicts(&conflicts, db, local_abspath,
                                     pool, pool));
 
@@ -176,11 +173,6 @@ resolve_conflict_on_node(svn_boolean_t *
         prop_reject_file = desc->their_abspath;
     }
 
-  if (kind == svn_kind_dir)
-    conflict_dir_abspath = local_abspath;
-  else
-    conflict_dir_abspath = svn_dirent_dirname(local_abspath, pool);
-
   if (resolve_text)
     {
       const char *auto_resolve_src;
@@ -215,7 +207,7 @@ resolve_conflict_on_node(svn_boolean_t *
                   : svn_diff_conflict_display_modified;
 
                 SVN_ERR(svn_wc__db_temp_wcroot_tempdir(&temp_dir, db,
-                                                       conflict_dir_abspath,
+                                                       local_abspath,
                                                        pool, pool));
                 SVN_ERR(svn_stream_open_unique(&tmp_stream,
                                                &auto_resolve_src,
@@ -252,9 +244,7 @@ resolve_conflict_on_node(svn_boolean_t *
         {
           SVN_ERR(svn_wc__wq_build_file_copy_translated(
                     &work_item, db, local_abspath,
-                    svn_dirent_join(conflict_dir_abspath,
-                                    auto_resolve_src, pool),
-                    local_abspath, pool, pool));
+                    auto_resolve_src, local_abspath, pool, pool));
           work_items = svn_wc__wq_merge(work_items, work_item, pool);
         }
     }