Emily Shaffer <emilyshaffer@xxxxxxxxxx> writes: > diff --git a/hook.c b/hook.c > new file mode 100644 > index 0000000000..d3e28aa73a > --- /dev/null > +++ b/hook.c > @@ -0,0 +1,120 @@ > +#include "cache.h" > + > +#include "hook.h" > +#include "config.h" > + > +void free_hook(struct hook *ptr) Should this be "static" (and be removed from hook.h)? Even after the whole series, nobody outside seems to use this function.