summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeff King <peff@peff.net>2016-08-11 05:25:32 -0400
committerJunio C Hamano <gitster@pobox.com>2016-08-11 10:43:24 -0700
commitca79c985727653656443da8d92c9f5c2b4da93ed (patch)
tree2d96eeda177a2e92f3cffa244463ab9782b7e84f
parent27b5c1a0653026645bd6908ee5abfdceb1c95082 (diff)
downloadgit-ca79c985727653656443da8d92c9f5c2b4da93ed.tar.gz
sha1_file: make packed_object_info public
Some code may have a pack/offset pair for an object, but would like to look up more information. Using sha1_object_info() is too heavy-weight; it starts from the sha1 and has to find the pack again (so not only does it waste time, it might not even find the same instance). In some cases, this problem is solved by helpers like get_size_from_delta(), which is used by pack-objects to take a shortcut for objects whose packed representation has already been found. But there's no similar function for getting the object type, for instance. Rather than introduce one, let's just make the whole packed_object_info() available. It is smart enough to spend effort only on the items the caller wants. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r--cache.h1
-rw-r--r--sha1_file.c4
2 files changed, 3 insertions, 2 deletions
diff --git a/cache.h b/cache.h
index 1fa2871b43..1064017d13 100644
--- a/cache.h
+++ b/cache.h
@@ -1550,6 +1550,7 @@ struct object_info {
#define OBJECT_INFO_INIT {NULL}
extern int sha1_object_info_extended(const unsigned char *, struct object_info *, unsigned flags);
+extern int packed_object_info(struct packed_git *pack, off_t offset, struct object_info *);
/* Dumb servers support */
extern int update_server_info(int);
diff --git a/sha1_file.c b/sha1_file.c
index 090cedb8eb..66edb5efcf 100644
--- a/sha1_file.c
+++ b/sha1_file.c
@@ -1970,8 +1970,8 @@ unwind:
goto out;
}
-static int packed_object_info(struct packed_git *p, off_t obj_offset,
- struct object_info *oi)
+int packed_object_info(struct packed_git *p, off_t obj_offset,
+ struct object_info *oi)
{
struct pack_window *w_curs = NULL;
unsigned long size;