To libify `git apply` functionality we must make 'struct apply_state' usable outside "builtin/apply.c". Let's do that by creating a new "apply.h" and moving 'struct apply_state' there. Signed-off-by: Christian Couder <chriscool@xxxxxxxxxxxxx> --- apply.h | 115 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++ builtin/apply.c | 112 +----------------------------------------------------- 2 files changed, 116 insertions(+), 111 deletions(-) create mode 100644 apply.h diff --git a/apply.h b/apply.h new file mode 100644 index 0000000..aa11ea6 --- /dev/null +++ b/apply.h @@ -0,0 +1,115 @@ +#ifndef APPLY_H +#define APPLY_H + +enum ws_error_action { + nowarn_ws_error, + warn_on_ws_error, + die_on_ws_error, + correct_ws_error +}; + + +enum ws_ignore { + ignore_ws_none, + ignore_ws_change +}; + +/* + * We need to keep track of how symlinks in the preimage are + * manipulated by the patches. A patch to add a/b/c where a/b + * is a symlink should not be allowed to affect the directory + * the symlink points at, but if the same patch removes a/b, + * it is perfectly fine, as the patch removes a/b to make room + * to create a directory a/b so that a/b/c can be created. + * + * See also "struct string_list symlink_changes" in "struct + * apply_state". + */ +#define SYMLINK_GOES_AWAY 01 +#define SYMLINK_IN_RESULT 02 + +struct apply_state { + const char *prefix; + int prefix_length; + + /* + * Since lockfile.c keeps a linked list of all created + * lock_file structures, it isn't safe to free(lock_file). + */ + struct lock_file *lock_file; + int newfd; + + int apply; + int allow_overlap; + int apply_in_reverse; + int apply_with_reject; + int apply_verbosely; + + /* --cached updates only the cache without ever touching the working tree. */ + int cached; + + /* --stat does just a diffstat, and doesn't actually apply */ + int diffstat; + + /* --numstat does numeric diffstat, and doesn't actually apply */ + int numstat; + + int summary; + int threeway; + int no_add; + const char *fake_ancestor; + const char *patch_input_file; + struct string_list limit_by_name; + int has_include; + struct strbuf root; + struct string_list symlink_changes; + + /* + * --check turns on checking that the working tree matches the + * files that are being modified, but doesn't apply the patch + */ + int check; + + /* --index updates the cache as well. */ + int check_index; + + int unidiff_zero; + int update_index; + int unsafe_paths; + int line_termination; + + /* + * For "diff-stat" like behaviour, we keep track of the biggest change + * we've seen, and the longest filename. That allows us to do simple + * scaling. + */ + int max_change; + int max_len; + + /* + * Various "current state", notably line numbers and what + * file (and how) we're patching right now.. The "is_xxxx" + * things are flags, where -1 means "don't know yet". + */ + int linenr; + + /* + * Records filenames that have been touched, in order to handle + * the case where more than one patches touch the same file. + */ + struct string_list fn_table; + + int p_value; + int p_value_known; + unsigned int p_context; + + const char *whitespace_option; + int whitespace_error; + int squelch_whitespace_errors; + int applied_after_fixing_ws; + + enum ws_error_action ws_error_action; + enum ws_ignore ws_ignore_action; +}; + +#endif diff --git a/builtin/apply.c b/builtin/apply.c index 58bcfeb..e42c8fe 100644 --- a/builtin/apply.c +++ b/builtin/apply.c @@ -20,117 +20,7 @@ #include "xdiff-interface.h" #include "ll-merge.h" #include "rerere.h" - -enum ws_error_action { - nowarn_ws_error, - warn_on_ws_error, - die_on_ws_error, - correct_ws_error -}; - - -enum ws_ignore { - ignore_ws_none, - ignore_ws_change -}; - -/* - * We need to keep track of how symlinks in the preimage are - * manipulated by the patches. A patch to add a/b/c where a/b - * is a symlink should not be allowed to affect the directory - * the symlink points at, but if the same patch removes a/b, - * it is perfectly fine, as the patch removes a/b to make room - * to create a directory a/b so that a/b/c can be created. - * - * See also "struct string_list symlink_changes" in "struct - * apply_state". - */ -#define SYMLINK_GOES_AWAY 01 -#define SYMLINK_IN_RESULT 02 - -struct apply_state { - const char *prefix; - int prefix_length; - - /* - * Since lockfile.c keeps a linked list of all created - * lock_file structures, it isn't safe to free(lock_file). - */ - struct lock_file *lock_file; - int newfd; - - int apply; - int allow_overlap; - int apply_in_reverse; - int apply_with_reject; - int apply_verbosely; - - /* --cached updates only the cache without ever touching the working tree. */ - int cached; - - /* --stat does just a diffstat, and doesn't actually apply */ - int diffstat; - - /* --numstat does numeric diffstat, and doesn't actually apply */ - int numstat; - - int summary; - int threeway; - int no_add; - const char *fake_ancestor; - const char *patch_input_file; - struct string_list limit_by_name; - int has_include; - struct strbuf root; - struct string_list symlink_changes; - - /* - * --check turns on checking that the working tree matches the - * files that are being modified, but doesn't apply the patch - */ - int check; - - /* --index updates the cache as well. */ - int check_index; - - int unidiff_zero; - int update_index; - int unsafe_paths; - int line_termination; - - /* - * For "diff-stat" like behaviour, we keep track of the biggest change - * we've seen, and the longest filename. That allows us to do simple - * scaling. - */ - int max_change; - int max_len; - - /* - * Various "current state", notably line numbers and what - * file (and how) we're patching right now.. The "is_xxxx" - * things are flags, where -1 means "don't know yet". - */ - int linenr; - - /* - * Records filenames that have been touched, in order to handle - * the case where more than one patches touch the same file. - */ - struct string_list fn_table; - - int p_value; - int p_value_known; - unsigned int p_context; - - const char *whitespace_option; - int whitespace_error; - int squelch_whitespace_errors; - int applied_after_fixing_ws; - - enum ws_error_action ws_error_action; - enum ws_ignore ws_ignore_action; -}; +#include "apply.h" static const char * const apply_usage[] = { N_("git apply [<options>] [<patch>...]"), -- 2.8.2.490.g3dabe57 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html