Author: kotkov
Date: Wed Oct 12 11:23:32 2016
New Revision: 1764441

URL: http://svn.apache.org/viewvc?rev=1764441&view=rev
Log:
Following up on r1764435, remove two unused functions.

The merge_incoming_added_file_replace() function is now always called with
'merge_files' set to true, and this code can be simplified, but I'll do it
separately.

* subversion/libsvn_client/conflicts.c
  (resolve_merge_incoming_added_file_replace,
   resolve_update_incoming_added_file_replace): Remove these unused functions.

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

Modified: subversion/trunk/subversion/libsvn_client/conflicts.c
URL: 
http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_client/conflicts.c?rev=1764441&r1=1764440&r2=1764441&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_client/conflicts.c (original)
+++ subversion/trunk/subversion/libsvn_client/conflicts.c Wed Oct 12 11:23:32 
2016
@@ -5412,21 +5412,6 @@ unlock_wc:
 
 /* Implements conflict_option_resolve_func_t. */
 static svn_error_t *
-resolve_merge_incoming_added_file_replace(
-  svn_client_conflict_option_t *option,
-  svn_client_conflict_t *conflict,
-  svn_client_ctx_t *ctx,
-  apr_pool_t *scratch_pool)
-{
-  return svn_error_trace(merge_incoming_added_file_replace(option,
-                                                           conflict,
-                                                           ctx,
-                                                           FALSE,
-                                                           scratch_pool));
-}
-
-/* Implements conflict_option_resolve_func_t. */
-static svn_error_t *
 resolve_merge_incoming_added_file_replace_and_merge(
   svn_client_conflict_option_t *option,
   svn_client_conflict_t *conflict,
@@ -5440,79 +5425,6 @@ resolve_merge_incoming_added_file_replac
                                                            scratch_pool));
 }
 
-/* Implements conflict_option_resolve_func_t. */
-static svn_error_t *
-resolve_update_incoming_added_file_replace(svn_client_conflict_option_t 
*option,
-                                           svn_client_conflict_t *conflict,
-                                           svn_client_ctx_t *ctx,
-                                           apr_pool_t *scratch_pool)
-{
-  svn_client_conflict_option_id_t option_id;
-  const char *local_abspath;
-  const char *lock_abspath;
-  svn_error_t *err;
-  const char *backup_path;
-
-  option_id = svn_client_conflict_option_get_id(option);
-  local_abspath = svn_client_conflict_get_local_abspath(conflict);
-
-  /* ### The following WC modifications should be atomic. */
-  SVN_ERR(svn_wc__acquire_write_lock_for_resolve(&lock_abspath, ctx->wc_ctx,
-                                                 local_abspath,
-                                                 scratch_pool, scratch_pool));
-
-  err = verify_local_state_for_incoming_add_upon_update(conflict, option, ctx,
-                                                        scratch_pool);
-  if (err)
-    goto unlock_wc;
-
-  /* We create a backup of the original file. This file is a local addition
-   * which means it does not exist in the repository. So it's a good idea 
-   * to keep a backup, just in case someone picks this option by accident.
-   * First, reserve a name in the filesystem. */
-  err = svn_io_open_uniquely_named(NULL, &backup_path,
-                                   svn_dirent_dirname(local_abspath,
-                                                      scratch_pool), 
-                                   svn_dirent_basename(local_abspath,
-                                                       scratch_pool),
-                                   ".local-add.bak",
-                                   svn_io_file_del_none,
-                                   scratch_pool,scratch_pool);
-  if (err)
-    goto unlock_wc;
-
-  /* Create a backup by renaming the file on top of the 'reserved' name.
-   * Renaming is equally fast for big and small files. */
-  err = svn_io_file_rename2(local_abspath, backup_path, FALSE, scratch_pool);
-  if (err)
-    goto unlock_wc;
-
-  /* Revert to the BASE working copy node to restore the incoming new node.
-   * This also clears the conflict marker. */
-  err = svn_wc_revert5(ctx->wc_ctx, local_abspath, svn_depth_empty, FALSE,
-                       NULL, FALSE, FALSE,
-                       NULL, NULL, /* do not allow the user to cancel here */
-                       ctx->notify_func2, ctx->notify_baton2,
-                       scratch_pool);
-
-  if (ctx->notify_func2)
-    ctx->notify_func2(ctx->notify_baton2,
-                      svn_wc_create_notify(local_abspath,
-                                           svn_wc_notify_resolved_tree,
-                                           scratch_pool),
-                      scratch_pool);
-
-unlock_wc:
-  err = svn_error_compose_create(err, svn_wc__release_write_lock(ctx->wc_ctx,
-                                                                 lock_abspath,
-                                                                 
scratch_pool));
-  SVN_ERR(err);
-
-  conflict->resolution_tree = option_id;
-
-  return SVN_NO_ERROR;
-}
-
 static svn_error_t *
 raise_tree_conflict(const char *local_abspath,
                     svn_wc_conflict_action_t incoming_change,


Reply via email to