From fe3403c3205de44faa926a086b4b99f157fc63fe Mon Sep 17 00:00:00 2001 From: Junio C Hamano Date: Sat, 23 Feb 2008 16:59:16 -0800 Subject: ws_fix_copy(): move the whitespace fixing function to ws.c This is used by git-apply but we can use it elsewhere by slightly generalizing it. Signed-off-by: Junio C Hamano --- builtin-apply.c | 111 ++------------------------------------------------------ 1 file changed, 3 insertions(+), 108 deletions(-) (limited to 'builtin-apply.c') diff --git a/builtin-apply.c b/builtin-apply.c index 5ed4e918c..64471a27e 100644 --- a/builtin-apply.c +++ b/builtin-apply.c @@ -1515,110 +1515,6 @@ static int read_old_data(struct stat *st, const char *path, struct strbuf *buf) } } -static int copy_wsfix(char *output, const char *patch, int plen, - unsigned ws_rule, int count_error) -{ - /* - * plen is number of bytes to be copied from patch, starting - * at patch. Typically patch[plen-1] is '\n', unless this is - * the incomplete last line. - */ - int i; - int add_nl_to_tail = 0; - int add_cr_to_tail = 0; - int fixed = 0; - int last_tab_in_indent = -1; - int last_space_in_indent = -1; - int need_fix_leading_space = 0; - char *buf; - - /* - * Strip trailing whitespace - */ - if ((ws_rule & WS_TRAILING_SPACE) && - (2 < plen && isspace(patch[plen-2]))) { - if (patch[plen - 1] == '\n') { - add_nl_to_tail = 1; - plen--; - if (1 < plen && patch[plen - 1] == '\r') { - add_cr_to_tail = !!(ws_rule & WS_CR_AT_EOL); - plen--; - } - } - if (0 < plen && isspace(patch[plen - 1])) { - while (0 < plen && isspace(patch[plen-1])) - plen--; - fixed = 1; - } - } - - /* - * Check leading whitespaces (indent) - */ - for (i = 0; i < plen; i++) { - char ch = patch[i]; - if (ch == '\t') { - last_tab_in_indent = i; - if ((ws_rule & WS_SPACE_BEFORE_TAB) && - 0 <= last_space_in_indent) - need_fix_leading_space = 1; - } else if (ch == ' ') { - last_space_in_indent = i; - if ((ws_rule & WS_INDENT_WITH_NON_TAB) && - 8 <= i - last_tab_in_indent) - need_fix_leading_space = 1; - } else - break; - } - - buf = output; - if (need_fix_leading_space) { - /* Process indent ourselves */ - int consecutive_spaces = 0; - int last = last_tab_in_indent + 1; - - if (ws_rule & WS_INDENT_WITH_NON_TAB) { - /* have "last" point at one past the indent */ - if (last_tab_in_indent < last_space_in_indent) - last = last_space_in_indent + 1; - else - last = last_tab_in_indent + 1; - } - - /* - * between patch[0..last-1], strip the funny spaces, - * updating them to tab as needed. - */ - for (i = 0; i < last; i++) { - char ch = patch[i]; - if (ch != ' ') { - consecutive_spaces = 0; - *output++ = ch; - } else { - consecutive_spaces++; - if (consecutive_spaces == 8) { - *output++ = '\t'; - consecutive_spaces = 0; - } - } - } - while (0 < consecutive_spaces--) - *output++ = ' '; - plen -= last; - patch += last; - fixed = 1; - } - - memcpy(output, patch, plen); - if (add_cr_to_tail) - output[plen++] = '\r'; - if (add_nl_to_tail) - output[plen++] = '\n'; - if (fixed && count_error) - applied_after_fixing_ws++; - return output + plen - buf; -} - static void update_pre_post_images(struct image *preimage, struct image *postimage, char *buf, @@ -1740,14 +1636,14 @@ static int match_fragment(struct image *img, int match; /* Try fixing the line in the preimage */ - fixlen = copy_wsfix(buf, orig, oldlen, ws_rule, 0); + fixlen = ws_fix_copy(buf, orig, oldlen, ws_rule, NULL); /* Try fixing the line in the target */ if (sizeof(tgtfixbuf) < tgtlen) tgtfix = tgtfixbuf; else tgtfix = xmalloc(tgtlen); - tgtfixlen = copy_wsfix(tgtfix, target, tgtlen, ws_rule, 0); + tgtfixlen = ws_fix_copy(tgtfix, target, tgtlen, ws_rule, NULL); /* * If they match, either the preimage was based on @@ -2006,8 +1902,7 @@ static int apply_one_fragment(struct image *img, struct fragment *frag, added = plen; } else { - added = copy_wsfix(new, patch + 1, plen, - ws_rule, 1); + added = ws_fix_copy(new, patch + 1, plen, ws_rule, &applied_after_fixing_ws); } add_line_info(&postimage, new, added, (first == '+' ? 0 : LINE_COMMON)); -- cgit v1.2.1