On Thu, 1 Sep 2005, Junio C Hamano wrote:
> Daniel, I do not know what your current status is, but I think
> you need something like this.
Yup, I forgot to actually test that functionality.
> diff --git a/tree.c b/tree.c
> --- a/tree.c
> +++ b/tree.c
> @@ -224,10 +224,12 @@ struct tree *parse_tree_indirect(const u
> if (obj->type == tree_type)
> return (struct tree *) obj;
> else if (obj->type == commit_type)
> - return ((struct commit *) obj)->tree;
> + obj = (struct object *)(((struct commit *) obj)->tree);
obj = &((struct commit *) obj)->tree->object;
Multiple sequential casts always bother me, and we do actually have a
field for this.
> else if (obj->type == tag_type)
> - obj = ((struct tag *) obj)->tagged;
> + obj = deref_tag(obj);
Shouldn't be necessary (once you've got the parse_object below); we're
already in a loop dereferencing things.
> return NULL;
> + if (!obj->parsed)
> + parse_object(obj->sha1);
> } while (1);
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html