summaryrefslogtreecommitdiff
path: root/compat/obstack.c
diff options
context:
space:
mode:
authorLibravatar Michael Haggerty <mhagger@alum.mit.edu>2012-05-24 14:16:50 +0200
committerLibravatar Junio C Hamano <gitster@pobox.com>2012-05-24 12:16:06 -0700
commit654ad400c27653a138b028cdd35a09b0c69c7ef0 (patch)
tree685f4861910e520476ea2ceca5484a41b1dec364 /compat/obstack.c
parentdo_for_each_ref(): only iterate over the subtree that was requested (diff)
downloadtgif-654ad400c27653a138b028cdd35a09b0c69c7ef0.tar.xz
Avoid sorting if references are added to ref_cache in order
The old code allowed many references to be efficiently added to a single directory, because it just appended the references to the containing directory unsorted without doing any searching (and therefore without requiring any intermediate sorting). But the old code was inefficient when a large number of subdirectories were added to a directory, because the directory always had to be searched to see if the new subdirectory already existed, and this search required the directory to be sorted first. The same was repeated for every new subdirectory, so the time scaled like O(N^2), where N is the number of subdirectories within a single directory. In practice, references are often added to the ref_cache in lexicographic order, for example when reading the packed-refs file. So build some intelligence into add_entry_to_dir() to optimize for the case of references and/or subdirectories being added in lexicographic order: if the existing entries were already sorted, and the new entry comes after the last existing entry, then adjust ref_dir::sorted to reflect the fact that the ref_dir is still sorted. Thanks to Peff for pointing out the performance regression that inspired this change. Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'compat/obstack.c')
0 files changed, 0 insertions, 0 deletions