aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEmily Shaffer <emilyshaffer@google.com>2021-09-26 21:03:27 +0200
committerJunio C Hamano <gitster@pobox.com>2021-09-27 09:44:54 -0700
commit330155ed8af3b2e050ac74554993ba68d303e8c3 (patch)
treed3223aec6d21a36bfddf3667f3ae6b3146fc5ba3
parent5e3aba33da26803e48b0099c9dabfd327f7f8b8b (diff)
downloadgit-330155ed8af3b2e050ac74554993ba68d303e8c3.tar.gz
hook.c: add a hook_exists() wrapper and use it in bugreport.c
Add a boolean version of the find_hook() function for those callers who are only interested in checking whether the hook exists, not what the path to it is. Signed-off-by: Emily Shaffer <emilyshaffer@google.com> Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r--builtin/bugreport.c2
-rw-r--r--hook.c5
-rw-r--r--hook.h5
3 files changed, 11 insertions, 1 deletions
diff --git a/builtin/bugreport.c b/builtin/bugreport.c
index c30a360d69..a02c2540bb 100644
--- a/builtin/bugreport.c
+++ b/builtin/bugreport.c
@@ -82,7 +82,7 @@ static void get_populated_hooks(struct strbuf *hook_info, int nongit)
}
for (i = 0; i < ARRAY_SIZE(hook); i++)
- if (find_hook(hook[i]))
+ if (hook_exists(hook[i]))
strbuf_addf(hook_info, "%s\n", hook[i]);
}
diff --git a/hook.c b/hook.c
index ba70b31471..55e1145a4b 100644
--- a/hook.c
+++ b/hook.c
@@ -35,3 +35,8 @@ const char *find_hook(const char *name)
}
return path.buf;
}
+
+int hook_exists(const char *name)
+{
+ return !!find_hook(name);
+}
diff --git a/hook.h b/hook.h
index 68624f1605..6aa36fc7ff 100644
--- a/hook.h
+++ b/hook.h
@@ -8,4 +8,9 @@
*/
const char *find_hook(const char *name);
+/**
+ * A boolean version of find_hook()
+ */
+int hook_exists(const char *hookname);
+
#endif