mirror of
https://git.proxmox.com/git/libgit2
synced 2025-08-12 02:57:24 +00:00
Merge pull request #1269 from arrbee/tree-iterator-case-sensitivity-bug
Fix case sensitivity bug with tree iterators
This commit is contained in:
commit
f0f073b740
@ -255,15 +255,17 @@ static int tree_iterator__icase_map_cmp(const void *a, const void *b, void *data
|
||||
git_tree *tree = data;
|
||||
const git_tree_entry *te1 = git_tree_entry_byindex(tree, (size_t)a);
|
||||
const git_tree_entry *te2 = git_tree_entry_byindex(tree, (size_t)b);
|
||||
|
||||
return te1 ? (te2 ? git_tree_entry_icmp(te1, te2) : 1) : -1;
|
||||
}
|
||||
|
||||
static int tree_iterator__frame_start_icmp(const void *key, const void *element)
|
||||
static int tree_iterator__frame_start_icmp(const void *key, const void *el)
|
||||
{
|
||||
const tree_iterator_frame *tf = (const tree_iterator_frame *)key;
|
||||
const git_tree_entry *te = git_tree_entry_byindex(tf->tree, (size_t)element);
|
||||
const git_tree_entry *te = git_tree_entry_byindex(tf->tree, (size_t)el);
|
||||
size_t minlen = min(tf->startlen, te->filename_len);
|
||||
|
||||
return memcmp(tf->start, te->filename, min(tf->startlen, te->filename_len));
|
||||
return git__strncasecmp(tf->start, te->filename, minlen);
|
||||
}
|
||||
|
||||
static void tree_iterator__frame_seek_start(tree_iterator_frame *tf)
|
||||
|
@ -843,8 +843,22 @@ void test_diff_iterator__tree_handles_icase_range(void)
|
||||
|
||||
check_tree_range(repo, "B", "C", false, 0);
|
||||
check_tree_range(repo, "B", "C", true, 1);
|
||||
check_tree_range(repo, "b", "c", false, 1);
|
||||
check_tree_range(repo, "b", "c", true, 1);
|
||||
|
||||
check_tree_range(repo, "a", "z", false, 3);
|
||||
check_tree_range(repo, "a", "z", true, 4);
|
||||
check_tree_range(repo, "A", "Z", false, 1);
|
||||
check_tree_range(repo, "A", "Z", true, 4);
|
||||
check_tree_range(repo, "a", "Z", false, 0);
|
||||
check_tree_range(repo, "a", "Z", true, 4);
|
||||
check_tree_range(repo, "A", "z", false, 4);
|
||||
check_tree_range(repo, "A", "z", true, 4);
|
||||
|
||||
check_tree_range(repo, "new.txt", "new.txt", true, 1);
|
||||
check_tree_range(repo, "new.txt", "new.txt", false, 1);
|
||||
check_tree_range(repo, "README", "README", true, 1);
|
||||
check_tree_range(repo, "README", "README", false, 1);
|
||||
}
|
||||
|
||||
static void check_index_range(
|
||||
|
Loading…
Reference in New Issue
Block a user