Christian Couder schrieb:
> +static void unparse_commit_list(struct commit_list *list)
> +{
> + for (; list; list = list->next)
> + unparse_commit(list->item);
> +}
> +
> +void unparse_commit(struct commit *item)
> +{
> + item->object.flags = 0;
> + item->object.used = 0;
> + if (item->object.parsed) {
> + item->object.parsed = 0;
> + if (item->parents) {
> + unparse_commit_list(item->parents);
> + free_commit_list(item->parents);
> + item->parents = NULL;
> + }
> + }
> +}
I see a recursion here. Could this not overflow the stack if there is a
long ancestry chain?
-- Hannes
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html