summaryrefslogtreecommitdiff
path: root/builtin-count-objects.c
diff options
context:
space:
mode:
authorShawn O. Pearce <spearce@spearce.org>2007-05-30 06:12:28 (GMT)
committerJunio C Hamano <junkio@cox.net>2007-05-30 06:30:11 (GMT)
commiteaa867703927c1f383637979d16c40d996cea240 (patch)
tree482ee4084d7a9b3792129bdbc26227eebf0536b7 /builtin-count-objects.c
parent7ff895c0d229c2c60b73e91b0c389a4e3ce69e46 (diff)
downloadgit-eaa867703927c1f383637979d16c40d996cea240.zip
git-eaa867703927c1f383637979d16c40d996cea240.tar.gz
git-eaa867703927c1f383637979d16c40d996cea240.tar.bz2
Simplify index access condition in count-objects, pack-redundant
My earlier lazy index opening patch changed this condition to check index_data and call open_pack_index if it was NULL. In truth we only care about num_objects. Since open_pack_index does no harm if the index is already open, and all indexes are likely to be closed in this application, the "performance optimization" of inlining the index_data check here was wrong. Signed-off-by: Shawn O. Pearce <spearce@spearce.org> Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 'builtin-count-objects.c')
-rw-r--r--builtin-count-objects.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/builtin-count-objects.c b/builtin-count-objects.c
index ac65e03..4274ec1 100644
--- a/builtin-count-objects.c
+++ b/builtin-count-objects.c
@@ -111,7 +111,7 @@ int cmd_count_objects(int ac, const char **av, const char *prefix)
for (p = packed_git; p; p = p->next) {
if (!p->pack_local)
continue;
- if (!p->index_data && open_pack_index(p))
+ if (open_pack_index(p))
continue;
packed += p->num_objects;
num_pack++;