From: Jonathan Nieder Date: Thu, 16 Jan 2014 00:16:07 GMT Subject: Re: [PATCH v2 1/2] revision: mark contents of an uninteresting tree uninteresting Message-ID: <20140116001607.GN18964@google.com> In-Reply-To: <1389830384-22851-2-git-send-email-gitster@pobox.com> Junio C Hamano wrote: > we see the top-level tree marked as uninteresting (i.e. ^A^{tree} in > the above example) and call mark_tree_uninteresting() on it; this > unfortunately prevents us from recursing into the tree and marking > the objects in the tree as uninteresting. So the tree is marked uninteresting twice --- once by setting in the UNINTERESTING flag in handle_revision_arg() and a second attempted time in mark_tree_uninteresting()? Makes sense. Reviewed-by: Jonathan Nieder