diff options
| author | Jeff King <peff@peff.net> | 2024-10-25 09:00:55 +0200 |
|---|---|---|
| committer | Taylor Blau <me@ttaylorr.com> | 2024-10-25 23:35:46 +0200 |
| commit | c2dc4c9fbb1f119be6ab55ff8676bf18b4b9446a (patch) | |
| tree | b32a953088493f598aea4023c1cb682b11e1b1d8 /packfile.h | |
| parent | packfile: drop has_pack_index() (diff) | |
| download | git-c2dc4c9fbb1f119be6ab55ff8676bf18b4b9446a.tar.xz git-c2dc4c9fbb1f119be6ab55ff8676bf18b4b9446a.zip | |
packfile: drop sha1_pack_name()
The sha1_pack_name() function has a few ugly bits:
- it writes into a static strbuf (and not even a ring buffer of them),
which can lead to subtle invalidation problems
- it uses the term "sha1", but it's really using the_hash_algo, which
could be sha256
There's only one caller of it left. And in fact that caller is better
off using the underlying odb_pack_name() function itself, since it's
just copying the result into its own strbuf anyway.
Converting that caller lets us get rid of this now-obselete function.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Taylor Blau <me@ttaylorr.com>
Diffstat (limited to 'packfile.h')
| -rw-r--r-- | packfile.h | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/packfile.h b/packfile.h index b4df3546a3..2bbcc58571 100644 --- a/packfile.h +++ b/packfile.h @@ -32,13 +32,6 @@ struct pack_entry { char *odb_pack_name(struct strbuf *buf, const unsigned char *sha1, const char *ext); /* - * Return the name of the (local) packfile with the specified sha1 in - * its name. The return value is a pointer to memory that is - * overwritten each time this function is called. - */ -char *sha1_pack_name(const unsigned char *sha1); - -/* * Return the name of the (local) pack index file with the specified * sha1 in its name. The return value is a pointer to memory that is * overwritten each time this function is called. |