diff options
| author | Junio C Hamano <gitster@pobox.com> | 2015-01-16 13:21:13 -0800 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2015-01-22 12:57:24 -0800 |
| commit | 0a80bc9f13bf4373778e787b3a050f5b784faf30 (patch) | |
| tree | de8383662567b2502a3b4bac5e9462f374626544 | |
| parent | 407a792ef7e3c142a47c6b09a80f9ede4e14f444 (diff) | |
| download | git-jc/apply-ws-fix-expands-report.tar.gz | |
apply: detect and mark whitespace errors in context lines when fixingjc/apply-ws-fix-expands-report
When the incoming patch has whitespace errors in a common context
line (i.e. a line that is expected to be found and is not modified
by the patch), "apply --whitespace=fix" corrects the whitespace
errors the line has, in addition to the whitespace error on a line
that is updated by the patch. However, we did not count and report
that we fixed whitespace errors on such lines.
[jc: This is iffy. What if the whitespace error has been fixed in
the target since the patch was written? A common context line we
see in the patch has errors, and it matches a line in the target
that has the errors already corrected, resulting in no change, which
we may not want to count after all. On the other hand, we are
reporting whitespace errors _in_ the incoming patch, so...]
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rw-r--r-- | builtin/apply.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/builtin/apply.c b/builtin/apply.c index e8953404d2..a51a1b0872 100644 --- a/builtin/apply.c +++ b/builtin/apply.c @@ -1606,6 +1606,9 @@ static int parse_fragment(const char *line, unsigned long size, if (!deleted && !added) leading++; trailing++; + if (!apply_in_reverse && + ws_error_action == correct_ws_error) + check_whitespace(line, len, patch->ws_rule); break; case '-': if (apply_in_reverse && |