[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

[PATCH v2] Conflict option labels

From: Patrick Steinhardt <patrick.steinhardt_at_elegosoft.com>
Date: Thu, 13 Oct 2016 15:46:32 +0200

Hi,

here's the second version of the conflict option label patch.
Changes:

- reworded some labels
- now using apr_array to pass around options
- renamed and simplified svn_client_resolver_option_label

The functionality has been lightly tested by creating conflict
scenarios.

[[
Move conflict resolution options' labels out of the client

* subversion/include/svn_client.h:
  - new function `svn_client_conflict_option_get_label`
* subversion/libsvn_client/conflicts.c:
  - svn_client_conflict_option_t: add label
  - add_resolution_option: add label argument
  - implement function `svn_client_conflict_option_get_label`
  - (svn_client_conflict_text_get_reslution_options,
     svn_client_conflict_prop_get_resolution_options,
     configure_option_accept_current_wc_state,
     configure_option_move_destination,
     configure_option_update_raise_moved_away_children,
     configure_option_incoming_add_ignore,
     configure_option_incoming_added_file_text_merge,
     configure_option_incoming_added_file_replace_and_merge,
     configure_option_incoming_added_dir_merge,
     configure_option_incoming_added_dir_replace,
     configure_option_incoming_added_dir_replace_and_merge,
     configure_option_incoming_delete_ignore,
     configure_option_incoming_delete_accept,
     configure_option_incoming_move_file_merge,
     configure_option_incoming_dir_merge,
     svn_client_conflict_tree_get_resolution_options): set
     resolution option labels
* subversion/svn/conflict-callbacks.c:
  - resolver_option_t: remove short_desc and long_desc
  - client_option_t: new struct for client options
  - builtin_resolver_options: remove short_desc and long_desc
  - (extra_resolver_options,
     extra_resolver_options_text,
     extra_resolver_options_prop,
     extra_resolver_options_tree): convert to client_option_t
  - find_option: accept options as apr_array_header_t
  - find_option_by_builtin: function to create provided options
     from builtin library options
  - find_option_by_id: replaced by find_option_by_builtin
  - (prompt_string,
     help_string,
     prompt_user,
     build_text_conflict_options,
     build_prop_conflict_options,
     build_prop_text_conflict_options,
     handle_one_prop_conflict.
     build_tree_conflict_options,
     handle_tree_conflict): accept options as apr_array_header_t
]]

Regards

-- 
Patrick Steinhardt, Entwickler
elego Software Solutions GmbH, http://www.elego.de
Gebäude 12 (BIG), Gustav-Meyer-Allee 25, 13355 Berlin, Germany
Sitz der Gesellschaft: Berlin, USt-IdNr.: DE 163214194
Handelsregister: Amtsgericht Charlottenburg HRB 77719
Geschäftsführer: Olaf Wagner

Received on 2016-10-13 15:46:49 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.