aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJunio C Hamano <junkio@cox.net>2007-04-01 21:13:27 -0700
committerJunio C Hamano <junkio@cox.net>2007-04-05 15:07:16 -0700
commit0424138d5715dbf8605bacfbd22210aac85a1a8f (patch)
treed3e540a33cad4074ceb50f04e8b5a2b3bb6c5fc8
parentb18825876ae60e67e646097945c8365779175ee2 (diff)
downloadgit-0424138d5715dbf8605bacfbd22210aac85a1a8f.tar.gz
git-0424138d5715dbf8605bacfbd22210aac85a1a8f.tar.xz
Fix bogus error message from merge-recursive error path
This error message should not usually trigger, but the function make_cache_entry() called by add_cacheinfo() can return early without calling into refresh_cache_entry() that sets cache_errno. Also the error message had a wrong function name reported, and it did not say anything about which path failed either. Signed-off-by: Junio C Hamano <junkio@cox.net>
-rw-r--r--cache.h1
-rw-r--r--merge-recursive.c2
-rw-r--r--read-cache.c2
3 files changed, 2 insertions, 3 deletions
diff --git a/cache.h b/cache.h
index 1f128d55d..467e6a579 100644
--- a/cache.h
+++ b/cache.h
@@ -128,7 +128,6 @@ static inline unsigned int ce_mode_from_stat(struct cache_entry *ce, unsigned in
extern struct cache_entry **active_cache;
extern unsigned int active_nr, active_alloc, active_cache_changed;
extern struct cache_tree *active_cache_tree;
-extern int cache_errno;
enum object_type {
OBJ_BAD = -1,
diff --git a/merge-recursive.c b/merge-recursive.c
index e1aebd772..3611a2bdb 100644
--- a/merge-recursive.c
+++ b/merge-recursive.c
@@ -221,7 +221,7 @@ static int add_cacheinfo(unsigned int mode, const unsigned char *sha1,
struct cache_entry *ce;
ce = make_cache_entry(mode, sha1 ? sha1 : null_sha1, path, stage, refresh);
if (!ce)
- return error("cache_addinfo failed: %s", strerror(cache_errno));
+ return error("addinfo_cache failed for path '%s'", path);
return add_cache_entry(ce, options);
}
diff --git a/read-cache.c b/read-cache.c
index 6339a278d..8a7506d90 100644
--- a/read-cache.c
+++ b/read-cache.c
@@ -24,7 +24,7 @@ unsigned int active_nr, active_alloc, active_cache_changed;
struct cache_tree *active_cache_tree;
-int cache_errno;
+static int cache_errno;
static void *cache_mmap;
static size_t cache_mmap_size;