threads / patch / 46665

patchsha1-lookup: remove sha1_entry_pos() from header file

Subject: [PATCH] sha1-lookup: remove sha1_entry_pos() from header file

## tl;dr

2 messages between Aug 25, 2017 and Aug 25, 2017. Diffs are folded; open one to read it.

replies: 1people: 2as markdown or json

Christian Couder· Aug 25, 2017, 07:36 UTC · lore

Since f1068efefe (sha1_file: drop experimental GIT_USE_LOOKUP search, 2017-08-09) the definition of sha1_entry_pos() has been removed from "sha1-lookup.c", so there is no need anymore for its declaration in "sha1-lookup.h".

Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
---
 sha1-lookup.h | 6 ------
 1 file changed, 6 deletions(-)
Show changes to sha1-lookup.h +0 −6
diff --git a/sha1-lookup.h b/sha1-lookup.h
index 20af285681..cf5314f402 100644
--- a/sha1-lookup.h
+++ b/sha1-lookup.h
@@ -7,10 +7,4 @@ extern int sha1_pos(const unsigned char *sha1,
 		    void *table,
 		    size_t nr,
 		    sha1_access_fn fn);
-
-extern int sha1_entry_pos(const void *table,
-			  size_t elem_size,
-			  size_t key_offset,
-			  unsigned lo, unsigned hi, unsigned nr,
-			  const unsigned char *key);
 #endif
-- 
2.14.1.326.g3dc57ebfbd.dirty
Jeff King· Aug 25, 2017, 14:03 UTC · re: Christian Couder · lore

Re: [PATCH] sha1-lookup: remove sha1_entry_pos() from header file

On Fri, Aug 25, 2017 at 09:36:13AM +0200, Christian Couder wrote:
> Since f1068efefe (sha1_file: drop experimental GIT_USE_LOOKUP search, 2017-08-09)
> the definition of sha1_entry_pos() has been removed from "sha1-lookup.c", so
> there is no need anymore for its declaration in "sha1-lookup.h".
Doh, thanks for catching.
Patch is obviously good.
-Peff

← back to recent threads