summaryrefslogtreecommitdiff
path: root/attr.c
diff options
context:
space:
mode:
authorJunio C Hamano <gitster@pobox.com>2012-08-21 14:52:07 -0700
committerJunio C Hamano <gitster@pobox.com>2012-08-21 14:52:07 -0700
commit55b38a48e2a7ccfaaa7897a5fccb98327fa0e3c0 (patch)
tree54ba6e46f12bccd1af9faf258d9826a8001b22e6 /attr.c
parent11e50b2736f374f1608e0c5690405be1a74aa16d (diff)
downloadgit-55b38a48e2a7ccfaaa7897a5fccb98327fa0e3c0.tar.gz
warn_on_inaccessible(): a helper to warn on inaccessible paths
The previous series introduced warnings to multiple places, but it could become tiring to see the warning on the same path over and over again during a single run of Git. Making just one function responsible for issuing this warning, we could later choose to keep track of which paths we issued a warning (it would involve a hash table of paths after running them through real_path() or something) in order to reduce noise. Right now we do not know if the noise reduction is necessary, but it still would be a good code reduction/sharing anyway. Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'attr.c')
-rw-r--r--attr.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/attr.c b/attr.c
index cab01b8b57..f12c83f80a 100644
--- a/attr.c
+++ b/attr.c
@@ -354,7 +354,7 @@ static struct attr_stack *read_attr_from_file(const char *path, int macro_ok)
if (!fp) {
if (errno != ENOENT)
- warning(_("unable to access '%s': %s"), path, strerror(errno));
+ warn_on_inaccessible(path);
return NULL;
}
res = xcalloc(1, sizeof(*res));