diff options
author | Jeff King <peff@peff.net> | 2014-07-13 02:42:00 -0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2014-07-28 10:14:33 -0700 |
commit | fe0444b50b400a2c0b6263e8bd6ba464fea0ed4f (patch) | |
tree | 05a0616e98bb40d5f9077e2b2d4d28579891a528 | |
parent | move setting of object->type to alloc_* functions (diff) | |
download | tgif-fe0444b50b400a2c0b6263e8bd6ba464fea0ed4f.tar.xz |
parse_object_buffer: do not set object type
The only way that "obj" can be non-NULL is if it came from
one of the lookup_* functions. These functions always ensure
that the object has the expected type (and return NULL
otherwise), so there is no need for us to set the type.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r-- | object.c | 2 |
1 files changed, 0 insertions, 2 deletions
@@ -213,8 +213,6 @@ struct object *parse_object_buffer(const unsigned char *sha1, enum object_type t warning("object %s has unknown type id %d", sha1_to_hex(sha1), type); obj = NULL; } - if (obj && obj->type == OBJ_NONE) - obj->type = type; return obj; } |