Merge branch 'lt/apply'
* lt/apply: apply: force matching at the beginning. Add a test-case for git-apply trying to add an ending line apply: treat EOF as proper context.
This commit is contained in:
commit
cd01d9445d
@ -1335,6 +1335,7 @@ static int apply_line(char *output, const char *patch, int plen)
|
||||
|
||||
static int apply_one_fragment(struct buffer_desc *desc, struct fragment *frag)
|
||||
{
|
||||
int match_beginning, match_end;
|
||||
char *buf = desc->buffer;
|
||||
const char *patch = frag->patch;
|
||||
int offset, size = frag->size;
|
||||
@ -1397,10 +1398,22 @@ static int apply_one_fragment(struct buffer_desc *desc, struct fragment *frag)
|
||||
newlines = new;
|
||||
leading = frag->leading;
|
||||
trailing = frag->trailing;
|
||||
|
||||
/*
|
||||
* If we don't have any leading/trailing data in the patch,
|
||||
* we want it to match at the beginning/end of the file.
|
||||
*/
|
||||
match_beginning = !leading && (frag->oldpos == 1);
|
||||
match_end = !trailing;
|
||||
|
||||
lines = 0;
|
||||
pos = frag->newpos;
|
||||
for (;;) {
|
||||
offset = find_offset(buf, desc->size, oldlines, oldsize, pos, &lines);
|
||||
if (match_end && offset + oldsize != desc->size)
|
||||
offset = -1;
|
||||
if (match_beginning && offset)
|
||||
offset = -1;
|
||||
if (offset >= 0) {
|
||||
int diff = newsize - oldsize;
|
||||
unsigned long size = desc->size + diff;
|
||||
@ -1430,6 +1443,10 @@ static int apply_one_fragment(struct buffer_desc *desc, struct fragment *frag)
|
||||
/* Am I at my context limits? */
|
||||
if ((leading <= p_context) && (trailing <= p_context))
|
||||
break;
|
||||
if (match_beginning || match_end) {
|
||||
match_beginning = match_end = 0;
|
||||
continue;
|
||||
}
|
||||
/* Reduce the number of context lines
|
||||
* Reduce both leading and trailing if they are equal
|
||||
* otherwise just reduce the larger context.
|
||||
|
53
t/t4113-apply-ending.sh
Executable file
53
t/t4113-apply-ending.sh
Executable file
@ -0,0 +1,53 @@
|
||||
#!/bin/sh
|
||||
#
|
||||
# Copyright (c) 2006 Catalin Marinas
|
||||
#
|
||||
|
||||
test_description='git-apply trying to add an ending line.
|
||||
|
||||
'
|
||||
. ./test-lib.sh
|
||||
|
||||
# setup
|
||||
|
||||
cat >test-patch <<\EOF
|
||||
diff --git a/file b/file
|
||||
--- a/file
|
||||
+++ b/file
|
||||
@@ -1,2 +1,3 @@
|
||||
a
|
||||
b
|
||||
+c
|
||||
EOF
|
||||
|
||||
echo 'a' >file
|
||||
echo 'b' >>file
|
||||
echo 'c' >>file
|
||||
|
||||
test_expect_success setup \
|
||||
'git-update-index --add file'
|
||||
|
||||
# test
|
||||
|
||||
test_expect_failure 'apply at the end' \
|
||||
'git-apply --index test-patch'
|
||||
|
||||
cat >test-patch <<\EOF
|
||||
diff a/file b/file
|
||||
--- a/file
|
||||
+++ b/file
|
||||
@@ -1,2 +1,3 @@
|
||||
+a
|
||||
b
|
||||
c
|
||||
EOF
|
||||
|
||||
echo >file 'a
|
||||
b
|
||||
c'
|
||||
git-update-index file
|
||||
|
||||
test_expect_failure 'apply at the beginning' \
|
||||
'git-apply --index test-patch'
|
||||
|
||||
test_done
|
Loading…
Reference in New Issue
Block a user