diff options
| author | Thomas Rast <trast@inf.ethz.ch> | 2013-05-30 20:00:22 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2013-06-03 19:51:53 (GMT) |
| commit | dbea72a8c05d12c7dfd6a35376e62b21d5a655b5 (patch) | |
| tree | 92298d4eaefe742337c18158d8eebe4b8f4e588e /sha1_file.c | |
| parent | b1d04bfcf84cc3464d0bdf62b02443d0d55e3f2b (diff) | |
| download | git-dbea72a8c05d12c7dfd6a35376e62b21d5a655b5.zip git-dbea72a8c05d12c7dfd6a35376e62b21d5a655b5.tar.gz git-dbea72a8c05d12c7dfd6a35376e62b21d5a655b5.tar.bz2 | |
sha1_file: silence sha1_loose_object_info
sha1_object_info() returns -1 (OBJ_BAD) if it cannot find the object
for some reason, which suggests that it wants the _caller_ to report
this error. However, part of its work happens in
sha1_loose_object_info, which _does_ report errors itself. This is
doubly strange because:
* packed_object_info(), which is the other half of the duo, does _not_
report this.
* In the event that an object is packed and pruned while
sha1_object_info_extended() goes looking for it, we would
erroneously show the error -- even though the code of the latter
function purports to handle this case gracefully.
* A caller might invoke sha1_object_info() to find the type of an
object even if that object is not known to exist.
Silence this error. The others remain untouched as a corrupt object
is a much more grave error than it merely being absent.
Signed-off-by: Thomas Rast <trast@inf.ethz.ch>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'sha1_file.c')
| -rw-r--r-- | sha1_file.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sha1_file.c b/sha1_file.c index 40b2329..d804e25 100644 --- a/sha1_file.c +++ b/sha1_file.c @@ -2140,7 +2140,7 @@ static int sha1_loose_object_info(const unsigned char *sha1, unsigned long *size map = map_sha1_file(sha1, &mapsize); if (!map) - return error("unable to find %s", sha1_to_hex(sha1)); + return -1; if (unpack_sha1_header(&stream, map, mapsize, hdr, sizeof(hdr)) < 0) status = error("unable to unpack %s header", sha1_to_hex(sha1)); |