diff options
author | Junio C Hamano <gitster@pobox.com> | 2012-11-20 10:32:10 -0800 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2012-11-20 10:32:10 -0800 |
commit | 10022a6d02f10afada7990f232d184a7ebe82b90 (patch) | |
tree | d976802bd99cda8d9c6c59142c7ff644e0db69e8 /builtin/diff.c | |
parent | a9bb4e55a3a552a705f8a6222e4523ed27bb7cc8 (diff) | |
parent | 7349afd20e3b0ce3da6f192f593390472acec9d3 (diff) | |
download | git-10022a6d02f10afada7990f232d184a7ebe82b90.tar.gz |
Merge branch 'kb/preload-index-more'
Use preloadindex in more places, which has a nice speedup on systems
with slow stat calls (and even on Linux).
* kb/preload-index-more:
update-index/diff-index: use core.preloadindex to improve performance
Diffstat (limited to 'builtin/diff.c')
-rw-r--r-- | builtin/diff.c | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/builtin/diff.c b/builtin/diff.c index 9c70e40809..8c2af6cb43 100644 --- a/builtin/diff.c +++ b/builtin/diff.c @@ -130,8 +130,6 @@ static int builtin_diff_index(struct rev_info *revs, usage(builtin_diff_usage); argv++; argc--; } - if (!cached) - setup_work_tree(); /* * Make sure there is one revision (i.e. pending object), * and there is no revision filtering parameters. @@ -140,8 +138,14 @@ static int builtin_diff_index(struct rev_info *revs, revs->max_count != -1 || revs->min_age != -1 || revs->max_age != -1) usage(builtin_diff_usage); - if (read_cache_preload(revs->diffopt.pathspec.raw) < 0) { - perror("read_cache_preload"); + if (!cached) { + setup_work_tree(); + if (read_cache_preload(revs->diffopt.pathspec.raw) < 0) { + perror("read_cache_preload"); + return -1; + } + } else if (read_cache() < 0) { + perror("read_cache"); return -1; } return run_diff_index(revs, cached); |