summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLibravatar Jeff King <peff@peff.net>2020-06-23 11:25:01 -0400
committerLibravatar Junio C Hamano <gitster@pobox.com>2020-06-23 19:56:26 -0700
commit6416a865da44ac8e27e5dbcafd6a8b76caf09f5a (patch)
treee89e3b3029fabdd4bb77c90955e5496b7fed247c
parentfast-export: use a flex array to store anonymized entries (diff)
downloadtgif-6416a865da44ac8e27e5dbcafd6a8b76caf09f5a.tar.xz
fast-export: move global "idents" anonymize hashmap into function
All of the other anonymization functions keep their static mappings inside the function to avoid polluting the global namespace. Let's do the same for "idents", as nobody needs it outside of anonymize_ident_line(). Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r--builtin/fast-export.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/builtin/fast-export.c b/builtin/fast-export.c
index 99d4a2b404..16a1563e49 100644
--- a/builtin/fast-export.c
+++ b/builtin/fast-export.c
@@ -555,7 +555,6 @@ static char *anonymize_commit_message(const char *old)
return xstrfmt("subject %d\n\nbody\n", counter++);
}
-static struct hashmap idents;
static char *anonymize_ident(const char *old, size_t len)
{
static int counter;
@@ -572,6 +571,7 @@ static char *anonymize_ident(const char *old, size_t len)
*/
static void anonymize_ident_line(const char **beg, const char **end)
{
+ static struct hashmap idents;
static struct strbuf buffers[] = { STRBUF_INIT, STRBUF_INIT };
static unsigned which_buffer;