diff options
| author | Alex Riesen <raa.lkml@gmail.com> | 2007-05-01 02:27:05 +0400 |
|---|---|---|
| committer | Junio C Hamano <junkio@cox.net> | 2007-05-01 03:57:52 +0400 |
| commit | e44b5d106c2934b93cfba2ea7fb2ed1e32edaae2 (patch) | |
| tree | 48e3f8ed4347741b6f9644b23ada1ec9bda77364 /builtin-blame.c | |
| parent | 8503ee4394dd47af136019bd8da53e5adec17e5d (diff) | |
Remove pointless calls to access(2) when checking for .mailmap
read_mailmap already returns not 0 in case of error, and nothing
seem to be interested in it. It also is silent about the fact
(read_mailmap being to chatty would justify the call to access,
but there is no point for it to be and it isn't).
Signed-off-by: Alex Riesen <raa.lkml@gmail.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 'builtin-blame.c')
| -rw-r--r-- | builtin-blame.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/builtin-blame.c b/builtin-blame.c index 682edba9e5..3442d282aa 100644 --- a/builtin-blame.c +++ b/builtin-blame.c @@ -2375,7 +2375,7 @@ int cmd_blame(int argc, const char **argv, const char *prefix) die("reading graft file %s failed: %s", revs_file, strerror(errno)); - if (!no_mailmap && !access(".mailmap", R_OK)) + if (!no_mailmap) read_mailmap(&mailmap, ".mailmap", NULL); assign_blame(&sb, &revs, opt); |