Skip to content

Commit

Permalink
Merge branch 'lt/apply'
Browse files Browse the repository at this point in the history
* 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.
  • Loading branch information
Junio C Hamano committed May 29, 2006
2 parents 3f69d40 + 4868f37 commit cd01d94
Show file tree
Hide file tree
Showing 2 changed files with 70 additions and 0 deletions.
17 changes: 17 additions & 0 deletions builtin-apply.c
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down Expand Up @@ -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;
Expand Down Expand Up @@ -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.
Expand Down
53 changes: 53 additions & 0 deletions t/t4113-apply-ending.sh
Original file line number Diff line number Diff line change
@@ -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

0 comments on commit cd01d94

Please sign in to comment.