undef/git-commit-vandalism
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge branch 'pb/trim-trailing-spaces'

Browse Source 
Fix an error in parsing of .gitignore files that use a trailing
"\ " to mark pathnames that end with a SP.

* pb/trim-trailing-spaces:
  dir.c:trim_trailing_spaces(): fix for " \ " sequence
This commit is contained in:
Junio C Hamano 2014-06-09 11:27:47 -07:00
commit 53b4d8387b
2 changed files with 41 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
dir.c
View File

@ -508,21 +508,25 @@ void clear_exclude_list(struct exclude_list *el)
static void trim_trailing_spaces(char *buf)
{
int i, last_space = -1, nr_spaces, len = strlen(buf);
for (i = 0; i < len; i++)
if (buf[i] == '\\')
i++;
else if (buf[i] == ' ') {
if (last_space == -1) {
last_space = i;
nr_spaces = 1;
} else
nr_spaces++;
} else
last_space = -1;
char *p, *last_space = NULL;
if (last_space != -1 && last_space + nr_spaces == len)
buf[last_space] = '\0';
for (p = buf; *p; p++)
switch (*p) {
case ' ':
if (!last_space)
last_space = p;
break;
case '\\':
p++;
if (!*p)
return;
/* fallthrough */
default:
last_space = NULL;
}
if (last_space)
*last_space = '\0';
}
int add_excludes_from_file_to_list(const char *fname,

23
t/t0008-ignores.sh
View File

@ -806,4 +806,27 @@ test_expect_success !MINGW 'quoting allows trailing whitespace' '
test_cmp err.expect err
'
test_expect_success NOT_MINGW,NOT_CYGWIN 'correct handling of backslashes' '
rm -rf whitespace &&
mkdir whitespace &&
>"whitespace/trailing 1 " &&
>"whitespace/trailing 2 \\\\" &&
>"whitespace/trailing 3 \\\\" &&
>"whitespace/trailing 4 \\ " &&
>"whitespace/trailing 5 \\ \\ " &&
>"whitespace/trailing 6 \\a\\" &&
>whitespace/untracked &&
echo "whitespace/trailing 1 \\ " >ignore &&
echo "whitespace/trailing 2 \\\\\\\\\\\\\\\\" >>ignore &&
echo "whitespace/trailing 3 \\\\\\\\\\\\\\\\ " >>ignore &&
echo "whitespace/trailing 4 \\\\\\\\\\\\ " >>ignore &&
echo "whitespace/trailing 5 \\\\\\\\ \\\\\\\\\\\\ " >>ignore &&
echo "whitespace/trailing 6 \\\\\\\\a\\\\\\\\" >>ignore &&
echo whitespace/untracked >expect &&
>err.expect &&
git ls-files -o -X ignore whitespace >actual 2>err &&
test_cmp expect actual &&
test_cmp err.expect err
'
test_done